Update to libgit2 v0.28.0-rc1

tiennou · tiennou · commit 7ab537e53f81 · 2019-02-08T23:36:17.000+01:00
diff --git a/External/libgit2 b/External/libgit2
@@ -1 +1 @@
-Subproject commit f23dc5b29f1394928a940d7ec447f4bfd53dad1f
+Subproject commit 1a107fac0fc88a4d74b64ffc9ae2fd178ba631c0
diff --git a/ObjectiveGit/Categories/NSData+Git.m b/ObjectiveGit/Categories/NSData+Git.m
@@ -17,7 +17,7 @@ - (BOOL)git_getOid:(git_oid *)oid error:(NSError **)error {
     if ([self length] != sizeof(git_oid)) {
         if (error != NULL) {
             *error = [NSError errorWithDomain:GTGitErrorDomain 
-                                         code:GITERR_INVALID 
+                                         code:GIT_ERROR_INVALID
                                      userInfo:
                       [NSDictionary dictionaryWithObject:@"can't extract oid from data of incorrect length" 
                                                   forKey:NSLocalizedDescriptionKey]];
diff --git a/ObjectiveGit/Categories/NSError+Git.m b/ObjectiveGit/Categories/NSError+Git.m
@@ -98,10 +98,10 @@ + (NSError *)git_errorFor:(int)code {
 }
 
 + (NSString *)git_descriptionForErrorCode:(int)code {
-	const git_error *gitLastError = giterr_last();
+	const git_error *gitLastError = git_error_last();
 	if (gitLastError != NULL) {
 		return @(gitLastError->message);
-	} else if (code == GITERR_OS) {
+	} else if (code == GIT_ERROR_OS) {
 		return @(strerror(errno));
 	} else {
 		return nil;
diff --git a/ObjectiveGit/GTBlob.m b/ObjectiveGit/GTBlob.m
@@ -63,7 +63,7 @@ - (instancetype)initWithOid:(const git_oid *)oid inRepository:(GTRepository *)re
 	NSParameterAssert(repository != nil);
 
 	git_object *obj;
-    int gitError = git_object_lookup(&obj, repository.git_repository, oid, (git_otype) GTObjectTypeBlob);
+    int gitError = git_object_lookup(&obj, repository.git_repository, oid, (git_object_t) GTObjectTypeBlob);
     if (gitError < GIT_OK) {
         if (error != NULL) {
             *error = [NSError git_errorFor:gitError description:@"Failed to lookup blob"];
diff --git a/ObjectiveGit/GTCredential.m b/ObjectiveGit/GTCredential.m
@@ -105,7 +105,7 @@ int GTCredentialAcquireCallback(git_cred **git_cred, const char *url, const char
 	GTCredentialProvider *provider = info->credProvider;
 
 	if (provider == nil) {
-		giterr_set_str(GIT_EUSER, "No GTCredentialProvider set, but authentication was requested.");
+		git_error_set_str(GIT_EUSER, "No GTCredentialProvider set, but authentication was requested.");
 		return GIT_ERROR;
 	}
 
@@ -114,7 +114,7 @@ int GTCredentialAcquireCallback(git_cred **git_cred, const char *url, const char
 
 	GTCredential *cred = [provider credentialForType:(GTCredentialType)allowed_types URL:URL userName:userName];
 	if (cred == nil) {
-		giterr_set_str(GIT_EUSER, "GTCredentialProvider failed to provide credentials.");
+		git_error_set_str(GIT_EUSER, "GTCredentialProvider failed to provide credentials.");
 		return GIT_ERROR;
 	}
 
diff --git a/ObjectiveGit/GTDiffPatch.m b/ObjectiveGit/GTDiffPatch.m
@@ -69,7 +69,7 @@ - (NSData *)patchData {
 	git_patch_to_buf(&buf, self.git_patch);
 
 	NSData *buffer = [[NSData alloc] initWithBytes:buf.ptr length:buf.size];
-	git_buf_free(&buf);
+	git_buf_dispose(&buf);
 	
 	return buffer;
 }
diff --git a/ObjectiveGit/GTIndexEntry.m b/ObjectiveGit/GTIndexEntry.m
@@ -127,7 +127,7 @@ + (instancetype)objectWithIndexEntry:(GTIndexEntry *)indexEntry error:(NSError *
 
 - (instancetype)initWithIndexEntry:(GTIndexEntry *)indexEntry error:(NSError **)error {
 	git_object *obj;
-	int gitError = git_object_lookup(&obj, indexEntry.repository.git_repository, indexEntry.OID.git_oid, (git_otype)GTObjectTypeAny);
+	int gitError = git_object_lookup(&obj, indexEntry.repository.git_repository, indexEntry.OID.git_oid, (git_object_t)GTObjectTypeAny);
 
 	if (gitError < GIT_OK) {
 		if (error != NULL) {
diff --git a/ObjectiveGit/GTNote.m b/ObjectiveGit/GTNote.m
@@ -98,7 +98,7 @@ + (NSString *)defaultReferenceNameForRepository:(GTRepository *)repository error
 		if (error != NULL) *error = [NSError git_errorFor:GIT_ERROR description:@"Unable to get default git notes reference name"];
 	}
 	
-	git_buf_free(&default_ref_name);
+	git_buf_dispose(&default_ref_name);
 	
 	return noteRef;
 }
diff --git a/ObjectiveGit/GTOID.m b/ObjectiveGit/GTOID.m
@@ -133,7 +133,7 @@ + (instancetype)OIDByHashingData:(NSData *)data type:(GTObjectType)type error:(N
 	NSParameterAssert(data != nil);
 
 	git_oid oid;
-	int gitError = git_odb_hash(&oid, data.bytes, data.length, (git_otype)type);
+	int gitError = git_odb_hash(&oid, data.bytes, data.length, (git_object_t)type);
 	if (gitError != GIT_OK) {
 		if (error != NULL) *error = [NSError git_errorFor:gitError description:@"Failed to hash"];
 		return nil;
diff --git a/ObjectiveGit/GTObject.h b/ObjectiveGit/GTObject.h
@@ -31,16 +31,14 @@
 #import "git2/types.h"
 
 typedef NS_ENUM(int, GTObjectType) {
-	GTObjectTypeAny = GIT_OBJ_ANY,				/**< Object can be any of the following */
-	GTObjectTypeBad = GIT_OBJ_BAD,				/**< Object is invalid. */
-	GTObjectTypeExt1 = GIT_OBJ__EXT1,			/**< Reserved for future use. */
-	GTObjectTypeCommit = GIT_OBJ_COMMIT,		/**< A commit object. */
-	GTObjectTypeTree = GIT_OBJ_TREE,			/**< A tree (directory listing) object. */
-	GTObjectTypeBlob = GIT_OBJ_BLOB,			/**< A file revision object. */
-	GTObjectTypeTag = GIT_OBJ_TAG,				/**< An annotated tag object. */
-	GTObjectTypeExt2 = GIT_OBJ__EXT2,			/**< Reserved for future use. */
-	GTObjectTypeOffsetDelta = GIT_OBJ_OFS_DELTA,/**< A delta, base is given by an offset. */
-	GTObjectTypeRefDelta = GIT_OBJ_REF_DELTA,	/**< A delta, base is given by object id. */
+	GTObjectTypeAny = GIT_OBJECT_ANY,				/**< Object can be any of the following */
+	GTObjectTypeBad = GIT_OBJECT_INVALID,				/**< Object is invalid. */
+	GTObjectTypeCommit = GIT_OBJECT_COMMIT,		/**< A commit object. */
+	GTObjectTypeTree = GIT_OBJECT_TREE,			/**< A tree (directory listing) object. */
+	GTObjectTypeBlob = GIT_OBJECT_BLOB,			/**< A file revision object. */
+	GTObjectTypeTag = GIT_OBJECT_TAG,				/**< An annotated tag object. */
+	GTObjectTypeOffsetDelta = GIT_OBJECT_OFS_DELTA,/**< A delta, base is given by an offset. */
+	GTObjectTypeRefDelta = GIT_OBJECT_REF_DELTA,	/**< A delta, base is given by object id. */
 };
 
 @class GTRepository;
diff --git a/ObjectiveGit/GTObject.m b/ObjectiveGit/GTObject.m
@@ -82,26 +82,26 @@ - (id)initWithObj:(git_object *)object inRepository:(GTRepository *)repo {
 	NSAssert(object_repo == repo.git_repository, @"object %p doesn't belong to repo %@", object, repo);
 
 	Class objectClass = nil;
-	git_otype t = git_object_type(object);
+	git_object_t t = git_object_type(object);
 	switch (t) {
-		case GIT_OBJ_COMMIT:
+		case GIT_OBJECT_COMMIT:
 			objectClass = [GTCommit class];
 			break;
-		case GIT_OBJ_TREE:
+		case GIT_OBJECT_TREE:
 			objectClass = [GTTree class];
 			break;
-		case GIT_OBJ_BLOB:
+		case GIT_OBJECT_BLOB:
 			objectClass = [GTBlob class];
 			break;
-		case GIT_OBJ_TAG:
+		case GIT_OBJECT_TAG:
 			objectClass = [GTTag class];
 			break;
 		default:
 			break;
 	}
 
 	if (!objectClass) {
-		NSLog(@"Unknown git_otype %s (%d)", git_object_type2string(t), (int)t);
+		NSLog(@"Unknown git_object_t %s (%d)", git_object_type2string(t), (int)t);
 		return nil;
 	}
 	
@@ -149,7 +149,7 @@ - (GTOdbObject *)odbObjectWithError:(NSError **)error {
 
 - (id)objectByPeelingToType:(GTObjectType)type error:(NSError **)error {
 	git_object *peeled = NULL;
-	int gitError = git_object_peel(&peeled, self.git_object, (git_otype)type);
+	int gitError = git_object_peel(&peeled, self.git_object, (git_object_t)type);
 	if (gitError != GIT_OK) {
 		if (error != NULL) *error = [NSError git_errorFor:gitError description:@"Cannot peel object"];
 		return nil;
diff --git a/ObjectiveGit/GTObjectDatabase.m b/ObjectiveGit/GTObjectDatabase.m
@@ -102,7 +102,7 @@ - (GTOID *)writeData:(NSData *)data type:(GTObjectType)type error:(NSError **)er
 	NSParameterAssert(data != nil);
 
 	git_odb_stream *stream;
-	int gitError = git_odb_open_wstream(&stream, self.git_odb, data.length, (git_otype)type);
+	int gitError = git_odb_open_wstream(&stream, self.git_odb, data.length, (git_object_t)type);
 	@onExit {
 		if (stream != NULL) git_odb_stream_free(stream);
 	};
diff --git a/ObjectiveGit/GTReference.h b/ObjectiveGit/GTReference.h
@@ -34,9 +34,9 @@ typedef NS_ENUM(NSInteger, GTReferenceErrorCode) {
 };
 
 typedef NS_OPTIONS(NSInteger, GTReferenceType) {
-	GTReferenceTypeInvalid =    GIT_REF_INVALID,  /** Invalid reference */
-	GTReferenceTypeOid =        GIT_REF_OID,      /** A reference which points at an object id */
-	GTReferenceTypeSymbolic =   GIT_REF_SYMBOLIC, /** A reference which points at another reference */
+	GTReferenceTypeInvalid =    GIT_REFERENCE_INVALID,  /** Invalid reference */
+	GTReferenceTypeDirect =     GIT_REFERENCE_DIRECT,   /** A reference which points at an object id */
+	GTReferenceTypeSymbolic =   GIT_REFERENCE_SYMBOLIC, /** A reference which points at another reference */
 };
 
 NS_ASSUME_NONNULL_BEGIN
diff --git a/ObjectiveGit/GTReference.m b/ObjectiveGit/GTReference.m
@@ -45,7 +45,7 @@ @interface GTReference ()
 		case GTReferenceTypeInvalid:
 			return @"invalid";
 
-		case GTReferenceTypeOid:
+		case GTReferenceTypeDirect:
 			return @"direct";
 
 		case GTReferenceTypeSymbolic:
@@ -144,7 +144,7 @@ - (GTReferenceType)referenceType {
 }
 
 - (id)unresolvedTarget {
-	if (self.referenceType == GTReferenceTypeOid) {
+	if (self.referenceType == GTReferenceTypeDirect) {
 		const git_oid *oid = git_reference_target(self.git_reference);
 		if (oid == NULL) return nil;
 
@@ -160,7 +160,7 @@ - (id)unresolvedTarget {
 
 - (id)resolvedTarget {
 	git_object *obj;
-	if (git_reference_peel(&obj, self.git_reference, GIT_OBJ_ANY) != GIT_OK) {
+	if (git_reference_peel(&obj, self.git_reference, GIT_OBJECT_ANY) != GIT_OK) {
 		return nil;
 	}
 
@@ -181,7 +181,7 @@ - (GTReference *)referenceByUpdatingTarget:(NSString *)newTarget message:(NSStri
 
 	int gitError;
 	git_reference *newRef = NULL;
-	if (git_reference_type(self.git_reference) == GIT_REF_OID) {
+	if (git_reference_type(self.git_reference) == GIT_REFERENCE_DIRECT) {
 		GTOID *oid = [[GTOID alloc] initWithSHA:newTarget error:error];
 		if (oid == nil) return nil;
 
diff --git a/ObjectiveGit/GTRepository.m b/ObjectiveGit/GTRepository.m
@@ -317,7 +317,7 @@ + (instancetype _Nullable)cloneFromURL:(NSURL *)originURL toWorkingDirectory:(NS
 - (id)lookUpObjectByGitOid:(const git_oid *)oid objectType:(GTObjectType)type error:(NSError **)error {
 	git_object *obj;
 
-	int gitError = git_object_lookup(&obj, self.git_repository, oid, (git_otype)type);
+	int gitError = git_object_lookup(&obj, self.git_repository, oid, (git_object_t)type);
 	if (gitError < GIT_OK) {
 		if (error != NULL) {
 			char oid_str[GIT_OID_HEXSZ+1];
@@ -683,13 +683,13 @@ - (NSString *)preparedMessageWithError:(NSError * __autoreleasing *)error {
 	int errorCode = git_repository_message(&msg, self.git_repository);
 	if (errorCode != GIT_OK) {
 		setErrorFromCode(errorCode);
-		git_buf_free(&msg);
+		git_buf_dispose(&msg);
 		return nil;
 	}
 
 	NSString *message = [[NSString alloc] initWithBytes:msg.ptr length:msg.size encoding:NSUTF8StringEncoding];
 
-	git_buf_free(&msg);
+	git_buf_dispose(&msg);
 
 	return message;
 }
diff --git a/ObjectiveGit/GTTreeBuilder.m b/ObjectiveGit/GTTreeBuilder.m
@@ -155,7 +155,7 @@ - (GTTree *)writeTree:(NSError **)error {
 	}
 
 	git_object *object = NULL;
-	status = git_object_lookup(&object, self.repository.git_repository, &treeOid, GIT_OBJ_TREE);
+	status = git_object_lookup(&object, self.repository.git_repository, &treeOid, GIT_OBJECT_TREE);
 	if (status != GIT_OK) {
 		if (error != NULL) *error = [NSError git_errorFor:status description:@"Failed to lookup tree in repository."];
 		return nil;
diff --git a/ObjectiveGitTests/GTReferenceSpec.m b/ObjectiveGitTests/GTReferenceSpec.m
@@ -135,7 +135,7 @@
 		expect(ref).notTo(beNil());
 		expect(error).to(beNil());
 
-		expectValidReference(ref, @"36060c58702ed4c2a40832c51758d5344201d89a", GTReferenceTypeOid, @"refs/heads/master");
+		expectValidReference(ref, @"36060c58702ed4c2a40832c51758d5344201d89a", GTReferenceTypeDirect, @"refs/heads/master");
 	});
 
 	it(@"should return a valid reference to a tag", ^{
@@ -144,7 +144,7 @@
 		expect(ref).notTo(beNil());
 		expect(error).to(beNil());
 
-		expectValidReference(ref, @"5b5b025afb0b4c913b4c338a42934a3863bf3644", GTReferenceTypeOid, @"refs/tags/v0.9");
+		expectValidReference(ref, @"5b5b025afb0b4c913b4c338a42934a3863bf3644", GTReferenceTypeDirect, @"refs/tags/v0.9");
 	});
 });
 
@@ -170,7 +170,7 @@
 		expect(error).to(beNil());
 		expect(ref).notTo(beNil());
 
-		expectValidReference(ref, @"36060c58702ed4c2a40832c51758d5344201d89a", GTReferenceTypeOid, @"refs/heads/unit_test");
+		expectValidReference(ref, @"36060c58702ed4c2a40832c51758d5344201d89a", GTReferenceTypeDirect, @"refs/heads/unit_test");
 	});
 });
 
diff --git a/ObjectiveGitTests/GTRepositorySpec.m b/ObjectiveGitTests/GTRepositorySpec.m
@@ -114,7 +114,7 @@
 			expect(head).notTo(beNil());
 			expect(error).to(beNil());
 			expect(head.targetOID.SHA).to(equal(@"36060c58702ed4c2a40832c51758d5344201d89a"));
-			expect(@(head.referenceType)).to(equal(@(GTReferenceTypeOid)));
+			expect(@(head.referenceType)).to(equal(@(GTReferenceTypeDirect)));
 		});
 
 		it(@"should handle bare clones", ^{
@@ -139,7 +139,7 @@
 			expect(head).notTo(beNil());
 			expect(error).to(beNil());
 			expect(head.targetOID.SHA).to(equal(@"36060c58702ed4c2a40832c51758d5344201d89a"));
-			expect(@(head.referenceType)).to(equal(@(GTReferenceTypeOid)));
+			expect(@(head.referenceType)).to(equal(@(GTReferenceTypeDirect)));
 		});
 
 		it(@"should have set a valid remote URL", ^{
@@ -212,7 +212,7 @@
 		expect(head).notTo(beNil());
 		expect(error).to(beNil());
 		expect(head.targetOID.SHA).to(equal(@"36060c58702ed4c2a40832c51758d5344201d89a"));
-		expect(@(head.referenceType)).to(equal(@(GTReferenceTypeOid)));
+		expect(@(head.referenceType)).to(equal(@(GTReferenceTypeDirect)));
 	});
 
 	it(@"should fail to return HEAD for an unborn repo", ^{

Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@ int GTCredentialAcquireCallback(git_cred *git_cred, const char url, const char`
`105`	`105`	`GTCredentialProvider *provider = info->credProvider;`
`106`	`106`
`107`	`107`	`if (provider == nil) {`
`108`		`- giterr_set_str(GIT_EUSER, "No GTCredentialProvider set, but authentication was requested.");`
	`108`	`+ git_error_set_str(GIT_EUSER, "No GTCredentialProvider set, but authentication was requested.");`
`109`	`109`	`return GIT_ERROR;`
`110`	`110`	`}`
`111`	`111`
`@@ -114,7 +114,7 @@ int GTCredentialAcquireCallback(git_cred *git_cred, const char url, const char`
`114`	`114`
`115`	`115`	`GTCredential *cred = [provider credentialForType:(GTCredentialType)allowed_types URL:URL userName:userName];`
`116`	`116`	`if (cred == nil) {`
`117`		`- giterr_set_str(GIT_EUSER, "GTCredentialProvider failed to provide credentials.");`
	`117`	`+ git_error_set_str(GIT_EUSER, "GTCredentialProvider failed to provide credentials.");`
`118`	`118`	`return GIT_ERROR;`
`119`	`119`	`}`
`120`	`120`
Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ - (NSData *)patchData {`
`69`	`69`	`git_patch_to_buf(&buf, self.git_patch);`
`70`	`70`
`71`	`71`	`NSData *buffer = [[NSData alloc] initWithBytes:buf.ptr length:buf.size];`
`72`		`- git_buf_free(&buf);`
	`72`	`+ git_buf_dispose(&buf);`
`73`	`73`
`74`	`74`	`return buffer;`
`75`	`75`	`}`
Original file line number	Diff line number	Diff line change
`@@ -98,7 +98,7 @@ + (NSString )defaultReferenceNameForRepository:(GTRepository )repository error`
`98`	`98`	`if (error != NULL) *error = [NSError git_errorFor:GIT_ERROR description:@"Unable to get default git notes reference name"];`
`99`	`99`	`}`
`100`	`100`
`101`		`- git_buf_free(&default_ref_name);`
	`101`	`+ git_buf_dispose(&default_ref_name);`
`102`	`102`
`103`	`103`	`return noteRef;`
`104`	`104`	`}`