[
https://issues.apache.org/jira/browse/THRIFT-3028?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14500231#comment-14500231
]
Hudson commented on THRIFT-3028:
--------------------------------
SUCCESS: Integrated in Thrift #1512 (See
[https://builds.apache.org/job/Thrift/1512/])
THRIFT-3028 Regression caused by THRIFT-2180 (jensg: rev
2aa04bf384a65e8a5ea3f996b790d231aa279626)
* lib/cocoa/src/protocol/TBinaryProtocol.m
> Regression caused by THRIFT-2180
> --------------------------------
>
> Key: THRIFT-3028
> URL: https://issues.apache.org/jira/browse/THRIFT-3028
> Project: Thrift
> Issue Type: Bug
> Components: Cocoa - Library
> Affects Versions: 0.9.3
> Environment: Reported against ios-arm64.
> Reporter: Jeremy W. Sherman
> Assignee: Jens Geyer
> Labels: endianness
> Fix For: 0.9.3
>
> Attachments: 3028.patch
>
>
> Changes made in THRIFT-2180 to TBinaryProtocol broke little-endian message
> reading. Message-writing was also affected.
> This was not caught by tests, because the cross-tests appear not to include
> the Cocoa platform.
> A minimal patch is attached to address this.
> I wrote a local test using a StreamTransport, which lets me use NSStream to
> read/write via in-memory buffers easily (or file handles - the existing
> transports probably should have been written in terms of this standard
> Foundation abstraction) to verify that write/read of a message agree on how
> to read-write.
> [~gurubook]: Would you be so kind as to apply the attached patch, and verify
> it addresses the issue you reported on THRIFT-2180?
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
