Unify BuildFromPtr/CreateFromPtr implementation and usage

Author: yorah

LibGit2Sharp/GitObject.cs

@@ -51,7 +51,7 @@ public virtual string Sha
             get { return Id.Sha; }
         }
 
-        internal static GitObject CreateFromPtr(GitObjectSafeHandle obj, ObjectId id, Repository repo, FilePath path)
+        internal static GitObject BuildFromPtr(GitObjectSafeHandle obj, ObjectId id, Repository repo, FilePath path)
         {
             GitObjectType type = Proxy.git_object_type(obj);
             switch (type)

LibGit2Sharp/Index.cs

@@ -81,7 +81,7 @@ private IndexEntry this[uint index]
             get
             {
                 IndexEntrySafeHandle entryHandle = Proxy.git_index_get(handle, index);
-                return IndexEntry.CreateFromPtr(repo, entryHandle);
+                return IndexEntry.BuildFromPtr(repo, entryHandle);
             }
         }
 

LibGit2Sharp/IndexEntry.cs

@@ -38,7 +38,7 @@ public FileStatus State
         /// </summary>
         public ObjectId Id { get; private set; }
 
-        internal static IndexEntry CreateFromPtr(Repository repo, IndexEntrySafeHandle handle)
+        internal static IndexEntry BuildFromPtr(Repository repo, IndexEntrySafeHandle handle)
         {
             GitIndexEntry entry = handle.MarshalAsGitIndexEntry();
 

LibGit2Sharp/Note.cs

@@ -44,7 +44,7 @@ private Note(ObjectId blobId, string message, ObjectId targetObjectId, string @n
         /// </summary>
         public virtual ObjectId TargetObjectId { get; private set; }
 
-        internal static Note BuildFromPtr(string @namespace, ObjectId targetObjectId, NoteSafeHandle note)
+        internal static Note BuildFromPtr(NoteSafeHandle note, string @namespace, ObjectId targetObjectId)
         {
             ObjectId oid = Proxy.git_note_oid(note);
             string message = Proxy.git_note_message(note);

LibGit2Sharp/NoteCollection.cs

@@ -120,7 +120,7 @@ internal Note RetrieveNote(ObjectId targetObjectId, string canonicalNamespace)
             using (NoteSafeHandle noteHandle = Proxy.git_note_read(repo.Handle, canonicalNamespace, targetObjectId))
             {
                 return noteHandle == null ? null :
-                    Note.BuildFromPtr(UnCanonicalizeName(canonicalNamespace), targetObjectId, noteHandle);
+                    Note.BuildFromPtr(noteHandle, UnCanonicalizeName(canonicalNamespace), targetObjectId);
             }
         }
 
@@ -211,7 +211,7 @@ public virtual void Remove(ObjectId targetId, Signature author, Signature commit
             Ensure.ArgumentNotNullOrEmptyString(@namespace, "@namespace");
 
             string canonicalNamespace = NormalizeToCanonicalName(@namespace);
-    
+
             Proxy.git_note_remove(repo.Handle, canonicalNamespace, author, committer, targetId);
         }
 

LibGit2Sharp/Remote.cs

@@ -18,13 +18,8 @@ public class Remote : IEquatable<Remote>
         protected Remote()
         { }
 
-        internal static Remote CreateFromPtr(RemoteSafeHandle handle)
+        internal static Remote BuildFromPtr(RemoteSafeHandle handle)
         {
-            if (handle == null)
-            {
-                return null;
-            }
-
             string name = Proxy.git_remote_name(handle);
             string url = Proxy.git_remote_url(handle);
 

LibGit2Sharp/RemoteCollection.cs

@@ -39,7 +39,7 @@ private Remote RemoteForName(string name)
         {
             using (RemoteSafeHandle handle = Proxy.git_remote_load(repository.Handle, name, false))
             {
-                return Remote.CreateFromPtr(handle);
+                return handle == null ? null : Remote.BuildFromPtr(handle);
             }
         }
 
@@ -80,7 +80,7 @@ public virtual Remote Add(string name, string url)
 
             using (RemoteSafeHandle handle = Proxy.git_remote_add(repository.Handle, name, url))
             {
-                return Remote.CreateFromPtr(handle);
+                return Remote.BuildFromPtr(handle);
             }
         }
 
@@ -115,7 +115,7 @@ public virtual Remote Add(string name, string url, string fetchRefSpec)
             using (RemoteSafeHandle handle = Proxy.git_remote_new(repository.Handle, name, url, fetchRefSpec))
             {
                 Proxy.git_remote_save(handle);
-                return Remote.CreateFromPtr(handle);
+                return Remote.BuildFromPtr(handle);
             }
         }
 

LibGit2Sharp/Repository.cs

@@ -306,7 +306,7 @@ internal GitObject LookupInternal(ObjectId id, GitObjectType type, FilePath know
                     obj = Proxy.git_object_lookup(handle, id, type);
                 }
 
-                return obj == null ? null : GitObject.CreateFromPtr(obj, id, this, knownPath);
+                return obj == null ? null : GitObject.BuildFromPtr(obj, id, this, knownPath);
             }
             finally
             {
@@ -363,7 +363,7 @@ internal GitObject Lookup(string objectish, GitObjectType type, LookUpOptions lo
                     return null;
                 }
 
-                obj = GitObject.CreateFromPtr(sh, GitObject.ObjectIdOf(sh), this, PathFromRevparseSpec(objectish));
+                obj = GitObject.BuildFromPtr(sh, GitObject.ObjectIdOf(sh), this, PathFromRevparseSpec(objectish));
             }
 
             if (lookUpOptions.Has(LookUpOptions.DereferenceResultToCommit))

LibGit2Sharp/Tree.cs

@@ -126,7 +126,7 @@ IEnumerator IEnumerable.GetEnumerator()
 
         #endregion
 
-        internal static Tree BuildFromPtr(GitObjectSafeHandle obj, ObjectId id, Repository repo, FilePath path)
+        internal new static Tree BuildFromPtr(GitObjectSafeHandle obj, ObjectId id, Repository repo, FilePath path)
         {
             var tree = new Tree(id, path, Proxy.git_tree_entrycount(obj), repo);
             return tree;