Cleaned up some warnings from the static analyzer

Author: gonzalonunez

Parse/PFObject.h

@@ -491,7 +491,7 @@ NS_REQUIRES_PROPERTY_DEFINITIONS
 
  @deprecated Please use `-fetch` instead.
  */
-- (void)refresh PARSE_DEPRECATED("Please use `-fetch` instead.");
+- (instancetype)refresh PARSE_DEPRECATED("Please use `-fetch` instead.");
 
 /*!
  @abstract *Synchronously* refreshes the `PFObject` with the current data from the server and sets an error if it occurs.
@@ -500,7 +500,7 @@ NS_REQUIRES_PROPERTY_DEFINITIONS
 
  @deprecated Please use `-fetch:` instead.
  */
-- (void)refresh:(NSError **)error PARSE_DEPRECATED("Please use `-fetch:` instead.");
+- (instancetype)refresh:(NSError **)error PARSE_DEPRECATED("Please use `-fetch:` instead.");
 
 /*!
  @abstract *Asynchronously* refreshes the `PFObject` and executes the given callback block.
@@ -531,13 +531,13 @@ NS_REQUIRES_PROPERTY_DEFINITIONS
 /*!
  @abstract *Synchronously* fetches the PFObject with the current data from the server.
  */
-- (void)fetch;
+- (instancetype)fetch;
 /*!
  @abstract *Synchronously* fetches the PFObject with the current data from the server and sets an error if it occurs.
 
  @param error Pointer to an `NSError` that will be set if necessary.
  */
-- (void)fetch:(NSError **)error;
+- (instancetype)fetch:(NSError **)error;
 
 /*!
  @abstract *Synchronously* fetches the `PFObject` data from the server if <isDataAvailable> is `NO`.
@@ -614,7 +614,7 @@ NS_REQUIRES_PROPERTY_DEFINITIONS
 
  @param objects The list of objects to fetch.
  */
-+ (void)fetchAll:(PF_NULLABLE NSArray *)objects;
++ (instancetype)fetchAll:(PF_NULLABLE NSArray *)objects;
 
 /*!
  @abstract *Synchronously* fetches all of the `PFObject` objects with the current data from the server
@@ -623,13 +623,13 @@ NS_REQUIRES_PROPERTY_DEFINITIONS
  @param objects The list of objects to fetch.
  @param error Pointer to an `NSError` that will be set if necessary.
  */
-+ (void)fetchAll:(PF_NULLABLE NSArray *)objects error:(NSError **)error;
++ (instancetype)fetchAll:(PF_NULLABLE NSArray *)objects error:(NSError **)error;
 
 /*!
  @abstract *Synchronously* fetches all of the `PFObject` objects with the current data from the server.
  @param objects The list of objects to fetch.
  */
-+ (void)fetchAllIfNeeded:(PF_NULLABLE NSArray *)objects;
++ (instancetype)fetchAllIfNeeded:(PF_NULLABLE NSArray *)objects;
 
 /*!
  @abstract *Synchronously* fetches all of the `PFObject` objects with the current data from the server
@@ -638,7 +638,7 @@ NS_REQUIRES_PROPERTY_DEFINITIONS
  @param objects The list of objects to fetch.
  @param error Pointer to an `NSError` that will be set if necessary.
  */
-+ (void)fetchAllIfNeeded:(PF_NULLABLE NSArray *)objects error:(NSError **)error;
++ (instancetype)fetchAllIfNeeded:(PF_NULLABLE NSArray *)objects error:(NSError **)error;
 
 /*!
  @abstract Fetches all of the `PFObject` objects with the current data from the server *asynchronously*.
@@ -718,7 +718,7 @@ NS_REQUIRES_PROPERTY_DEFINITIONS
  @abstract *Synchronously* loads data from the local datastore into this object,
  if it has not been fetched from the server already.
  */
-- (void)fetchFromLocalDatastore;
+- (instancetype)fetchFromLocalDatastore;
 
 /*!
  @abstract *Synchronously* loads data from the local datastore into this object, if it has not been fetched
@@ -729,7 +729,7 @@ NS_REQUIRES_PROPERTY_DEFINITIONS
 
  @param error Pointer to an `NSError` that will be set if necessary.
  */
