[ https://issues.apache.org/jira/browse/THRIFT-3028?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14499899#comment-14499899 ]
Jeremy W. Sherman commented on THRIFT-3028: ------------------------------------------- [~jensg] Would a GitHub pull request be preferable to an attached patch? > 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 > Labels: endianness > 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)