Code Review / common / thrift.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 58dfb4f) | patch
Fixed possible Thrift serialization bug
authorDavid Reiss <dreiss@apache.org>
Fri, 6 Jul 2007 21:43:48 +0000 (21:43 +0000)
committerDavid Reiss <dreiss@apache.org>
Fri, 6 Jul 2007 21:43:48 +0000 (21:43 +0000)
commitf79031e87531464b84d911acf9aee57cac8b83d8
treef24a3e9989a757cf75e15eef59d43bbc24628628tree | snapshot
parent58dfb4fbe82c346814c2e4f33b6045468bfd68c6commit | diff
Fixed possible Thrift serialization bug

Summary: TBinaryProtocol.cpp was making assumptions about evaluation order
that are not guaranteed by the standard.  Now it does not (at least in these
functions).

Trac Bug: #

Blame Rev:

Reviewed By: mcslee

Test Plan: still compiles.

Revert Plan: svn revert

Notes:

EImportant:

git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@665152 13f79535-47bb-0310-9956-ffa450edef68
lib/cpp/src/protocol/TBinaryProtocol.cpp diff | blob | history
lib/cpp/src/protocol/TBinaryProtocol.h diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.