-- (void)fetchFromLocalDatastore:(NSError **)error;
+- (instancetype)fetchFromLocalDatastore:(NSError **)error;
 
 /*!
  @abstract *Asynchronously* loads data from the local datastore into this object,

Parse/PFObject.m

@@ -1110,7 +1110,7 @@ - (void)mergeFromRESTDictionary:(NSDictionary *)object withDecoder:(PFDecoder *)
                 if (localOperationSet.updatedAt != nil &&
                     [localOperationSet.updatedAt compare:remoteOperationSet.updatedAt] != NSOrderedAscending) {
                     [localOperationSet mergeOperationSet:remoteOperationSet];
-                } else {
+                } else if (remoteOperationSet) {
                     NSUInteger index = [operationSetQueue indexOfObject:localOperationSet];
                     [remoteOperationSet mergeOperationSet:localOperationSet];
                     [operationSetQueue replaceObjectAtIndex:index withObject:remoteOperationSet];
@@ -2102,12 +2102,12 @@ - (BOOL)isDataAvailable {
     return self._state.complete;
 }
 
-- (void)refresh {
-    [self fetch];
+- (instancetype)refresh {
+    return [self fetch];
 }
 
-- (void)refresh:(NSError **)error {
-    [self fetch:error];
+- (instancetype)refresh:(NSError **)error {
+    return [self fetch:error];
 }
 
 - (void)refreshInBackgroundWithTarget:(id)target selector:(SEL)selector {
@@ -2118,12 +2118,12 @@ - (void)refreshInBackgroundWithBlock:(PFObjectResultBlock)block {
     [self fetchInBackgroundWithBlock:block];
 }
 
-- (void)fetch {
-    [self fetch:nil];
+- (instancetype)fetch {
+    return [self fetch:nil];
 }
 
-- (void)fetch:(NSError **)error {
-    [[self fetchInBackground] waitForResult:error];
+- (instancetype)fetch:(NSError **)error {
+    return [[self fetchInBackground] waitForResult:error];
 }
 
 - (BFTask *)fetchInBackground {
@@ -2175,20 +2175,20 @@ - (void)fetchIfNeededInBackgroundWithTarget:(id)target selector:(SEL)selector {
 #pragma mark - Fetching Many Objects
 ///--------------------------------------
 
-+ (void)fetchAll:(NSArray *)objects {
-    [PFObject fetchAll:objects error:nil];
++ (instancetype)fetchAll:(NSArray *)objects {
+    return [PFObject fetchAll:objects error:nil];
 }
 
-+ (void)fetchAllIfNeeded:(NSArray *)objects {
-    [PFObject fetchAllIfNeeded:objects error:nil];
++ (instancetype)fetchAllIfNeeded:(NSArray *)objects {
+    return [PFObject fetchAllIfNeeded:objects error:nil];
 }
 
-+ (void)fetchAll:(NSArray *)objects error:(NSError **)error {
-    [[self fetchAllInBackground:objects] waitForResult:error];
++ (instancetype)fetchAll:(NSArray *)objects error:(NSError **)error {
+    return [[self fetchAllInBackground:objects] waitForResult:error];
 }
 
-+ (void)fetchAllIfNeeded:(NSArray *)objects error:(NSError **)error {
-    [[self fetchAllIfNeededInBackground:objects] waitForResult:error];
++ (instancetype)fetchAllIfNeeded:(NSArray *)objects error:(NSError **)error {
+    return [[self fetchAllIfNeededInBackground:objects] waitForResult:error];
 }
 
 + (BFTask *)fetchAllInBackground:(NSArray *)objects {
@@ -2249,12 +2249,12 @@ + (void)fetchAllIfNeededInBackground:(NSArray *)objects block:(PFArrayResultBloc
 #pragma mark - Fetch From Local Datastore
 ///--------------------------------------
 
-- (void)fetchFromLocalDatastore {
-    [self fetchFromLocalDatastore:nil];
+- (instancetype)fetchFromLocalDatastore {
+    return [self fetchFromLocalDatastore:nil];
 }
 
-- (void)fetchFromLocalDatastore:(NSError **)error {
-    [[self fetchFromLocalDatastoreInBackground] waitForResult:error];
+- (instancetype)fetchFromLocalDatastore:(NSError **)error {
+    return [[self fetchFromLocalDatastoreInBackground] waitForResult:error];
 }
 
 - (void)fetchFromLocalDatastoreInBackgroundWithBlock:(PFObjectResultBlock)block {

Parse/PFUser.m

@@ -1113,11 +1113,11 @@ - (BFTask *)fetchAsync:(BFTask *)toAwait {
     }];
 }
 
-- (void)fetch:(NSError **)error {
+- (instancetype)fetch:(NSError **)error {
     if (self.isLazy) {
-        return;
+        return nil;
     }
-    [super fetch:error];
+    return [super fetch:error];
 }
 
 - (void)fetchInBackgroundWithBlock:(PFObjectResultBlock)block {