bugfix: preserve 'empty_array_mt' behavior upon multiple loadings of the module.

Prior to this fix, when the module would be loaded several times
(by-passing `package.loaded`), the `lua_cjson_new` function would
override the `empty_array_mt` table in the registry with a new one.
Comparison for equality between those tables would then fail, and the
behavior would be broken.

This was discovered after loading `cjson` *and* `cjson.safe` in the same
application, resulting in two calls to `lua_cjson_new`.

Signed-off-by: Yichun Zhang (agentzh) <agentzh@gmail.com>

1 files changed, 26 insertions, 2 deletions

diff --git a/tests/agentzh.t b/tests/agentzh.t
index e76f910..3b0ecf1 100644
--- a/tests/agentzh.t
+++ b/tests/agentzh.t
@@ -113,7 +113,31 @@ print(cjson.encode(data))
 
 
 
-=== TEST 9: & in JSON
+=== TEST 9: multiple calls to lua_cjson_new (1/2)
+--- lua
+local cjson = require "cjson"
+package.loaded["cjson"] = nil
+require "cjson"
+local arr = setmetatable({}, cjson.empty_array_mt)
+print(cjson.encode(arr))
+--- out
+[]
+
+
+
+=== TEST 10: multiple calls to lua_cjson_new (2/2)
+--- lua
+local cjson = require "cjson.safe"
+-- load another cjson instance (not in package.loaded)
+require "cjson"
+local arr = setmetatable({}, cjson.empty_array_mt)
+print(cjson.encode(arr))
+--- out
+[]
+
+
+
+=== TEST 11: & in JSON
 --- lua
 local cjson = require "cjson"
 local a="[\"a=1&b=2\"]"
@@ -124,7 +148,7 @@ print(cjson.encode(b))
 
 
 
-=== TEST 10: default and max precision
+=== TEST 12: default and max precision
 --- lua
 local math = require "math"
 local cjson = require "cjson"