Looks good to me. Thanks David!

Andrei

David Simcha wrote:
Phobos devs,

Here's a fix for std.stdio.readlnImpl (bug 3763). <http://d.puremagic.com/issues/show_bug.cgi?id=3763> I ended up only fixing the immediate problem, rather than making any serious attempt to improve the readability of the code, or deal with the stomping issue discussed earlier, since much of the code seems to involve details of UTF (such as hard-coded magic numbers) that I'm afraid to mess with.

If there are no objections, I'll commit this fix, since it at least doesn't overflow the stack or reallocate a ridiculous amount of times.

--Dave


------------------------------------------------------------------------

_______________________________________________
phobos mailing list
[email protected]
http://lists.puremagic.com/mailman/listinfo/phobos
_______________________________________________
phobos mailing list
[email protected]
http://lists.puremagic.com/mailman/listinfo/phobos

Reply via email to