[phobos] phobos commit, revision 1627

[David Simcha]

Ok, yeah, you're right.  Getting rid of the code to strip off the = solves
this.  Thanks.  I've checked in the fix, after testing that it builds
properly on the beta.  Everything should be fine now.

On Fri, Jun 11, 2010 at 4:17 PM, Don Clugston <dclugston at googlemail.com>wrote:

> On 11 June 2010 18:50, David Simcha <dsimcha at gmail.com> wrote:
> > Ok, I see why.  I had 2.046 and the beta installed concurrently, and I
> > forgot to point my path variable at the beta before building Phobos.  Now
> I
> > can reproduce this, both using the Phobos build scripts and compiling the
> > file individually with unittests turned on from Codeblocks.  It still
> works
> > fine on 2.046, though I did notice that DMD does take a while to
> > (successfully) compile it and uses ~60 MB of memory.  It looks like a
> > compiler regression, or else CustomFloat was relying on a bug that got
> > fixed.
>
> It seems to be a recursive mixin. Every time it's instantiated, it
> gets 4 bytes bigger.
> Seems to be triggered by opOpAssign; maybe it's the change to
> opOpAssign in this release: "+=" was passed to opOpAssign
> now it's "+". There's code in opOpAssign which is clearly wrong - it's
> stripping off the "=", which no longer exists.
>
> If you don't think you can fix this really quickly, I think we should
> remove it completely from this release, and wait until the next one.
> It doesn't seem urgent to me, but getting a release out is urgent.
> _______________________________________________
> phobos mailing list
> phobos at puremagic.com
> http://lists.puremagic.com/mailman/listinfo/phobos
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.puremagic.com/pipermail/phobos/attachments/20100611/603bdae7/attachment.html>