Add Count property, skip enumeration when only # of changes is important

niik · niik · commit c37b48e44271 · 2016-04-03T13:24:01.000-07:00
diff --git a/LibGit2Sharp/TreeChanges.cs b/LibGit2Sharp/TreeChanges.cs
@@ -39,8 +39,7 @@ internal unsafe TreeChanges(DiffHandle diff)
         private IEnumerable<TreeEntryChanges> GetChangesOfKind(ChangeKind changeKind)
         {
             TreeEntryChanges entry;
-
-            for (int i = 0; i < count.Value; i++)
+            for (int i = 0; i < Count; i++)
             {
                 if (TryGetEntryWithChangeTypeAt(i, changeKind, out entry))
                 {
@@ -78,7 +77,7 @@ private unsafe bool TryGetEntryWithChangeTypeAt(int index, ChangeKind changeKind
         /// <returns>An <see cref="IEnumerator{T}"/> object that can be used to iterate through the collection.</returns>
         public virtual IEnumerator<TreeEntryChanges> GetEnumerator()
         {
-            for (int i = 0; i < count.Value; i++)
+            for (int i = 0; i < Count; i++)
             {
                 yield return GetEntryAt(i);
             }
@@ -171,6 +170,14 @@ public virtual IEnumerable<TreeEntryChanges> Conflicted
             get { return GetChangesOfKind(ChangeKind.Conflicted); }
         }
 
+        /// <summary>
+        /// Gets the number of <see cref="TreeEntryChanges"/> in this comparison.
+        /// </summary>
+        public virtual int Count
+        {
+            get { return count.Value; }
+        }
+
         private string DebuggerDisplay
         {
             get

Original file line number	Diff line number	Diff line change
`@@ -39,8 +39,7 @@ internal unsafe TreeChanges(DiffHandle diff)`
`39`	`39`	`private IEnumerable<TreeEntryChanges> GetChangesOfKind(ChangeKind changeKind)`
`40`	`40`	`{`
`41`	`41`	`TreeEntryChanges entry;`
`42`		`-`
`43`		`- for (int i = 0; i < count.Value; i++)`
	`42`	`+ for (int i = 0; i < Count; i++)`
`44`	`43`	`{`
`45`	`44`	`if (TryGetEntryWithChangeTypeAt(i, changeKind, out entry))`
`46`	`45`	`{`
`@@ -78,7 +77,7 @@ private unsafe bool TryGetEntryWithChangeTypeAt(int index, ChangeKind changeKind`
`78`	`77`	`/// <returns>An <see cref="IEnumerator{T}"/> object that can be used to iterate through the collection.</returns>`
`79`	`78`	`public virtual IEnumerator<TreeEntryChanges> GetEnumerator()`
`80`	`79`	`{`
`81`		`- for (int i = 0; i < count.Value; i++)`
	`80`	`+ for (int i = 0; i < Count; i++)`
`82`	`81`	`{`
`83`	`82`	`yield return GetEntryAt(i);`
`84`	`83`	`}`
`@@ -171,6 +170,14 @@ public virtual IEnumerable<TreeEntryChanges> Conflicted`
`171`	`170`	`get { return GetChangesOfKind(ChangeKind.Conflicted); }`
`172`	`171`	`}`
`173`	`172`
	`173`	`+ /// <summary>`
	`174`	`+ /// Gets the number of <see cref="TreeEntryChanges"/> in this comparison.`
	`175`	`+ /// </summary>`
	`176`	`+ public virtual int Count`
	`177`	`+ {`
	`178`	`+ get { return count.Value; }`
	`179`	`+ }`
	`180`	`+`
`174`	`181`	`private string DebuggerDisplay`
`175`	`182`	`{`
`176`	`183`	`get`