From 7fe656be30e550b97258579a6763a09c59bfd660 Mon Sep 17 00:00:00 2001
From: Glen Low <glen.low@pixelglow.com>
Date: Wed, 13 Nov 2013 07:18:02 +0800
Subject: [PATCH] Prefer ARC-scoped release rather than autorelease in the API
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* Creating old entry stream and data is now by convention “ns_returns_retained”, which stops autoreleasing on returning large data (see @f5abe34).
---
 zipzap/ZZArchiveEntry.h     | 4 ++--
 zipzap/ZZArchiveEntry.m     | 4 ++--
 zipzap/ZZOldArchiveEntry.h  | 4 ++--
 zipzap/ZZOldArchiveEntry.mm | 4 ++--
 zipzapTests/ZZUnzipTests.m  | 6 +++---
 5 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/zipzap/ZZArchiveEntry.h b/zipzap/ZZArchiveEntry.h
index ba4be7bc..b46d8419 100644
--- a/zipzap/ZZArchiveEntry.h
+++ b/zipzap/ZZArchiveEntry.h
@@ -162,14 +162,14 @@
  *
  * @return The new stream: *nil* for new entries.
  */
-- (NSInputStream*)stream;
+- (NSInputStream*)newStream;
 
 /**
  * Creates data to represent the entry file.
  *
  * @return The new data: *nil* for new entries.
  */
-- (NSData*)data;
+- (NSData*)newData;
 
 /**
  * Creates a data provider to represent the entry file.
diff --git a/zipzap/ZZArchiveEntry.m b/zipzap/ZZArchiveEntry.m
index 6f845596..536155b1 100644
--- a/zipzap/ZZArchiveEntry.m
+++ b/zipzap/ZZArchiveEntry.m
@@ -115,7 +115,7 @@ - (NSString*)fileName
 	return nil;
 }
 
-- (NSInputStream*)stream
+- (NSInputStream*)newStream
 {
 	return nil;
 }
@@ -125,7 +125,7 @@ - (BOOL)check:(NSError **)error
 	return YES;
 }
 
-- (NSData*)data
+- (NSData*)newData
 {
 	return nil;
 }
diff --git a/zipzap/ZZOldArchiveEntry.h b/zipzap/ZZOldArchiveEntry.h
index 95dd9884..4c33376c 100644
--- a/zipzap/ZZOldArchiveEntry.h
+++ b/zipzap/ZZOldArchiveEntry.h
@@ -33,8 +33,8 @@
 
 - (BOOL)check:(NSError **)error;
 
-- (NSInputStream*)stream;
-- (NSData*)data;
+- (NSInputStream*)newStream;
+- (NSData*)newData;
 - (CGDataProviderRef)newDataProvider;
 
 @end
diff --git a/zipzap/ZZOldArchiveEntry.mm b/zipzap/ZZOldArchiveEntry.mm
index aed52d74..c575ad18 100644
--- a/zipzap/ZZOldArchiveEntry.mm
+++ b/zipzap/ZZOldArchiveEntry.mm
@@ -186,7 +186,7 @@ - (BOOL)check:(NSError**)error
 	return YES;
 }
 
-- (NSInputStream*)stream
+- (NSInputStream*)newStream
 {
 	switch (_centralFileHeader->compressionMethod)
 	{
@@ -201,7 +201,7 @@ - (NSInputStream*)stream
 	}
 }
 
-- (NSData*)data
+- (NSData*)newData
 {
 	switch (_centralFileHeader->compressionMethod)
 	{
diff --git a/zipzapTests/ZZUnzipTests.m b/zipzapTests/ZZUnzipTests.m
index ea75c04a..f1380850 100644
--- a/zipzapTests/ZZUnzipTests.m
+++ b/zipzapTests/ZZUnzipTests.m
@@ -62,7 +62,7 @@ - (void)checkExtractingZipEntryStreamWithChunkSize:(NSUInteger)chunkSize
 		ZZArchiveEntry* nextEntry = _zipFile.entries[index];
 		NSString* nextEntryFilePath = _entryFilePaths[index];
 		
-		NSInputStream* stream = [nextEntry stream];
+		NSInputStream* stream = [nextEntry newStream];
 		
 		[stream open];
 		
@@ -210,7 +210,7 @@ - (void)testZipFromDataConsistentWithZipFromURL
 
 		XCTAssertEqual(zipEntry.crc32, zipFromDataEntry.crc32, @"zipFromDataEntry.entries[%lu].crc32 must match the reference entry.", (unsigned long)index);
 
-		XCTAssertEqualObjects(zipEntry.data, zipFromDataEntry.data, @"zipFromDataEntry.entries[%lu].data must match the reference entry.", (unsigned long)index);
+		XCTAssertEqualObjects([zipEntry newData], [zipFromDataEntry newData], @"zipFromDataEntry.entries[%lu].data must match the reference entry.", (unsigned long)index);
 
 		XCTAssertEqual(zipEntry.fileMode, zipFromDataEntry.fileMode, @"zipFromDataEntry.entries[%lu].fileMode must match the reference entry.", (unsigned long)index);
 
@@ -229,7 +229,7 @@ - (void)testExtractingZipEntryData
 		ZZArchiveEntry* nextEntry = _zipFile.entries[index];
 		NSString* nextEntryFilePath = _entryFilePaths[index];
 		
-		XCTAssertEqualObjects(nextEntry.data,
+		XCTAssertEqualObjects([nextEntry newData],
 							 [self dataAtFilePath:nextEntryFilePath],
 							 @"zipFile.entries[%lu].data must match the original file data.",
 							 (unsigned long)index);