GNOME.org
 

[glom] Fix for krazy2 warning: Pre-increment instead of post-increment in for().

commit 8f4d015b51495e4449d76a11e1ff546f2be2fff7
Author: Murray Cumming <murrayc murrayc com>
Date:   Mon Dec 14 16:47:25 2009 +0100

    Fix for krazy2 warning: Pre-increment instead of post-increment in for().

 glom.desktop.in.in                                 |    2 +-
 glom/base_db.cc                                    |    2 +-
 glom/base_db_table_data.cc                         |    2 +-
 glom/glom_privs.cc                                 |    2 +-
 glom/import_csv/dialog_import_csv.cc               |    2 +-
 glom/libglom/document/bakery/view/view_composite.h |    6 +++---
 glom/libglom/document/document.cc                  |   12 ++++++------
 glom/main.cc                                       |    2 +-
 glom/mode_data/box_data_details.cc                 |    2 +-
 glom/mode_data/flowtablewithfields.cc              |    2 +-
 glom/mode_design/box_db_table_relationships.cc     |    2 +-
 glom/mode_design/fields/box_db_table_definition.cc |    2 +-
 glom/mode_design/fields/combo_fieldtype.cc         |    2 +-
 glom/mode_design/iso_codes.cc                      |    6 +++---
 glom/mode_design/layout/dialog_choose_field.cc     |    2 +-
 .../layout/dialog_layout_calendar_related.cc       |    2 +-
 glom/mode_design/layout/dialog_layout_details.cc   |    2 +-
 glom/mode_design/layout/dialog_layout_export.cc    |    2 +-
 .../layout/dialog_layout_list_related.cc           |    2 +-
 .../dialog_groupby_secondaryfields.cc              |    2 +-
 .../dialog_groupby_sortfields.cc                   |    2 +-
 .../dialog_relationships_overview.cc               |    2 +-
 .../mode_design/translation/window_translations.cc |    2 +-
 glom/mode_find/box_data_details_find.cc            |    2 +-
 glom/navigation/box_tables.cc                      |    2 +-
 glom/navigation/maemo/pickerbutton_table.cc        |    2 +-
 glom/utility_widgets/adddel/adddel.cc              |    2 +-
 glom/utility_widgets/flowtable.cc                  |    2 +-
 glom/utility_widgets/flowtable_dnd.cc              |    4 ++--
 29 files changed, 39 insertions(+), 39 deletions(-)
---
diff --git a/glom.desktop.in.in b/glom.desktop.in.in
index 410920c..78e9099 100644
--- a/glom.desktop.in.in
+++ b/glom.desktop.in.in
@@ -5,7 +5,7 @@ Exec=glom
 Terminal=false
 Type=Application
 Icon=glom
-Categories=Application;GNOME;Office;GTK;Database
+Categories=Application;GNOME;Office;GTK;Database;
 StartupNotify=true
 MimeType=application/x-glom;
 X-GNOME-DocPath=glom/index.html
