Fix a bug in the parser

README.md

@@ -22,13 +22,7 @@ On Windows, you need to install MinGW.
     waf build --shared
 
 ### Running unit tests ###
-cpp-iniparses uses GoogleTest (http://code.google.com/p/googletest/)
-
-    waf build --test
-
-Alternatively, we can also explicitly specify the gtest headers and libraries location
-
-    waf build --test --gtest_include=/gtest/include --gtest_lib=/gtest/lib
+    ./run_tests.sh
 
 ### Installing cppiniparser ###
     waf install

run_tests.sh

@@ -0,0 +1,8 @@
+#!/bin/bash
+
+cd gtest-1.7.0/make
+make
+cp gtest_main.a libgtest_main.a
+cd -
+./waf clean
+./waf build --test --gtest_include=gtest-1.7.0/include --gtest_lib=gtest-1.7.0/make

src/INIParser.cpp

@@ -41,14 +41,14 @@ INIConfig INIParser::Read(const std::string& filename) {
     try {
         std::string line;
         while (getline(is, line)) {
-            if (utils::IsSection(line)) {
+            if (utils::IsEmptyLine(line) || utils::IsComment(line)) {
+                // ignore it
+            } else if (utils::IsSection(line)) {
                 section = utils::ParseSection(line);
                 config.AddSection(section);
             } else if (utils::IsOption(line)) {
                 std::pair<std::string, std::string> option = utils::ParseOption(line);
                 config.AddOption(section, option.first, option.second);
-            } else if (utils::IsEmptyLine(line) || utils::IsComment(line)) {
-                // ignore it
             } else {
                 std::string msg = "Invalid line: " + line;
                 throw INIReaderException(msg.c_str());

test/test1.ini

@@ -1,9 +1,13 @@
+;=========================================================================
 ; foo section
+;=========================================================================
 [foo]
 name=foo
 msg=hello
 
+;=========================================================================
 ; bar section
+;=========================================================================
 [bar]
 name=bar
 msg=bye