From 84f2296c5c6563f2d61a208d5c427d98003bfecd Mon Sep 17 00:00:00 2001 From: Lucas Di Pentima Date: Mon, 10 Oct 2016 08:33:40 -0300 Subject: [PATCH] 9701: Superfluous variable eliminated --- sdk/python/arvados/arvfile.py | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/sdk/python/arvados/arvfile.py b/sdk/python/arvados/arvfile.py index 85366d2fdc..c72225b433 100644 --- a/sdk/python/arvados/arvfile.py +++ b/sdk/python/arvados/arvfile.py @@ -564,13 +564,14 @@ class _BlockManager(object): if force or (pending_write_size >= config.KEEP_BLOCK_SIZE): new_bb = _BufferBlock("bufferblock%i" % len(self._bufferblocks), 2**14, None) self._bufferblocks[new_bb.blockid] = new_bb - size = 0 - while len(small_blocks) > 0 and (size + small_blocks[0].size()) <= config.KEEP_BLOCK_SIZE: + while len(small_blocks) > 0 and (new_bb.write_pointer + small_blocks[0].size()) <= config.KEEP_BLOCK_SIZE: bb = small_blocks.pop(0) - size += bb.size() arvfile = bb.owner new_bb.append(bb.buffer_view[0:bb.write_pointer].tobytes()) - arvfile.set_segments([Range(new_bb.blockid, 0, bb.size(), size-bb.size())]) + arvfile.set_segments([Range(new_bb.blockid, + 0, + bb.size(), + new_bb.write_pointer - bb.size())]) bb.clear() del self._bufferblocks[bb.blockid] self.commit_bufferblock(new_bb, sync=sync) -- 2.30.2