Fixed an error in the refactored clone method. Now it deep-copies for real.

node_factory.cpp

@@ -16,10 +16,12 @@ namespace Sass {
   Node_Impl* Node_Factory::alloc_Node_Impl(Node_Impl* ip)
   {
     Node_Impl* ip_cpy = new Node_Impl(*ip);
+    pool_.push_back(ip_cpy);
     if (ip_cpy->has_children) {
       for (size_t i = 0, S = ip_cpy->size(); i < S; ++i) {
         Node n(ip_cpy->at(i));
-        n.ip_ = alloc_Node_Impl(n.ip_);
+        // n.ip_ = alloc_Node_Impl(n.ip_);
+        ip_cpy->at(i) = (*this)(n);
       }
     }
     return ip_cpy;

test_node_factory.cpp

@@ -48,7 +48,19 @@ int main()
   cout << num4[3].numeric_value() << endl;
   num4[3] = new_Node("", 0, 0.4567);
   cout << num3[3].numeric_value() << endl;
-  cout << num4[3].numeric_value() << endl;
+  cout << num4[3].numeric_value() << endl << endl;
+
+  Node block1(new_Node(Node::block, "block", 1, 2));
+  block1 << num2 << num4;
+
+  Node block2(new_Node(block1));
+
+  cout << (block1 == block2) << endl;
+  cout << block1[1][3].numeric_value() << endl;
+  cout << block2[1][3].numeric_value() << endl;
+  block2[1][3] = new_Node("", 0, .9876);
+  cout << block1[1][3].numeric_value() << endl;
+  cout << block2[1][3].numeric_value() << endl;
 
   new_Node.free();