feedback

Author: geof90

CodePushDownloadHandler.m

@@ -2,7 +2,7 @@
 
 @implementation CodePushDownloadHandler {
     // Header chars used to determine if the file is a zip.
-    char header[4];
+    char _header[4];
 }
 
 - (id)init:(NSString *)downloadFilePath
@@ -46,14 +46,14 @@ -(void)connection:(NSURLConnection *)connection didReceiveResponse:(NSURLRespons
 
 -(void)connection:(NSURLConnection *)connection didReceiveData:(NSData *)data {
     if (self.receivedContentLength < 4) {
-        const char *bytes = [data bytes];
-        for (int i=0; i<[data length]; i++) {
+        for (int i = 0; i < [data length]; i++) {
             int headerOffset = (int)self.receivedContentLength + i;
             if (headerOffset >= 4) {
                 break;
             }
 
-            header[headerOffset] = bytes[i];
+            const char *bytes = [data bytes];
+            _header[headerOffset] = bytes[i];
         }
     }
 
@@ -94,7 +94,7 @@ -(void)connectionDidFinishLoading:(NSURLConnection *)connection {
     assert(self.receivedContentLength == self.expectedContentLength);
 
     [self.outputFileStream close];
-    BOOL isZip = header[0] == 'P' && header[1] == 'K' && header[2] == 3 && header[3] == 4;
+    BOOL isZip = _header[0] == 'P' && _header[1] == 'K' && _header[2] == 3 && _header[3] == 4;
     self.doneCallback(isZip);
 }
 

CodePushPackage.m

@@ -207,8 +207,8 @@ + (void)downloadPackage:(NSDictionary *)updatePackage
         return failCallback(error);
     }
 
-    NSString * downloadFilePath = [self getDownloadFilePath];
-    NSString * bundleFilePath = [newPackageFolderPath stringByAppendingPathComponent:UpdateBundleFileName];
+    NSString *downloadFilePath = [self getDownloadFilePath];
+    NSString *bundleFilePath = [newPackageFolderPath stringByAppendingPathComponent:UpdateBundleFileName];
 
     CodePushDownloadHandler *downloadHandler = [[CodePushDownloadHandler alloc]
         init:downloadFilePath
@@ -229,12 +229,11 @@ + (void)downloadPackage:(NSDictionary *)updatePackage
                     nonFailingError = nil;
                 }
 
-                NSString * diffManifestFilePath = [unzippedFolderPath
-                                                   stringByAppendingPathComponent:DiffManifestFileName];
+                NSString *diffManifestFilePath = [unzippedFolderPath stringByAppendingPathComponent:DiffManifestFileName];
 
                 if ([[NSFileManager defaultManager] fileExistsAtPath:diffManifestFilePath]) {
                     // Copy the current package to the new package.
-                    NSString* currentPackageFolderPath = [self getCurrentPackageFolderPath:&error];
+                    NSString *currentPackageFolderPath = [self getCurrentPackageFolderPath:&error];
                     if (error) {
                         failCallback(error);
                         return;
@@ -258,15 +257,13 @@ + (void)downloadPackage:(NSDictionary *)updatePackage
                     }
 
                     NSData *data = [manifestContent dataUsingEncoding:NSUTF8StringEncoding];
-                    NSDictionary* manifestJSON = [NSJSONSerialization JSONObjectWithData:data
+                    NSDictionary *manifestJSON = [NSJSONSerialization JSONObjectWithData:data
                                                                                  options:kNilOptions
                                                                                    error:&error];
                     NSArray *deletedFiles = manifestJSON[@"deletedFiles"];
                     for (NSString *deletedFileName in deletedFiles) {
-                        [[NSFileManager defaultManager]
-                         removeItemAtPath:[newPackageFolderPath
-                                           stringByAppendingPathComponent:deletedFileName]
-                         error:&nonFailingError];
+                        [[NSFileManager defaultManager] removeItemAtPath:[newPackageFolderPath stringByAppendingPathComponent:deletedFileName]
+                                                                   error:&nonFailingError];
 
                         if (nonFailingError) {
                             NSLog(@"Error deleting file from current package: %@", nonFailingError);
@@ -286,7 +283,7 @@ + (void)downloadPackage:(NSDictionary *)updatePackage
                 }
 
                 NSString *relativeBundlePath = [self findMainBundleInFolder:newPackageFolderPath
-                                                                   error:&error];
+                                                                      error:&error];
                 if (error) {
                     failCallback(error);
                     return;
@@ -314,13 +311,11 @@ + (void)downloadPackage:(NSDictionary *)updatePackage
                 }
             }
 
-            NSData *updateSerializedData = [NSJSONSerialization
-                                            dataWithJSONObject:mutableUpdatePackage
-                                            options:0
-                                            error:&error];
-            NSString *packageJsonString = [[NSString alloc]
-                                           initWithData:updateSerializedData
-                                           encoding:NSUTF8StringEncoding];
+            NSData *updateSerializedData = [NSJSONSerialization dataWithJSONObject:mutableUpdatePackage
+                                                                           options:0
+                                                                             error:&error];
+            NSString *packageJsonString = [[NSString alloc] initWithData:updateSerializedData
+                                                                encoding:NSUTF8StringEncoding];
 
             [packageJsonString writeToFile:[newPackageFolderPath stringByAppendingPathComponent:@"app.json"]
                                 atomically:YES