finishedGraphing wasn't getting called when updateHistoryForRev:

This resulted in the progress indicator getting stuck on when viewing a stash.

Author: ksuther

Classes/git/PBGitHistoryList.m

@@ -318,11 +318,14 @@ - (void) updateProjectHistoryForRev:(PBGitRevSpecifier *)rev
 		lastRemoteRef = nil;
 		lastOID = nil;
 		self.commits = [NSMutableArray array];
-		[projectRevList loadRevisons];
-		return;
+		[projectRevList loadRevisonsWithCompletionBlock:^{
+			dispatch_async(dispatch_get_main_queue(), ^{
+				[self finishedGraphing];
+			});
+		}];
+	} else {
+		[graphQueue addOperation:[self operationForCommits:projectRevList.commits]];
 	}
-
-	[graphQueue addOperation:[self operationForCommits:projectRevList.commits]];
 }
 
 
@@ -337,7 +340,11 @@ - (void) updateHistoryForRev:(PBGitRevSpecifier *)rev
 	lastOID = nil;
 	self.commits = [NSMutableArray array];
 
-	[otherRevListParser loadRevisons];
+	[otherRevListParser loadRevisonsWithCompletionBlock:^{
+		dispatch_async(dispatch_get_main_queue(), ^{
+			[self finishedGraphing];
+		});
+	}];
 }
 
 

Classes/git/PBGitRevList.h

@@ -18,7 +18,7 @@
 @property (nonatomic, strong) NSMutableArray<PBGitCommit *> *commits;
 
 - (id) initWithRepository:(PBGitRepository *)repo rev:(PBGitRevSpecifier *)rev shouldGraph:(BOOL)graph;
-- (void) loadRevisons;
+- (void)loadRevisonsWithCompletionBlock:(void(^)(void))completionBlock;
 - (void)cancel;
 
 @end

Classes/git/PBGitRevList.mm

@@ -62,7 +62,7 @@ - (id) initWithRepository:(PBGitRepository *)repo rev:(PBGitRevSpecifier *)rev s
 	return self;
 }
 
-- (void) loadRevisons
+- (void)loadRevisonsWithCompletionBlock:(void(^)(void))completionBlock
 {
 	[self cancel];
 
@@ -83,6 +83,7 @@ - (void) loadRevisons
 		[weakSelf setupEnumerator:enu forRevspec:weakSelf.currentRev];
 		[weakSelf addCommitsFromEnumerator:enu inPBRepo:pbRepo operation:weakParseOperation];
 	}];
+	[parseOperation setCompletionBlock:completionBlock];
 
 	[self.operationQueue addOperation:parseOperation];
 }