Changeset 9722 in orxonox.OLD for branches/new_class_id/src/lib/parser/ini_parser/ini_parser.cc

Timestamp:

Sep 4, 2006, 11:54:30 PM (18 years ago)

Author:

bensch

Message:

orxonox: now it is gcc-4.1.1 conform

strangely before this, it was possible to compare and set std::list::iterators to NULL values.
also it now is not allowed to use
void ClassName::functionName(); inside of the header… quite nice feature, and i think compiling is faster too

Gentoo Rocks
GCC rocks also with nptl

File:

• branches/new_class_id/src/lib/parser/ini_parser/ini_parser.cc (modified) (4 diffs)

branches/new_class_id/src/lib/parser/ini_parser/ini_parser.cc

@@ -165 +165 @@
       else if( (ptr = strchr( lineBegin, '=')) != NULL)
       {
-        if (currentSection == NULL)
+        if (currentSection == this->sections.end())
         {
           PRINTF(2)("Not in a Section yet for %s\n", lineBegin);
@@ -470 +470 @@
   {
     std::list<IniEntry>::const_iterator entry = this->getEntryIT(entryName, sectionName);
-    if (entry != NULL &&  (*entry).name == entryName)
+    if (/** FIXME entry != NULL && */  (*entry).name == entryName)
       return (*entry).value;
     PRINTF(2)("Entry '%s' in section '%s' not found.\n", entryName.c_str(), sectionName.c_str());
@@ -627 +627 @@
       if ((*entry).name == entryName)
         break;
-  if (entry == (*section).entries.end())
-    return NULL;
-  else
-    return entry;
+  return entry;
 }
 
@@ -650 +647 @@
       if ((*entry).name == entryName)
         break;
-  if (entry == (*section).entries.end())
-    return NULL;
-  else
-    return entry;
+  return entry;
 }