[
https://issues.apache.org/jira/browse/THRIFT-815?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12887006#action_12887006
]
Jordan commented on THRIFT-815:
-------------------------------
Jake,
I stepped through the browser using firefox, and I could see that the first
element from a list is obtained correctly and the second one is incorrect. I'm
not sure what is causing it. There's all sorts of crazy variable names such as
rstack, rpos, r, fid, ftype that have no explanation or documentation. I've
exhausted a couple hours trying to see what's going on, and I can't seem to
make sense of the code. Can you take a look this weekend? I think that this is
the most critical bug so far in the js-compiler.
> Deserialization of lists is critically broken.
> ----------------------------------------------
>
> Key: THRIFT-815
> URL: https://issues.apache.org/jira/browse/THRIFT-815
> Project: Thrift
> Issue Type: Bug
> Components: Compiler (JavaScript)
> Affects Versions: 0.4
> Reporter: Jordan
> Priority: Critical
>
> Edit the test code that comes with the js language target:
> var list = [1,2,3];
> var ret = client.testList(list);
> debugger;
> ret comes back as [3,3,3] when it should be echoed back as [1,2,3]
> The test case never caught this because it only verified the size, and not
> the contents of the returned array.
> I cannot find an immediate workaround, but I will try wrapping it in a dummy
> type like ListServiceResponse.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
