D-Programming-Language/phobos — pull request #855 by monarchdodra (2012-10-10 08:54:04+00:00)
Title: 3 fixes in range

I was re-viewing range for correct usage of save. This PR contains 3 things:

1. fixes the save behavior of "takeExactly"
2. Repeat

* Repeat did not originally save the passed-in range. This is (I think), by design, hence the doc note.

* Because of this, saving repeat should not duplicate the underlying buffer either, just duplicate the current iteration

3. FrontTransversal: Formalizing "RangeType" and "T" aliases

* Using "ElementType" as an alias is always a bad idea (forces ".ElementType!BLA", notice the dot).

* using the alias "RangeType" makes the conditionals cleaner.

2 or 3 indent changes.

I can splice the PR if anything seems fishy...