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

Nathan Beyer updated THRIFT-1699:
---------------------------------

    Attachment: THRIFT-1699-remove-extra-read-field-end.patch

I applied the patch and tested it on Mac OS X Lion with Ruby 1.9.3-p327. All of 
the Ruby tests passed fine.

The original patch wasn't a unified patch and a bit painful to apply via 
'patch', so I regenerated it via 'svn diff' and uploaded the file as 
[^THRIFT-1699-remove-extra-read-field-end.patch].
                
> Native Union#read has extra read_field_end call
> -----------------------------------------------
>
>                 Key: THRIFT-1699
>                 URL: https://issues.apache.org/jira/browse/THRIFT-1699
>             Project: Thrift
>          Issue Type: Bug
>          Components: Ruby - Library
>    Affects Versions: 0.8
>            Reporter: Kevin Radloff
>         Attachments: rb-struct.diff, 
> THRIFT-1699-remove-extra-read-field-end.patch
>
>
> The native implementation of the Union#read method has an extraneous call to 
> read_field_end. This makes no difference to most protocols because 
> read_field_end is a no-op, but it breaks JSON (in this case my own class, not 
> the one in trunk), which does do something in this method.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to