[phobos] readlnImpl

Andrei Alexandrescu andrei at erdani.com
Sun Feb 14 21:44:31 PST 2010


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
> phobos at puremagic.com
> http://lists.puremagic.com/mailman/listinfo/phobos


More information about the phobos mailing list