[ 
https://issues.apache.org/jira/browse/THRIFT-550?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Roger Meier updated THRIFT-550:
-------------------------------

    Attachment: THRIFT-550_initialize_with_null.patch

I had a problem with the exception avaliable on the tutorial.

i32 calculate(1:i32 logid, 2:Work w) throws (1:InvalidOperation ouch),

It couldn't be thrown because initialization of numeric base types is done with 
0 instead of null and the checks do compare against null.
This was not visible with the Tests provided with first patch for JavaScript 
bindings above, the ThriftTest.thrift definition does not have a combination of 
a base type return value and an exception.

I've made a patch that initializes the base types I16,I32, I64 and DOUBLE with 
null. This could probably solve other issues as well

Regards

Roger

> JavaScript language target
> --------------------------
>
>                 Key: THRIFT-550
>                 URL: https://issues.apache.org/jira/browse/THRIFT-550
>             Project: Thrift
>          Issue Type: New Feature
>            Reporter: T Jake Luciani
>            Assignee: T Jake Luciani
>             Fix For: 0.3
>
>         Attachments: THRIFT-550_initialize_with_null.patch, thrift_js.patch, 
> thrift_js_v2.patch, thrift_js_v3.patch
>
>
> This allows javascript clients to be built for browser based access to thrift 
> services.
> The supported transport is XHTTPRequest and the supported Protocol is 
> compatible with TJSONProtocol
> I've added a test in lib/js that is a java based HTTP server which implements 
> ThriftTest service.
> I've put a version of this test service here 
> http://3.rdrail.net:8080/test/test.html if you'd like to see it in action.
> It's been tested in ie 6,7,8, FF, and Safari.
> Let me know if you hit any issues.  The protocol and transport code aren't 
> very error tolerant, but overall it seems to work quite well.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to