<div dir="ltr"><div>Have a GitHub account, bearophile? Just hit Edit at the top of </div><div>that link you gave, make the change, and click "Proposal File </div><div>Change" and it'll roll up a pull request. Should take less than a </div>
<div>minute. I use this for almost every minor pull requests I do. The </div><div>autotester takes care of the rest.</div><div><br></div><div>Looks like line 9221 is essentially the same code so you may want </div><div>to change that line too.</div>
<div><br></div><div>BA</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Feb 5, 2013 at 5:33 PM, bearophile <span dir="ltr"><<a href="mailto:bearophileHUGS@lycos.com" target="_blank">bearophileHUGS@lycos.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This used to print true-false, now prints true-true. Is this a bug?<br>
<br>
<br>
import std.stdio, std.algorithm;<br>
void main() {<br>
auto x = "abcd";<br>
writeln(isSorted(x));<br>
auto y = "acbd";<br>
writeln(isSorted(y));<br>
}<br>
<br>
<br>
Maybe this line:<br>
<br>
<a href="https://github.com/D-Programming-Language/phobos/blob/master/std/algorithm.d#L9237" target="_blank">https://github.com/D-<u></u>Programming-Language/phobos/<u></u>blob/master/std/algorithm.d#<u></u>L9237</a><br>
<br>
for (; !ahead.empty; ahead.popFront(), ++i)<br>
<br>
Should be:<br>
<br>
for (; !ahead.empty; ahead.popFront(), r.popFront(), ++i)<br>
<br>
Bye,<br>
bearophile<br>
</blockquote></div><br></div>