From a6f8bae69f36a582c0eb55988c1441421a3c042e Mon Sep 17 00:00:00 2001 From: Grzegorz Adam Hankiewicz Date: Sat, 29 Mar 2014 12:27:18 +0100 Subject: [PATCH] Uses swap() to avoid variable copies. Refs #1048. --- lib/wrappers/zip/zlib.nim | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/wrappers/zip/zlib.nim b/lib/wrappers/zip/zlib.nim index f09b8b97ab..2d8ac9058a 100644 --- a/lib/wrappers/zip/zlib.nim +++ b/lib/wrappers/zip/zlib.nim @@ -287,10 +287,10 @@ proc uncompress*(sourceBuf: cstring, sourceLen: int): string = # Verify that the input is a valid gzip stream. if status != Z_STREAM_END: # Incomplete gzip stream. - return nil + return decompressed.setLen(have) - result = decompressed + swap(result, decompressed) proc inflate*(buffer: var string): bool {.discardable.} = @@ -303,7 +303,7 @@ proc inflate*(buffer: var string): bool {.discardable.} = ## Returns true if `buffer` was successfully inflated. assert (not buffer.isNil) if buffer.len < 1: return - let temp = uncompress(addr(buffer[0]), buffer.len) + var temp = uncompress(addr(buffer[0]), buffer.len) if not temp.isNil: - buffer = temp + swap(buffer, temp) result = true