Fix nullability warnings introduced in Xcode 10

tiennou · tiennou · commit cfb487c421ab · 2018-11-18T17:16:06.000+01:00
diff --git a/ObjectiveGit/GTBlameHunk.m b/ObjectiveGit/GTBlameHunk.m
@@ -39,7 +39,9 @@ - (GTSignature *)finalSignature {
 }
 
 - (NSString *)originalPath {
-	return @(self.git_blame_hunk.orig_path);
+	NSString *path = @(self.git_blame_hunk.orig_path);
+	NSAssert(path, @"string was nil");
+	return path;
 }
 
 - (BOOL)isBoundary {
diff --git a/ObjectiveGit/GTConfiguration.m b/ObjectiveGit/GTConfiguration.m
@@ -112,7 +112,10 @@ - (BOOL)deleteValueForKey:(NSString *)key error:(NSError **)error {
 static int configCallback(const git_config_entry *entry, void *payload) {
 	NSMutableArray *configurationKeysArray = (__bridge NSMutableArray *)payload;
 
-	[configurationKeysArray addObject:@(entry->name)];
+	NSString *name = @(entry->name);
+	NSCAssert(name, @"string was nil");
+
+	[configurationKeysArray addObject:name];
 
 	return 0;
 }
diff --git a/ObjectiveGit/GTDiffFile.m b/ObjectiveGit/GTDiffFile.m
@@ -22,8 +22,9 @@ - (instancetype)initWithGitDiffFile:(git_diff_file)file {
 	self = [super init];
 	if (self == nil) return nil;
 
-	_path = @(file.path);
-	if (_path == nil) return nil;
+	NSString *path = @(file.path);
+	if (path == nil) return nil;
+	_path = path;
 
 	_git_diff_file = file;
 	_size = (NSUInteger)file.size;
diff --git a/ObjectiveGit/GTFilterSource.m b/ObjectiveGit/GTFilterSource.m
@@ -27,10 +27,13 @@ - (instancetype)initWithGitFilterSource:(const git_filter_source *)source {
 	self = [super init];
 	if (self == nil) return nil;
 
-	const char *path = git_repository_workdir(git_filter_source_repo(source));
-	_repositoryURL = [NSURL fileURLWithPath:@(path)];
-	
-	_path = @(git_filter_source_path(source));
+	NSString *path = @(git_repository_workdir(git_filter_source_repo(source)));
+	NSAssert(path, @"workdir was nil");
+	_repositoryURL = [NSURL fileURLWithPath:path];
+
+	path = @(git_filter_source_path(source));
+	NSAssert(path, @"path was nil");
+	_path = path;
 
 	const git_oid *gitOid = git_filter_source_id(source);
 	if (gitOid != NULL) _OID = [[GTOID alloc] initWithGitOid:gitOid];
diff --git a/ObjectiveGit/GTIndexEntry.m b/ObjectiveGit/GTIndexEntry.m
@@ -74,7 +74,9 @@ - (instancetype)initWithGitIndexEntry:(const git_index_entry *)entry {
 #pragma mark Properties
 
 - (NSString *)path {
-	return @(self.git_index_entry->path);
+	NSString *path = @(self.git_index_entry->path);
+	NSAssert(path, @"path is nil");
+	return path;
 }
 
 - (int)flags {
diff --git a/ObjectiveGit/GTNote.m b/ObjectiveGit/GTNote.m
@@ -42,7 +42,9 @@ - (git_note *)git_note {
 }
 
 - (NSString *)note {
-	return @(git_note_message(self.git_note));
+	NSString *message = @(git_note_message(self.git_note));
+	NSAssert(message, @"message is nil");
+	return message;
 }
 
 - (GTSignature *)author {
diff --git a/ObjectiveGit/GTReference.m b/ObjectiveGit/GTReference.m
@@ -118,10 +118,12 @@ - (BOOL)isNote {
 }
 
 - (NSString *)name {
-	const char *refName = git_reference_name(self.git_reference);
-	NSAssert(refName != nil, @"Unexpected nil name");
+	const char *cRefName = git_reference_name(self.git_reference);
+	NSAssert(cRefName != nil, @"Unexpected nil name");
 
-	return @(refName);
+	NSString *refName = @(cRefName);
+	NSAssert(refName, @"refname is nil");
+	return refName;
 }
 
 - (GTReference *)referenceByRenaming:(NSString *)newName error:(NSError **)error {
diff --git a/ObjectiveGit/GTRepository+RemoteOperations.m b/ObjectiveGit/GTRepository+RemoteOperations.m
@@ -126,9 +126,15 @@ int GTFetchHeadEntriesCallback(const char *ref_name, const char *remote_url, con
 	GTRepository *repository = entriesPayload->repository;
 	GTRemoteEnumerateFetchHeadEntryBlock enumerationBlock = entriesPayload->enumerationBlock;
 
-	GTReference *reference = [repository lookUpReferenceWithName:@(ref_name) error:NULL];
+	NSString *refName = @(ref_name);
+	NSCAssert(refName, @"refName is nil");
 
-	GTFetchHeadEntry *entry = [[GTFetchHeadEntry alloc] initWithReference:reference remoteURLString:@(remote_url) targetOID:[GTOID oidWithGitOid:oid] isMerge:(BOOL)is_merge];
+	NSString *remoteURL = @(remote_url);
+	NSCAssert(remote_url, @"remoteURL is nil");
+
+	GTReference *reference = [repository lookUpReferenceWithName:refName error:NULL];
+
+	GTFetchHeadEntry *entry = [[GTFetchHeadEntry alloc] initWithReference:reference remoteURLString:remoteURL targetOID:[GTOID oidWithGitOid:oid] isMerge:(BOOL)is_merge];
 
 	BOOL stop = NO;
 
diff --git a/ObjectiveGit/GTRepository.m b/ObjectiveGit/GTRepository.m
@@ -637,18 +637,22 @@ - (NSArray *)referenceNamesWithError:(NSError **)error {
 }
 
 - (NSURL *)fileURL {
-	const char *path = git_repository_workdir(self.git_repository);
+	const char *cPath = git_repository_workdir(self.git_repository);
 	// bare repository, you may be looking for gitDirectoryURL
-	if (path == NULL) return nil;
+	if (cPath == NULL) return nil;
 
-	return [NSURL fileURLWithPath:@(path) isDirectory:YES];
+	NSString *path = @(cPath);
+	NSAssert(path, @"workdir is nil");
+	return [NSURL fileURLWithPath:path isDirectory:YES];
 }
 
 - (NSURL *)gitDirectoryURL {
-	const char *path = git_repository_path(self.git_repository);
-	if (path == NULL) return nil;
+	const char *cPath = git_repository_path(self.git_repository);
+	if (cPath == NULL) return nil;
 
-	return [NSURL fileURLWithPath:@(path) isDirectory:YES];
+	NSString *path = @(cPath);
+	NSAssert(path, @"gitdirectory is nil");
+	return [NSURL fileURLWithPath:path isDirectory:YES];
 }
 
 - (BOOL)isBare {
@@ -737,7 +741,9 @@ static int submoduleEnumerationCallback(git_submodule *git_submodule, const char
 
 	NSError *error;
 	// Use -submoduleWithName:error: so that we get a git_submodule that we own.
-	GTSubmodule *submodule = [info->parentRepository submoduleWithName:@(name) error:&error];
+	NSString *submoduleName = @(name);
+	NSCAssert(submoduleName, @"submodule name is nil");
+	GTSubmodule *submodule = [info->parentRepository submoduleWithName:submoduleName error:&error];
 
 	BOOL stop = NO;
 	info->block(submodule, error, &stop);
diff --git a/ObjectiveGit/GTSubmodule.m b/ObjectiveGit/GTSubmodule.m
@@ -62,21 +62,30 @@ - (NSString *)name {
 	const char *cName = git_submodule_name(self.git_submodule);
 	NSAssert(cName != NULL, @"Unexpected nil submodule name");
 
-	return @(cName);
+	NSString *name = @(cName);
+	NSAssert(name, @"name is nil");
+
+	return name;
 }
 
 - (NSString *)path {
 	const char *cPath = git_submodule_path(self.git_submodule);
 	NSAssert(cPath != NULL, @"Unexpected nil submodule path");
 
-	return @(cPath);
+	NSString *path = @(cPath);
+	NSAssert(path, @"message is nil");
+
+	return path;
 }
 
 - (NSString *)URLString {
 	const char *cURL = git_submodule_url(self.git_submodule);
 	NSAssert(cURL != NULL, @"Unexpected nil submodule URL");
 
-	return @(cURL);
+	NSString *URL = @(cURL);
+	NSAssert(URL, @"URL is nil");
+
+	return URL;
 }
 
 #pragma mark Lifecycle
diff --git a/ObjectiveGit/GTTag.m b/ObjectiveGit/GTTag.m
@@ -47,11 +47,15 @@ - (NSString *)description {
 #pragma mark API
 
 - (NSString *)message {
-	return @(git_tag_message(self.git_tag));
+	NSString *message = @(git_tag_message(self.git_tag));
+	NSAssert(message, @"message is nil");
+	return message;
 }
 
 - (NSString *)name {
-	return @(git_tag_name(self.git_tag));
+	NSString *name = @(git_tag_name(self.git_tag));
+	NSAssert(name, @"message is nil");
+	return name;
 }
 
 - (GTObject *)target {
diff --git a/ObjectiveGit/GTTreeEntry.m b/ObjectiveGit/GTTreeEntry.m
@@ -94,7 +94,9 @@ + (instancetype)entryWithEntry:(const git_tree_entry *)theEntry parentTree:(GTTr
 }
 
 - (NSString *)name {
-	return @(git_tree_entry_name(self.git_tree_entry));
+	NSString *name = @(git_tree_entry_name(self.git_tree_entry));
+	NSAssert(name, @"name was nil");
+	return name;
 }
 
 - (NSInteger)attributes {

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,9 @@ - (GTSignature *)finalSignature {`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`- (NSString *)originalPath {`
`42`		`- return @(self.git_blame_hunk.orig_path);`
	`42`	`+ NSString *path = @(self.git_blame_hunk.orig_path);`
	`43`	`+ NSAssert(path, @"string was nil");`
	`44`	`+ return path;`
`43`	`45`	`}`
`44`	`46`
`45`	`47`	`- (BOOL)isBoundary {`
Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,9 @@ - (instancetype)initWithGitIndexEntry:(const git_index_entry *)entry {`
`74`	`74`	`#pragma mark Properties`
`75`	`75`
`76`	`76`	`- (NSString *)path {`
`77`		`- return @(self.git_index_entry->path);`
	`77`	`+ NSString *path = @(self.git_index_entry->path);`
	`78`	`+ NSAssert(path, @"path is nil");`
	`79`	`+ return path;`
`78`	`80`	`}`
`79`	`81`
`80`	`82`	`- (int)flags {`
Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,9 @@ - (git_note *)git_note {`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`- (NSString *)note {`
`45`		`- return @(git_note_message(self.git_note));`
	`45`	`+ NSString *message = @(git_note_message(self.git_note));`
	`46`	`+ NSAssert(message, @"message is nil");`
	`47`	`+ return message;`
`46`	`48`	`}`
`47`	`49`
`48`	`50`	`- (GTSignature *)author {`
Original file line number	Diff line number	Diff line change
`@@ -118,10 +118,12 @@ - (BOOL)isNote {`
`118`	`118`	`}`
`119`	`119`
`120`	`120`	`- (NSString *)name {`
`121`		`- const char *refName = git_reference_name(self.git_reference);`
`122`		`- NSAssert(refName != nil, @"Unexpected nil name");`
	`121`	`+ const char *cRefName = git_reference_name(self.git_reference);`
	`122`	`+ NSAssert(cRefName != nil, @"Unexpected nil name");`
`123`	`123`
`124`		`- return @(refName);`
	`124`	`+ NSString *refName = @(cRefName);`
	`125`	`+ NSAssert(refName, @"refname is nil");`
	`126`	`+ return refName;`
`125`	`127`	`}`
`126`	`128`
`127`	`129`	`- (GTReference )referenceByRenaming:(NSString )newName error:(NSError **)error {`
Original file line number	Diff line number	Diff line change
`@@ -94,7 +94,9 @@ + (instancetype)entryWithEntry:(const git_tree_entry *)theEntry parentTree:(GTTr`
`94`	`94`	`}`
`95`	`95`
`96`	`96`	`- (NSString *)name {`
`97`		`- return @(git_tree_entry_name(self.git_tree_entry));`
	`97`	`+ NSString *name = @(git_tree_entry_name(self.git_tree_entry));`
	`98`	`+ NSAssert(name, @"name was nil");`
	`99`	`+ return name;`
`98`	`100`	`}`
`99`	`101`
`100`	`102`	`- (NSInteger)attributes {`