removed Value constructor that wraps around a hash table

Author: EmielBruijntjes

include/value.h

@@ -112,12 +112,6 @@ class Value
      */
     Value(struct _zval_struct *zval, bool ref = false);
 
-    /**
-     *  Wrap around a hash table
-     *  @param  ht          Hashtable to wrap
-     */
-    Value(struct _hashtable *ht);
-    
     /**
      *  Wrap around an object implemented by us
      *  @param  object      Object to be wrapped
@@ -445,6 +439,8 @@ class Value
     template <typename T>
     std::vector<T> vectorValue() const
     {
+        
+        
         // only works for arrays, other types return an empty vector
         if (!isArray()) return std::vector<T>();
 

src/value.cpp

@@ -169,22 +169,6 @@ Value::Value(struct _zval_struct *val, bool ref)
     Z_SET_ISREF_P(_val);
 }
 
-/**
- *  Wrap around a hash table
- *  @param  ht          Hashtable to wrap
- */
-Value::Value(HashTable *ht)
-{
-    // construct a zval
-    MAKE_STD_ZVAL(_val);
-    Z_ARRVAL_P(_val) = ht;
-    Z_TYPE_P(_val) = IS_ARRAY;
-
-    // add a reference
-    // @todo this may be wrong
-    Z_ADDREF_P(_val);
-}
-
 /**
  *  Wrap around an object
  *  @param  object
@@ -1572,8 +1556,6 @@ ValueIterator Value::begin() const
     if (isObject()) return ValueIterator(Z_OBJ_HT_P(_val)->get_properties(_val), true);
 
     // invalid
-    // @todo fix iterators without a hashtable
-    // @todo test Php::empty() function
     return ValueIterator(nullptr,true);
 }