let Queue be a range (#220)

let Queue be a range
merged-on-behalf-of: BBasile <BBasile@users.noreply.github.com>

source/dyaml/emitter.d

@@ -273,14 +273,8 @@ struct Emitter(Range, CharType) if (isOutputRange!(Range, CharType))
         {
             int level;
 
-            //Rather ugly, but good enough for now.
+            foreach(const event; events_.range)
-            //Couldn't be bothered writing a range as events_ should eventually
-            //become a Phobos queue/linked list.
-            events_.startIteration();
-            events_.next();
-            while(!events_.iterationOver())
             {
-                const event = events_.next();
                 static starts = [EventID.documentStart, EventID.sequenceStart, EventID.mappingStart];
                 static ends   = [EventID.documentEnd, EventID.sequenceEnd, EventID.mappingEnd];
                 if(starts.canFind(event.id))   {++level;}

source/dyaml/queue.d

@@ -36,8 +36,6 @@ private:
     Node* first_;
     // Last element of the linked list - last element added in time (start of the queue).
     Node* last_;
-    // Cursor pointing to the current node in iteration.
-    Node* cursor_;
     // free-list
     Node* stock;
 
@@ -119,30 +117,28 @@ public:
         freeStock();
     }
 
-    /// Start iterating over the queue.
+    /// Returns a forward range iterating over this queue.
-    void startIteration() @safe pure nothrow @nogc
+    auto range() @safe pure nothrow @nogc
     {
-        cursor_ = first_;
+        static struct Result
-    }
+        {
+            private Node* cursor;
 
-    /// Returns: The next element in the queue.
+            void popFront() @safe pure nothrow @nogc
-    ref const(T) next() @safe pure nothrow @nogc
+            {
-    in
+                cursor = cursor.next_;
-    {
+            }
-        assert(!empty);
+            ref T front() @safe pure nothrow @nogc
-        assert(cursor_ !is null);
+            in(cursor !is null)
-    }
+            {
-    do
+                return cursor.payload_;
-    {
+            }
-        const previous = cursor_;
+            bool empty() @safe pure nothrow @nogc const
-        cursor_ = cursor_.next_;
+            {
-        return previous.payload_;
+                return cursor is null;
-    }
+            }
-
+        }
-    /// Returns: true if itrating is not possible anymore, false otherwise.
+        return Result(first_);
-    bool iterationOver() @safe pure nothrow const @nogc
-    {
-        return cursor_ is null;
     }
 
     /// Push a new item to the queue.