diff --git a/glom/base_db.cc b/glom/base_db.cc
index 0f6e9a8..e011682 100644
--- a/glom/base_db.cc
+++ b/glom/base_db.cc
@@ -555,7 +555,7 @@ Base_DB::type_vec_strings Base_DB::util_vecStrings_from_Fields(const type_vec_fi
   //Get vector of field names, suitable for a combo box:
 
   type_vec_strings vecNames;
-  for(type_vec_fields::size_type i = 0; i < fields.size(); i++)
+  for(type_vec_fields::size_type i = 0; i < fields.size(); ++i)
   {
     vecNames.push_back(fields[i]->get_name());
   }
diff --git a/glom/base_db_table_data.cc b/glom/base_db_table_data.cc
index 7a272c7..cb989bc 100644
--- a/glom/base_db_table_data.cc
+++ b/glom/base_db_table_data.cc
@@ -510,7 +510,7 @@ void Base_DB_Table_Data::refresh_related_fields(const LayoutFieldInRecord& field
           std::cerr << "Base_DB_Table_Data::refresh_related_fields(): The result had 0 columns" << std::endl;
         }
 
-        for(guint uiCol = 0; uiCol < cols_count; uiCol++)
+        for(guint uiCol = 0; uiCol < cols_count; ++uiCol)
         {
 #ifdef GLIBMM_EXCEPTIONS_ENABLED        
           const Gnome::Gda::Value value = result->get_value_at(uiCol, 0 /* row */);
diff --git a/glom/glom_privs.cc b/glom/glom_privs.cc
index c63374e..1c08d69 100644
--- a/glom/glom_privs.cc
+++ b/glom/glom_privs.cc
@@ -73,7 +73,7 @@ bool Privs::get_developer_user_exists_with_password()
   const Glib::ustring default_user = get_default_developer_user_name(default_password);
 
   const type_vec_strings users = get_database_users();
-  for(type_vec_strings::const_iterator iter = users.begin(); iter != users.end(); iter++)
+  for(type_vec_strings::const_iterator iter = users.begin(); iter != users.end(); ++iter)
   {
     const Glib::ustring user = *iter;
     if(user == default_user)
diff --git a/glom/import_csv/dialog_import_csv.cc b/glom/import_csv/dialog_import_csv.cc
index 3f9241b..0f0f43e 100644
--- a/glom/import_csv/dialog_import_csv.cc
+++ b/glom/import_csv/dialog_import_csv.cc
@@ -88,7 +88,7 @@ Dialog_Import_CSV::Dialog_Import_CSV(BaseObjectType* cobject, const Glib::RefPtr
   m_encoding_model->append();
 
   const FileEncodings::type_list_encodings list_encodings =  FileEncodings::get_list_of_encodings();
-  for(FileEncodings::type_list_encodings::const_iterator encodings_iter = list_encodings.begin(); encodings_iter  != list_encodings.end(); encodings_iter ++)
+  for(FileEncodings::type_list_encodings::const_iterator encodings_iter = list_encodings.begin(); encodings_iter  != list_encodings.end(); ++encodings_iter)
   {
     const FileEncodings::Encoding encoding = *encodings_iter;
     if(encoding.get_name().empty())
diff --git a/glom/libglom/document/bakery/view/view_composite.h b/glom/libglom/document/bakery/view/view_composite.h
index 8b85d40..6595381 100644
--- a/glom/libglom/document/bakery/view/view_composite.h
+++ b/glom/libglom/document/bakery/view/view_composite.h
@@ -69,7 +69,7 @@ public:
     View<T_Document>::set_document(pDocument);
 
     //Change the document in the child views.
-    for(typename type_vec_views::iterator iter = m_vecViews.begin(); iter != m_vecViews.end(); iter++)
+    for(typename type_vec_views::iterator iter = m_vecViews.begin(); iter != m_vecViews.end(); ++iter)
     {
       type_view* pView = *iter;
       if(pView)
@@ -80,7 +80,7 @@ public:
   virtual void load_from_document()
   {
     //Delegate to the child views:
-    for(typename type_vec_views::iterator iter = m_vecViews.begin(); iter != m_vecViews.end(); iter++)
+    for(typename type_vec_views::iterator iter = m_vecViews.begin(); iter != m_vecViews.end(); ++iter)
     {
       type_view* pView = *iter;
       if(pView)
@@ -91,7 +91,7 @@ public:
   virtual void save_to_document()
   {
     //Delegate to the child views:
-    for(typename type_vec_views::iterator iter = m_vecViews.begin(); iter != m_vecViews.end(); iter++)
+    for(typename type_vec_views::iterator iter = m_vecViews.begin(); iter != m_vecViews.end(); ++iter)
     {
       type_view* pView = *iter;
       if(pView)
diff --git a/glom/libglom/document/document.cc b/glom/libglom/document/document.cc
index 3b647a0..c9f340f 100644
--- a/glom/libglom/document/document.cc
+++ b/glom/libglom/document/document.cc
@@ -1355,7 +1355,7 @@ void Document::set_tables(const type_listTableInfo& tables)
   //Look at each "table":
 
   bool something_changed = false;
-  for(type_tables::iterator iter = m_tables.begin(); iter != m_tables.end(); iter++)
+  for(type_tables::iterator iter = m_tables.begin(); iter != m_tables.end(); ++iter)
   {
     const DocumentTableInfo& doctableinfo = iter->second;
 
@@ -2441,7 +2441,7 @@ bool Document::load_after(int& failure_code)
 
       //Look at each "table" node.
       xmlpp::Node::NodeList listNodes = nodeRoot->get_children(GLOM_NODE_TABLE);
-      for(xmlpp::Node::NodeList::const_iterator iter = listNodes.begin(); iter != listNodes.end(); iter++)
+      for(xmlpp::Node::NodeList::const_iterator iter = listNodes.begin(); iter != listNodes.end(); ++iter)
       {
         xmlpp::Element* nodeTable = dynamic_cast<xmlpp::Element*>(*iter);
         if(nodeTable)
@@ -2470,7 +2470,7 @@ bool Document::load_after(int& failure_code)
           if(nodeRelationships)
           {
             const xmlpp::Node::NodeList listNodes = nodeRelationships->get_children(GLOM_NODE_RELATIONSHIP);
-            for(xmlpp::Node::NodeList::const_iterator iter = listNodes.begin(); iter != listNodes.end(); iter++)
+            for(xmlpp::Node::NodeList::const_iterator iter = listNodes.begin(); iter != listNodes.end(); ++iter)
             {
               const xmlpp::Element* nodeChild = dynamic_cast<xmlpp::Element*>(*iter);
               if(nodeChild)
@@ -2503,7 +2503,7 @@ bool Document::load_after(int& failure_code)
 
             //Loop through Field child nodes:
             xmlpp::Node::NodeList listNodes = nodeFields->get_children(GLOM_NODE_FIELD);
-            for(xmlpp::Node::NodeList::const_iterator iter = listNodes.begin(); iter != listNodes.end(); iter++)
+            for(xmlpp::Node::NodeList::const_iterator iter = listNodes.begin(); iter != listNodes.end(); ++iter)
             {
               const xmlpp::Element* nodeChild = dynamic_cast<xmlpp::Element*>(*iter);
               if(nodeChild)
@@ -2621,7 +2621,7 @@ bool Document::load_after(int& failure_code)
       //Look at each "table" node.
       //We do load the layouts separately, because we needed to load all the tables' relationships and tables 
       //before we can load layouts that can use them.
-      for(xmlpp::Node::NodeList::const_iterator iter = listNodes.begin(); iter != listNodes.end(); iter++)
+      for(xmlpp::Node::NodeList::const_iterator iter = listNodes.begin(); iter != listNodes.end(); ++iter)
       {
         xmlpp::Element* nodeTable = dynamic_cast<xmlpp::Element*>(*iter);
         if(nodeTable)
@@ -3429,7 +3429,7 @@ bool Document::save_before()
         xmlpp::Element* elemRelationships = nodeTable->add_child(GLOM_NODE_RELATIONSHIPS);
 
         //Add each <relationship> node:
-        for(type_vec_relationships::const_iterator iter = doctableinfo.m_relationships.begin(); iter != doctableinfo.m_relationships.end(); iter++)
+        for(type_vec_relationships::const_iterator iter = doctableinfo.m_relationships.begin(); iter != doctableinfo.m_relationships.end(); ++iter)
         {
           sharedptr<const Relationship> relationship = *iter;
           if(relationship)
diff --git a/glom/main.cc b/glom/main.cc
index c64ec8b..dab78a9 100644
--- a/glom/main.cc
+++ b/glom/main.cc
@@ -154,7 +154,7 @@ pgwin32_is_admin(void)
 
   success = false;
 
-  for (x = 0; x < Groups->GroupCount; x++)
+  for (x = 0; x < Groups->GroupCount; ++x)
   {
     if((EqualSid(AdministratorsSid, Groups->Groups[x].Sid) && (Groups->Groups[x].Attributes & SE_GROUP_ENABLED)) ||
       (EqualSid(PowerUsersSid, Groups->Groups[x].Sid) && (Groups->Groups[x].Attributes & SE_GROUP_ENABLED)))
diff --git a/glom/mode_data/box_data_details.cc b/glom/mode_data/box_data_details.cc
index 067e2c3..8af11bb 100644
--- a/glom/mode_data/box_data_details.cc
+++ b/glom/mode_data/box_data_details.cc
@@ -844,7 +844,7 @@ void Box_Data_Details::on_flowtable_field_edited(const sharedptr<const LayoutIte
         /*
         bool bIsForeignKey = false;
         Document::type_vec_relationships vecRelationships = get_document()->get_relationships(m_table_name);
-        for(Document::type_vec_relationships::iterator iter = vecRelationships.begin(); iter != vecRelationships.end(); iter++)
+        for(Document::type_vec_relationships::iterator iter = vecRelationships.begin(); iter != vecRelationships.end(); ++iter)
         {
           const Relationship& relationship = *iter;
 
diff --git a/glom/mode_data/flowtablewithfields.cc b/glom/mode_data/flowtablewithfields.cc
index 7e2a8cc..58d1590 100644
--- a/glom/mode_data/flowtablewithfields.cc
+++ b/glom/mode_data/flowtablewithfields.cc
@@ -1396,7 +1396,7 @@ void FlowTableWithFields::on_dnd_remove_placeholder()
     { 
       LayoutGroup::type_list_items items = layout_group->get_items();
       for (LayoutGroup::type_list_items::iterator item = items.begin();
-           item != items.end(); item++)
+           item != items.end(); ++item)
       {
         sharedptr<LayoutItem_Placeholder> placeholder = 
           sharedptr<LayoutItem_Placeholder>::cast_dynamic(*item);
diff --git a/glom/mode_design/box_db_table_relationships.cc b/glom/mode_design/box_db_table_relationships.cc
index f865399..055c4e1 100644
--- a/glom/mode_design/box_db_table_relationships.cc
+++ b/glom/mode_design/box_db_table_relationships.cc
@@ -99,7 +99,7 @@ bool Box_DB_Table_Relationships::fill_from_database()
     //To Field choices are different for each row: set in on_adddel_signal_user_activated.
 
     //Add the relationships:
-    for(Document::type_vec_relationships::iterator iter = vecRelationships.begin(); iter != vecRelationships.end(); iter++)
+    for(Document::type_vec_relationships::iterator iter = vecRelationships.begin(); iter != vecRelationships.end(); ++iter)
     {
       sharedptr<const Relationship> relationship = *iter;
       if(relationship)
diff --git a/glom/mode_design/fields/box_db_table_definition.cc b/glom/mode_design/fields/box_db_table_definition.cc
index ff74cef..3dd85c7 100644
--- a/glom/mode_design/fields/box_db_table_definition.cc
+++ b/glom/mode_design/fields/box_db_table_definition.cc
@@ -138,7 +138,7 @@ bool Box_DB_Table_Definition::fill_from_database()
 
     Field::type_map_type_names mapFieldTypes = Field::get_type_names_ui();
 
-    for(type_vec_fields::iterator iter = m_vecFields.begin(); iter != m_vecFields.end(); iter++)
+    for(type_vec_fields::iterator iter = m_vecFields.begin(); iter != m_vecFields.end(); ++iter)
     {
       const sharedptr<const Field>& field = *iter;
 
diff --git a/glom/mode_design/fields/combo_fieldtype.cc b/glom/mode_design/fields/combo_fieldtype.cc
index 5c889c4..d13055b 100644
--- a/glom/mode_design/fields/combo_fieldtype.cc
+++ b/glom/mode_design/fields/combo_fieldtype.cc
@@ -45,7 +45,7 @@ void Combo_FieldType::init()
   //Set Type choices:
   Field::type_map_type_names map_names = Field::get_usable_type_names();
     
-  for(Field::type_map_type_names::iterator iter = map_names.begin(); iter != map_names.end(); iter++)
+  for(Field::type_map_type_names::iterator iter = map_names.begin(); iter != map_names.end(); ++iter)
   {
     Gtk::TreeModel::iterator iterModel = m_refTreeModel->append();
     if(iterModel)
diff --git a/glom/mode_design/iso_codes.cc b/glom/mode_design/iso_codes.cc
index 621d735..6010018 100644
--- a/glom/mode_design/iso_codes.cc
+++ b/glom/mode_design/iso_codes.cc
@@ -58,7 +58,7 @@ type_list_currencies get_list_of_currency_symbols()
         const xmlpp::Node* nodeRoot = parser.get_document()->get_root_node(); //deleted by DomParser.
 
         xmlpp::Node::NodeList listNodes = nodeRoot->get_children("iso_4217_entry");
-        for(xmlpp::Node::NodeList::const_iterator iter = listNodes.begin(); iter != listNodes.end(); iter++)
+        for(xmlpp::Node::NodeList::const_iterator iter = listNodes.begin(); iter != listNodes.end(); ++iter)
         {
           xmlpp::Element* nodeEntry = dynamic_cast<xmlpp::Element*>(*iter);
           if(nodeEntry)
@@ -151,7 +151,7 @@ Glib::ustring get_locale_name(const Glib::ustring& locale_id)
         const xmlpp::Node* nodeRoot = parser.get_document()->get_root_node(); //deleted by DomParser.
 
         xmlpp::Node::NodeList listNodes = nodeRoot->get_children("iso_639_entry");
-        for(xmlpp::Node::NodeList::const_iterator iter = listNodes.begin(); iter != listNodes.end(); iter++)
+        for(xmlpp::Node::NodeList::const_iterator iter = listNodes.begin(); iter != listNodes.end(); ++iter)
         {
           xmlpp::Element* nodeEntry = dynamic_cast<xmlpp::Element*>(*iter);
           if(nodeEntry)
@@ -206,7 +206,7 @@ Glib::ustring get_locale_name(const Glib::ustring& locale_id)
         const xmlpp::Node* nodeRoot = parser.get_document()->get_root_node(); //deleted by DomParser.
 
         xmlpp::Node::NodeList listNodes = nodeRoot->get_children("iso_3166_entry");
-        for(xmlpp::Node::NodeList::const_iterator iter = listNodes.begin(); iter != listNodes.end(); iter++)
+        for(xmlpp::Node::NodeList::const_iterator iter = listNodes.begin(); iter != listNodes.end(); ++iter)
         {
           xmlpp::Element* nodeEntry = dynamic_cast<xmlpp::Element*>(*iter);
           if(nodeEntry)
diff --git a/glom/mode_design/layout/dialog_choose_field.cc b/glom/mode_design/layout/dialog_choose_field.cc
index 44fb4a1..43b4967 100644
--- a/glom/mode_design/layout/dialog_choose_field.cc
+++ b/glom/mode_design/layout/dialog_choose_field.cc
@@ -221,7 +221,7 @@ Dialog_ChooseField::type_list_field_items Dialog_ChooseField::get_fields_chosen(
     
   typedef std::list<Gtk::TreeModel::Path> type_list_paths;
   type_list_paths list_paths = refTreeSelection->get_selected_rows();
-  for(type_list_paths::const_iterator iter = list_paths.begin(); iter != list_paths.end(); iter++)
+  for(type_list_paths::const_iterator iter = list_paths.begin(); iter != list_paths.end(); ++iter)
   {
     const Gtk::TreeModel::Path path = *iter;
     Gtk::TreeModel::iterator tree_iter = m_model->get_iter(path);
diff --git a/glom/mode_design/layout/dialog_layout_calendar_related.cc b/glom/mode_design/layout/dialog_layout_calendar_related.cc
index 36066e9..713ab6a 100644
--- a/glom/mode_design/layout/dialog_layout_calendar_related.cc
+++ b/glom/mode_design/layout/dialog_layout_calendar_related.cc
@@ -376,7 +376,7 @@ void Dialog_Layout_Calendar_Related::on_button_add_field()
   //std::cout << "debug table used =" << m_portal->get_table_used(m_table_name) << std::endl;
 
   type_list_field_items fields_list = offer_field_list(m_table_name, this);
-  for(type_list_field_items::iterator iter_chosen = fields_list.begin(); iter_chosen != fields_list.end(); iter_chosen++) 
+  for(type_list_field_items::iterator iter_chosen = fields_list.begin(); iter_chosen != fields_list.end(); ++iter_chosen) 
   {
     sharedptr<LayoutItem_Field> field = *iter_chosen;
     if(!field)
diff --git a/glom/mode_design/layout/dialog_layout_details.cc b/glom/mode_design/layout/dialog_layout_details.cc
index 0da1b69..cc56422 100644
--- a/glom/mode_design/layout/dialog_layout_details.cc
+++ b/glom/mode_design/layout/dialog_layout_details.cc
@@ -516,7 +516,7 @@ void Dialog_Layout_Details::on_button_down()
 void Dialog_Layout_Details::on_button_add_field()
 {
   type_list_field_items fields_list = offer_field_list(m_table_name, this);
-  for(type_list_field_items::iterator iter_chosen = fields_list.begin(); iter_chosen != fields_list.end(); iter_chosen++) 
+  for(type_list_field_items::iterator iter_chosen = fields_list.begin(); iter_chosen != fields_list.end(); ++iter_chosen) 
   {
     sharedptr<LayoutItem_Field> layout_item = *iter_chosen;
     if(!layout_item)
diff --git a/glom/mode_design/layout/dialog_layout_export.cc b/glom/mode_design/layout/dialog_layout_export.cc
index e558351..de298ff 100644
--- a/glom/mode_design/layout/dialog_layout_export.cc
+++ b/glom/mode_design/layout/dialog_layout_export.cc
@@ -258,7 +258,7 @@ void Dialog_Layout_Export::on_button_add_field()
 {
   //Get the chosen fields:
   type_list_field_items fields_list = offer_field_list(m_table_name, this);
-  for(type_list_field_items::iterator iter_chosen = fields_list.begin(); iter_chosen != fields_list.end(); iter_chosen++) 
+  for(type_list_field_items::iterator iter_chosen = fields_list.begin(); iter_chosen != fields_list.end(); ++iter_chosen) 
   {
     sharedptr<LayoutItem_Field> field = *iter_chosen;
     if(!field)
diff --git a/glom/mode_design/layout/dialog_layout_list_related.cc b/glom/mode_design/layout/dialog_layout_list_related.cc
index e968cc3..66a684e 100644
--- a/glom/mode_design/layout/dialog_layout_list_related.cc
+++ b/glom/mode_design/layout/dialog_layout_list_related.cc
@@ -419,7 +419,7 @@ void Dialog_Layout_List_Related::on_button_add_field()
   //std::cout << "debug table used =" << m_portal->get_table_used(m_table_name) << std::endl;
 
   type_list_field_items fields_list = offer_field_list(m_portal->get_table_used(m_table_name), this);
-  for(type_list_field_items::iterator iter_chosen = fields_list.begin(); iter_chosen != fields_list.end(); iter_chosen++) 
+  for(type_list_field_items::iterator iter_chosen = fields_list.begin(); iter_chosen != fields_list.end(); ++iter_chosen) 
   {
     sharedptr<LayoutItem_Field> field = *iter_chosen;
     if(!field)
diff --git a/glom/mode_design/layout/layout_item_dialogs/dialog_groupby_secondaryfields.cc b/glom/mode_design/layout/layout_item_dialogs/dialog_groupby_secondaryfields.cc
index 30b5644..5f09ab7 100644
--- a/glom/mode_design/layout/layout_item_dialogs/dialog_groupby_secondaryfields.cc
+++ b/glom/mode_design/layout/layout_item_dialogs/dialog_groupby_secondaryfields.cc
@@ -217,7 +217,7 @@ void Dialog_GroupBy_SecondaryFields::on_button_add_field()
 {
   //Get the chosen fields:
   type_list_field_items fields_list = offer_field_list(m_table_name, this);
-  for(type_list_field_items::iterator iter_chosen = fields_list.begin(); iter_chosen != fields_list.end(); iter_chosen++) 
+  for(type_list_field_items::iterator iter_chosen = fields_list.begin(); iter_chosen != fields_list.end(); ++iter_chosen) 
   {
     sharedptr<LayoutItem_Field> field = *iter_chosen; 
     if(!field)
diff --git a/glom/mode_design/layout/layout_item_dialogs/dialog_groupby_sortfields.cc b/glom/mode_design/layout/layout_item_dialogs/dialog_groupby_sortfields.cc
index 38cc3d6..01174d7 100644
--- a/glom/mode_design/layout/layout_item_dialogs/dialog_groupby_sortfields.cc
+++ b/glom/mode_design/layout/layout_item_dialogs/dialog_groupby_sortfields.cc
@@ -216,7 +216,7 @@ void Dialog_GroupBy_SortFields::on_button_add_field()
 {
   //Get the chosen fields:
   type_list_field_items fields_list = offer_field_list(m_table_name, this);
-  for(type_list_field_items::iterator iter_chosen = fields_list.begin(); iter_chosen != fields_list.end(); iter_chosen++) 
+  for(type_list_field_items::iterator iter_chosen = fields_list.begin(); iter_chosen != fields_list.end(); ++iter_chosen) 
   {
 
     sharedptr<LayoutItem_Field> field = *iter_chosen;
diff --git a/glom/mode_design/relationships_overview/dialog_relationships_overview.cc b/glom/mode_design/relationships_overview/dialog_relationships_overview.cc
index 7879037..8624bd2 100644
--- a/glom/mode_design/relationships_overview/dialog_relationships_overview.cc
+++ b/glom/mode_design/relationships_overview/dialog_relationships_overview.cc
@@ -227,7 +227,7 @@ void Dialog_RelationshipsOverview::draw_lines()
       Document::type_vec_relationships m_relationships = document->get_relationships(table_name);
       Document::type_vec_fields fields = document->get_table_fields(table_name);
       
-      for(Document::type_vec_relationships::const_iterator rit = m_relationships.begin(); rit != m_relationships.end(); rit++)
+      for(Document::type_vec_relationships::const_iterator rit = m_relationships.begin(); rit != m_relationships.end(); ++rit)
       {
         sharedptr<const Relationship> relationship = *rit;
         if(!relationship)
diff --git a/glom/mode_design/translation/window_translations.cc b/glom/mode_design/translation/window_translations.cc
index 1bc3a12..7ad3d4a 100644
--- a/glom/mode_design/translation/window_translations.cc
+++ b/glom/mode_design/translation/window_translations.cc
@@ -624,7 +624,7 @@ void Window_Translations::on_button_import()
 
       //Look at each domain (could there be more than one?):
       const char* const* domains = po_file_domains(po_file);
-      for (int i = 0; domains[i] != 0; i++)
+      for (int i = 0; domains[i] != 0; ++i)
       {
         //Look at each message:
         po_message_iterator_t iter = po_message_iterator(po_file, domains[i]);
diff --git a/glom/mode_find/box_data_details_find.cc b/glom/mode_find/box_data_details_find.cc
index 53954cc..6b40474 100644
--- a/glom/mode_find/box_data_details_find.cc
+++ b/glom/mode_find/box_data_details_find.cc
@@ -85,7 +85,7 @@ void Box_Data_Details_Find::fill_related()
   Document::type_vec_relationships vecRelationships = get_document()->get_relationships(m_table_name);
 
   //Add the relationships:
-  for(Document::type_vec_relationships::iterator iter = vecRelationships.begin(); iter != vecRelationships.end(); iter++)
+  for(Document::type_vec_relationships::iterator iter = vecRelationships.begin(); iter != vecRelationships.end(); ++iter)
   {
      /*
      const Relationship& relationship = *iter;
diff --git a/glom/navigation/box_tables.cc b/glom/navigation/box_tables.cc
index a73c052..87e3d5b 100644
--- a/glom/navigation/box_tables.cc
+++ b/glom/navigation/box_tables.cc
@@ -169,7 +169,7 @@ bool Box_Tables::fill_from_database()
 
     const type_vec_strings vecTables = get_table_names_from_database();
 
-    for(type_vec_strings::const_iterator iter = vecTables.begin(); iter != vecTables.end(); iter++)
+    for(type_vec_strings::const_iterator iter = vecTables.begin(); iter != vecTables.end(); ++iter)
     {
       const Glib::ustring strName = *iter;
 
diff --git a/glom/navigation/maemo/pickerbutton_table.cc b/glom/navigation/maemo/pickerbutton_table.cc
index 023f222..d19e62e 100644
--- a/glom/navigation/maemo/pickerbutton_table.cc
+++ b/glom/navigation/maemo/pickerbutton_table.cc
@@ -58,7 +58,7 @@ bool PickerButton_Table::fill_from_database()
   Document::type_listTableInfo listTablesDocument = document->get_tables();
 
   const type_vec_strings vecTables = get_table_names_from_database();
-  for(type_vec_strings::const_iterator iter = vecTables.begin(); iter != vecTables.end(); iter++)
+  for(type_vec_strings::const_iterator iter = vecTables.begin(); iter != vecTables.end(); ++iter)
   {
     const Glib::ustring strName = *iter;
     sharedptr<TableInfo> table_info;
diff --git a/glom/utility_widgets/adddel/adddel.cc b/glom/utility_widgets/adddel/adddel.cc
index 7ef3675..21715f3 100644
--- a/glom/utility_widgets/adddel/adddel.cc
+++ b/glom/utility_widgets/adddel/adddel.cc
@@ -1627,7 +1627,7 @@ bool AddDel::row_has_duplicates(const Gtk::TreeModel::iterator& iter) const
       //std::cout << "value_text=" << value_text << std::endl;
 
       //Look at each other row to see whether the value exists there already:
-      for(Gtk::TreeModel::iterator iterCheck = m_refListStore->children().begin(); iterCheck != m_refListStore->children().end(); iterCheck++)
+      for(Gtk::TreeModel::iterator iterCheck = m_refListStore->children().begin(); iterCheck != m_refListStore->children().end(); ++iterCheck)
       {
         if(iterCheck != iter) //Don't compare the row with itself
         {
diff --git a/glom/utility_widgets/flowtable.cc b/glom/utility_widgets/flowtable.cc
index 60d7ec5..02faee4 100644
--- a/glom/utility_widgets/flowtable.cc
+++ b/glom/utility_widgets/flowtable.cc
@@ -309,7 +309,7 @@ void FlowTable::insert_before(FlowTableItem& item, Gtk::Widget& before)
 {
   bool found = false;
   std::vector<FlowTableItem>::iterator pos;
-  for(pos = m_children.begin(); pos != m_children.end(); pos++)
+  for(pos = m_children.begin(); pos != m_children.end(); ++pos)
   {
     FlowTableItem* item = &(*pos);
     if(item->m_first)
diff --git a/glom/utility_widgets/flowtable_dnd.cc b/glom/utility_widgets/flowtable_dnd.cc
index a21d14a..354622f 100644
--- a/glom/utility_widgets/flowtable_dnd.cc
+++ b/glom/utility_widgets/flowtable_dnd.cc
@@ -240,7 +240,7 @@ FlowTableDnd::dnd_item_at_position(int drag_x, int drag_y)
     column = drag_x / column_width;
   
   for(std::vector<FlowTableItem>::iterator cur_item = m_children.begin(); cur_item != m_children.end(); 
-       cur_item++)
+       ++cur_item)
   {
     Gdk::Rectangle rect = cur_item->m_first->get_allocation();
     if(cur_item->m_second)
@@ -318,7 +318,7 @@ FlowTable::FlowTableItem* FlowTableDnd::find_current_dnd_item(Gtk::Widget* child
   FlowTableItem* item;
   type_vecChildren::iterator cur_child;
   for(cur_child = m_children.begin();
-      cur_child != m_children.end(); cur_child++)
+      cur_child != m_children.end(); ++cur_child)
   {
     // The widget was added directly to the FlowTableDnd
     if(cur_child->m_first == child ||
[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]