Author: jake
Date: Sat Jul 10 14:58:51 2010
New Revision: 962853
URL: http://svn.apache.org/viewvc?rev=962853&view=rev
Log:
THRIFT-813: fix for broken list deserialization
Modified:
incubator/thrift/trunk/lib/js/test/test.html
incubator/thrift/trunk/lib/js/thrift.js
Modified: incubator/thrift/trunk/lib/js/test/test.html
URL:
http://svn.apache.org/viewvc/incubator/thrift/trunk/lib/js/test/test.html?rev=962853&r1=962852&r2=962853&view=diff
==============================================================================
--- incubator/thrift/trunk/lib/js/test/test.html (original)
+++ incubator/thrift/trunk/lib/js/test/test.html Sat Jul 10 14:58:51 2010
@@ -46,7 +46,7 @@
document.write("client.testStruct() =>
"+(client.testStruct(xtr).string_thing == "worked")+"<br/>")
document.write("client.testNest() =>
"+(client.testNest(xtr2).struct_thing.string_thing == "worked")+"<br/>")
document.write("client.testMap() =>
"+(client.testMap(insanity.userMap)[ThriftTest.Numberz.ONE] == 1)+"<br/>")
- document.write("client.testList() => "+(client.testList(list).length ==
3)+"<br/>")
+ document.write("client.testList() => "+(client.testList(list).length == 3
&& client.testList(list)[1] == 2)+"<br/>")
document.write("client.testSet() => "+(client.testSet(list).length ==
3)+"<br/>")
document.write("client.testEnum() =>
"+(client.testEnum(ThriftTest.Numberz.ONE) == ThriftTest.Numberz.ONE)+"<br/>")
Modified: incubator/thrift/trunk/lib/js/thrift.js
URL:
http://svn.apache.org/viewvc/incubator/thrift/trunk/lib/js/thrift.js?rev=962853&r1=962852&r2=962853&view=diff
==============================================================================
--- incubator/thrift/trunk/lib/js/thrift.js (original)
+++ incubator/thrift/trunk/lib/js/thrift.js Sat Jul 10 14:58:51 2010
@@ -577,7 +577,7 @@ Thrift.Protocol.prototype = {
this.rpos.push(this.rstack.length);
- this.rstack.push(list.shift())
+ this.rstack.push(list)
return r;
