It is now safe to call begin() and end() on a cfg::Option with an emp…

…ty array.

configfile.h

@@ -19,6 +19,7 @@ See README.md for more information.
 TODO:
     Handle escape codes inside of strings.
     Handle multiple array elements on the same line.
+    Support sub-sections by using a tree, similar to Option arrays.
     Writing should optionally preserve formatting and comments.
 */
 class File

configoption.cpp

@@ -6,6 +6,8 @@
 namespace cfg
 {
 
+Option::OptionVector Option::emptyVector;
+
 Option::Option():
     str(),
     number(0),
@@ -191,22 +193,30 @@ void Option::clear()
 
 Option::OptionVector::iterator Option::begin()
 {
-    return options->begin();
+    if (options)
+        return options->begin();
+    return emptyVector.begin();
 }
 
 Option::OptionVector::iterator Option::end()
 {
-    return options->end();
+    if (options)
+        return options->end();
+    return emptyVector.end();
 }
 
 Option::OptionVector::const_iterator Option::begin() const
 {
-    return options->begin();
+    if (options)
+        return options->begin();
+    return emptyVector.begin();
 }
 
 Option::OptionVector::const_iterator Option::end() const
 {
-    return options->end();
+    if (options)
+        return options->end();
+    return emptyVector.end();
 }
 
 std::string Option::buildArrayString(const std::string& indentStr) const

configoption.h

@@ -100,6 +100,9 @@ class Option
         // Also, this is only created when push() is called for the first time
         // Also, this array is separate from the option itself, and nothing is kept in sync
             // This means that the first element can be different from the option.
+
+        static OptionVector emptyVector;
+        // This is used for returning iterators when the array isn't allocated
 };
 
 template <typename Type>