Separated class IniValue from class Ini

Also replaced Ini::get() on ini_get() see https://github.com/CopernicaMarketingSoftware/PHP-CPP/issues/64#issuecomment-40313791

3 files changed, 144 insertions, 129 deletions

diff --git a/include/call.h b/include/call.h
index 2fcc9b0..f881782 100644
--- a/include/call.h
+++ b/include/call.h
@@ -57,7 +57,16 @@ inline Value echo(const std::string &input) { out << input; return nullptr; }
 inline Value empty(const Value &value) { return value.isNull() || !value.boolValue(); }
 inline Value empty(const HashMember<std::string> &member) { return !member.exists() || empty(member.value()); }
 inline Value empty(const HashMember<int> &member) { return !member.exists() || empty(member.value()); }
-inline Value ini_get(const Value &value) { return call("ini_get", value); }
+
+inline IniValue ini_get(const char* name)
+{
+    return IniValue(name, false);
+}
+inline IniValue ini_get_orig(const char* name)
+{
+    return IniValue(name, true);
+}
+
 inline Value is_array(const Value &value) { return value.isArray(); }
 inline Value strlen(const Value &value) { return call("strlen", value); }
 inline void  unset(const HashMember<std::string> &member) { member.unset(); }
diff --git a/include/ini.h b/include/ini.h
index 8745fef..1d881e9 100644
--- a/include/ini.h
+++ b/include/ini.h
@@ -17,123 +17,6 @@ struct _zend_ini_entry;
  */
 namespace Php {
 
-/**
- *  Class IniValue designed for extracting values from ini entries.
- */
-class IniValue
-{
-public:
-    /**
-     *  Constructors for floating point values
-     * 
-     *  @param  name        Name of the php.ini variable
-     *  @param  isorig      Is the original value
-     */
-    IniValue(const char *name, const bool isorig) : _name(name), _isorig(isorig) {}
-
-    /**
-     *  Cast to a number
-     *  @return int32_t
-     */
-    operator int16_t () const
-    {
-        return (int16_t)numericValue();
-    }
-
-    /**
-     *  Cast to a number
-     *  @return int32_t
-     */
-    operator int32_t () const
-    {
-        return (int32_t)numericValue();
-    }
-
-    /**
-     *  Cast to a number
-     *  @return uint64_t
-     */
-    operator int64_t () const
-    {
-        return numericValue();
-    }
-    
-    /**
-     *  Cast to a boolean
-     *  @return boolean
-     */
-    operator bool () const
-    {
-        return (bool)numericValue();
-    }
-    
-    /**
-     *  Cast to a string
-     *  @return string
-     */
-    operator std::string () const
-    {
-        return rawValue();
-    }
-    
-    /**
-     *  Cast to byte array
-     *  @return const char *
-     */
-    operator const char * () const
-    {
-        return rawValue();
-    }
-    
-    /**
-     *  Cast to a floating point
-     *  @return double
-     */
-    operator double() const;
-
-
-private:
-
-    
-    /**
-     *  Retrieve the value as number
-     *
-     *  We force this to be a int64_t because we assume that most
-     *  servers run 64 bits nowadays, and because we use int32_t, int64_t
-     *  almost everywhere, instead of 'long' and on OSX neither of
-     *  these intxx_t types is defined as 'long'...
-     *
-     *  @return int64_t
-     */
-    int64_t numericValue() const;
-
-    /**
-     *  Get access to the raw buffer for read operationrs.
-     *  @return const char *
-     */
-    const char *rawValue() const;
-    
-    /**
-     *  ini entry name
-     *  @var    std::string
-     */
-    std::string _name;
-
-    /**
-     *  Is the orig value?
-     *  @var    bool
-     */
-    bool _isorig = false;
-};
-
-/**
- *  Custom output stream operator
- *  @param  stream
- *  @param  ini_val
- *  @return ostream
- */
-std::ostream &operator<<(std::ostream &stream, const IniValue &ini_val);
-
 
 /**
  *  Class definition
@@ -235,17 +118,6 @@ public:
      */
     void fill(struct _zend_ini_entry *ini_entry, int module_number);
 
-    
-    static IniValue get(const char* name)
-    {
-        return IniValue(name, false);
-    }
-
-    static IniValue get_orig(const char* name)
-    {
-        return IniValue(name, true);
-    }
-
 
 private:
     /**
diff --git a/include/inivalue.h b/include/inivalue.h
new file mode 100644
index 0000000..9f2e44c
--- /dev/null
+++ b/include/inivalue.h
@@ -0,0 +1,134 @@
+/**
+ *  IniValue.h
+ *
+ *  Class IniValue designed for extracting values from ini entries
+ *  
+ *  @copyright 2014 Copernica BV
+ */
+
+/**
+ *  Set up namespace
+ */
+namespace Php {
+
+/**
+ *  Class IniValue designed for extracting values from ini entries.
+ */
+class IniValue
+{
+public:
+    /**
+     *  Constructors
+     * 
+     *  @param  name        Name of the php.ini variable
+     *  @param  isorig      Is the original value
+     */
+    IniValue(const char *name, const bool isorig) : _name(name), _isorig(isorig) {}
+
+    /**
+     *  Cast to a number
+     *  @return int32_t
+     */
+    operator int16_t () const
+    {
+        return (int16_t)numericValue();
+    }
+
+    /**
+     *  Cast to a number
+     *  @return int32_t
+     */
+    operator int32_t () const
+    {
+        return (int32_t)numericValue();
+    }
+
+    /**
+     *  Cast to a number
+     *  @return uint64_t
+     */
+    operator int64_t () const
+    {
+        return numericValue();
+    }
+    
+    /**
+     *  Cast to a boolean
+     *  @return boolean
+     */
+    operator bool () const
+    {
+        return (bool)numericValue();
+    }
+    
+    /**
+     *  Cast to a string
+     *  @return string
+     */
+    operator std::string () const
+    {
+        return rawValue();
+    }
+    
+    /**
+     *  Cast to byte array
+     *  @return const char *
+     */
+    operator const char * () const
+    {
+        return rawValue();
+    }
+    
+    /**
+     *  Cast to a floating point
+     *  @return double
+     */
+    operator double() const;
+    
+private:
+    
+    /**
+     *  Retrieve the value as number
+     *
+     *  We force this to be a int64_t because we assume that most
+     *  servers run 64 bits nowadays, and because we use int32_t, int64_t
+     *  almost everywhere, instead of 'long' and on OSX neither of
+     *  these intxx_t types is defined as 'long'...
+     *
+     *  @return int64_t
+     */
+    int64_t numericValue() const;
+
+    /**
+     *  Get access to the raw buffer for read operationrs.
+     *  @return const char *
+     */
+    const char *rawValue() const;
+    
+    /**
+     *  ini entry name
+     *  @var    std::string
+     */
+    std::string _name;
+
+    /**
+     *  Is the orig value?
+     *  @var    bool
+     */
+    bool _isorig = false;
+};
+
+/**
+ *  Custom output stream operator
+ *  @param  stream
+ *  @param  ini_val
+ *  @return ostream
+ */
+std::ostream &operator<<(std::ostream &stream, const IniValue &ini_val);
+
+
+/**
+ *  End of namespace
+ */
+}
+