Merge pull request #1275 from libgit2/cmn/equality-over-id

Don't load an object just to check its ID

Author: Edward Thomson

LibGit2Sharp/Branch.cs

@@ -106,7 +106,15 @@ public virtual BranchTrackingDetails TrackingDetails
         /// </value>
         public virtual bool IsCurrentRepositoryHead
         {
-            get { return repo.Head == this; }
+            get
+            {
+                if (this is DetachedHead)
+                {
+                    return repo.Head.reference.TargetIdentifier == this.reference.TargetIdentifier;
+                }
+
+                return repo.Head.reference.TargetIdentifier == this.CanonicalName;
+            }
         }
 
         /// <summary>

LibGit2Sharp/ReferenceWrapper.cs

@@ -16,10 +16,11 @@ public abstract class ReferenceWrapper<TObject> : IEquatable<ReferenceWrapper<TO
         /// The repository.
         /// </summary>
         protected readonly Repository repo;
+        protected readonly Reference reference;
         private readonly Lazy<TObject> objectBuilder;
 
         private static readonly LambdaEqualityHelper<ReferenceWrapper<TObject>> equalityHelper =
-            new LambdaEqualityHelper<ReferenceWrapper<TObject>>(x => x.CanonicalName, x => x.TargetObject);
+            new LambdaEqualityHelper<ReferenceWrapper<TObject>>(x => x.CanonicalName, x => x.reference.TargetIdentifier);
 
         private readonly string canonicalName;
 
@@ -40,6 +41,7 @@ protected internal ReferenceWrapper(Repository repo, Reference reference, Func<R
 
             this.repo = repo;
             canonicalName = canonicalNameSelector(reference);
+            this.reference = reference;
             objectBuilder = new Lazy<TObject>(() => RetrieveTargetObject(reference));
         }