Repository: thrift Updated Branches: refs/heads/master 2d6060d88 -> de9c330b2
THRIFT-4007: TTransport.py micro-optimization: don't call len(chunk) twice Client: Python Patch: Nate Jensen <nathan.jen...@raytheon.com> This closes #1121 Project: http://git-wip-us.apache.org/repos/asf/thrift/repo Commit: http://git-wip-us.apache.org/repos/asf/thrift/commit/de9c330b Tree: http://git-wip-us.apache.org/repos/asf/thrift/tree/de9c330b Diff: http://git-wip-us.apache.org/repos/asf/thrift/diff/de9c330b Branch: refs/heads/master Commit: de9c330b24c9190078eefb68c864d2a41a4dee07 Parents: 2d6060d Author: James E. King, III <jk...@apache.org> Authored: Wed Feb 1 07:16:28 2017 -0500 Committer: James E. King, III <jk...@apache.org> Committed: Wed Feb 1 07:16:28 2017 -0500 ---------------------------------------------------------------------- lib/py/src/transport/TTransport.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/thrift/blob/de9c330b/lib/py/src/transport/TTransport.py ---------------------------------------------------------------------- diff --git a/lib/py/src/transport/TTransport.py b/lib/py/src/transport/TTransport.py index 5283fce..a3e9725 100644 --- a/lib/py/src/transport/TTransport.py +++ b/lib/py/src/transport/TTransport.py @@ -58,10 +58,11 @@ class TTransportBase(object): have = 0 while (have < sz): chunk = self.read(sz - have) - have += len(chunk) + chunkLen = len(chunk) + have += chunkLen buff += chunk - if len(chunk) == 0: + if chunkLen == 0: raise EOFError() return buff