Realistically, how would you recommend testing something like this? I checked to make sure that there was something instantiating each version of the structs to make sure that they compile (i.e. no silly syntax errors), and as for the logical correctness, it seemed like the code was so trivial that it would be silly to write an explicit test for it. Furthermore, the precedent in the rest of Phobos seems to be that such trivial code does not require testing.<br>
<br><div class="gmail_quote">On Tue, Jun 22, 2010 at 10:30 AM, Brad Roberts <span dir="ltr"><<a href="mailto:braddr@puremagic.com">braddr@puremagic.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">On 6/22/2010 5:31 AM, <a href="http://dsource.org" target="_blank">dsource.org</a> wrote:<br>
> phobos commit, revision 1678<br>
><br>
><br>
> user: dsimcha<br>
><br>
> msg:<br>
> Bug 4362: std.range.repeat and cycle do not have a .save() method<br>
><br>
> <a href="http://www.dsource.org/projects/phobos/changeset/1678" target="_blank">http://www.dsource.org/projects/phobos/changeset/1678</a><br>
<br>
</div>Don't forget to add unit tests. :)<br>
<div><div></div><div class="h5">_______________________________________________<br>
phobos mailing list<br>
<a href="mailto:phobos@puremagic.com">phobos@puremagic.com</a><br>
<a href="http://lists.puremagic.com/mailman/listinfo/phobos" target="_blank">http://lists.puremagic.com/mailman/listinfo/phobos</a><br>
</div></div></blockquote></div><br>