[jira] [Comment Edited] (THRIFT-2859) JSON generator: output complete descriptors

["Jens Geyer (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/THRIFT-2859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14234654#comment-14234654 ] 

Jens Geyer edited comment on THRIFT-2859 at 12/4/14 9:32 PM:
-------------------------------------------------------------

[~stigsb],

I still get a warning at a {{write_integer()}} call with VS 2013:

{code}
t_json_generator.cc(455): warning C4244: 'Argument': conversion from 'int64_t' into 'long', possible data loss.
{code}



was (Author: jensg):
[~stigsb],

I still get a warning at a {{write_integer()}} call:

{code}
t_json_generator.cc(455): warning C4244: 'Argument': conversion from 'int64_t' into 'long', possible data loss.
{code}


> JSON generator: output complete descriptors
> -------------------------------------------
>
>                 Key: THRIFT-2859
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2859
>             Project: Thrift
>          Issue Type: Improvement
>          Components: JSON - Compiler
>    Affects Versions: 0.9.2
>            Reporter: Stig Bakken
>            Assignee: Jens Geyer
>            Priority: Minor
>             Fix For: 0.9.3
>
>
> (Someone should probably add a Jira component for the JSON generator.)
> The attached pull request contains some improvements to the JSON generator:
> 1. programs are no longer merged by default, use "--gen json:merge"
> 2. "index" is renamed to "key" for fields to match the compiler's internal naming
> 3. types are described in a way that does not require string parsing
> 4. full support for exceptions, unions, namespaces, includes and required/optional fields



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)