[phobos] phobos commit, revision 2032

Andrei Alexandrescu andrei at erdani.com
Mon Sep 20 20:21:39 PDT 2010


I don't think there was reliance on the bug. It was more like 
accept-invalid kind of stuff - std.algorithm wasn't supposed to work on 
const arrays, but it sometimes did.

Andrei

On 09/20/2010 07:19 PM, David Simcha wrote:
> I agree for the part of the diff that was related to working around
> 3534, but most of the diff was to avoid relying on 3534, and to fix
> stuff that would break when 3534 was fixed. The changes in std.array
> work around 3534 by explicitly preventing popFront() and popBack() from
> being used on const/immutable arrays. The rest of the diffs fix issues
> caused by relying on popFront()/popBack() working on const/immutable
> arrays when they shouldn't.
>
> The reason why I committed this changeset was because I realized that
> there was wholesale reliance on this bug w.r.t. ranges, so I wanted to
> break this before even more code came to depend on it. I was concerned
> that if we waited and let even more code depend on this bug, it would
> become nearly impossible to fix because so much code would be relying on
> it.
>
> On 9/19/2010 6:58 PM, Andrei Alexandrescu wrote:
>> One important advantage we have in working on Phobos is that we have
>> Walter on board, who can help us. I think we should convince Walter of
>> giving priority to byg 3534 and then undo this diff.
>>
>> Andrei
>>
>> On 09/18/2010 04:00 PM, dsource.org wrote:
>>> phobos commit, revision 2032
>>>
>>>
>>> user: dsimcha
>>>
>>> msg:
>>> Bug 4888: Heavy reliance on Bug 3534 in Phobos range usage. I used a
>>> different approach here than the first one I tried. I only did what
>>> was necessary to make Phobos work instead of trying to fix the deeper
>>> issue of making std.algorithm to work w/ const/immutable arrays.
>>>
>>> http://www.dsource.org/projects/phobos/changeset/2032
>>>
>>> _______________________________________________
>>> phobos mailing list
>>> phobos at puremagic.com
>>> http://lists.puremagic.com/mailman/listinfo/phobos
>> _______________________________________________
>> phobos mailing list
>> phobos at puremagic.com
>> http://lists.puremagic.com/mailman/listinfo/phobos
>>
>
> _______________________________________________
> phobos mailing list
> phobos at puremagic.com
> http://lists.puremagic.com/mailman/listinfo/phobos


More information about the phobos mailing list