From c4a6c49c2c61cf0eaaf94ffaf24207ce212deeac Mon Sep 17 00:00:00 2001
From: Marius Schulz <marius.schulz@me.com>
Date: Sat, 5 Sep 2015 19:05:32 +0200
Subject: [PATCH] Documents `TakeSkip` and simplifies implementation

---
 src/app/ExtraLINQ/IEnumerable/TakeSkip.cs | 21 +++++++++++----------
 1 file changed, 11 insertions(+), 10 deletions(-)

diff --git a/src/app/ExtraLINQ/IEnumerable/TakeSkip.cs b/src/app/ExtraLINQ/IEnumerable/TakeSkip.cs
index 892fb21..42e8338 100644
--- a/src/app/ExtraLINQ/IEnumerable/TakeSkip.cs
+++ b/src/app/ExtraLINQ/IEnumerable/TakeSkip.cs
@@ -4,6 +4,15 @@ namespace ExtraLinq
 {
     public static partial class EnumerableExtensions
     {
+        /// <summary>
+        /// Iterates over the given sequence and repeatedly returns <paramref name="take"/> elements
+        /// and skips <paramref name="skip"/> elements.
+        /// </summary>
+        /// <typeparam name="TSource">The type of the elements of <paramref name="source"/>.</typeparam>
+        /// <param name="source">The sequence to iterate over.</param>
+        /// <param name="take">The number of elements to take at each step.</param>
+        /// <param name="skip">The number of elements to skip after each step.</param>
+        /// <returns>A sequence containing the taken elements.</returns>
         public static IEnumerable<TSource> TakeSkip<TSource>(this IEnumerable<TSource> source, int take, int skip)
         {
             ThrowIf.Argument.IsNull(source, "source");
@@ -21,24 +30,16 @@ private static IEnumerable<TSource> TakeSkipIterator<TSource>(IEnumerable<TSourc
             {
                 for (int i = 0; i < take; i++)
                 {
-                    bool hasNext = enumerator.MoveNext();
-
-                    if (!hasNext)
-                    {
+                    if (!enumerator.MoveNext())
                         yield break;
-                    }
 
                     yield return enumerator.Current;
                 }
 
                 for (int i = 0; i < skip; i++)
                 {
-                    bool hasNext = enumerator.MoveNext();
-
-                    if (!hasNext)
-                    {
+                    if (!enumerator.MoveNext())
                         yield break;
-                    }
                 }
             }
         }