[dmd-internals] [D-Programming-Language/dmd] 48ed15: Remove import of std.random:rand from test22

Brad Roberts braddr at puremagic.com
Tue Aug 16 11:08:02 PDT 2011


On Tuesday, August 16, 2011 4:41:30 AM, Don Clugston wrote:
> I've added it to bugzilla.
> http://d.puremagic.com/issues/show_bug.cgi?id=6505
> 
> On 16 August 2011 04:20, Jonathan M Davis <jmdavisProg at gmx.com> wrote:
>> On Tuesday, August 16, 2011 04:16:10 Don Clugston wrote:
>>> And now it fails on all platforms with -O. Yikes.  It seems that the
>>> original bug was never actually fixed. It's an optimizer bug, too.
>>> We've been running with a defective test case for years.
>>
>> Well, all the better that someone who actually has some clue what the tests
>> are for is fixing them rather than us using my attempt at fixing them.
> 
> It's funny -- I had thought I was wasting my time in tracking it down.

This thread is also really good indicators why having a pointer back to 
the original conversation / report / etc is valuable.  Being able to go 
back and re-examine what the original report was is rarely needed, but 
when it is, it's pretty useful.  The trend to name the t
est function 
after the bugzilla number is a good one, imho.

I did it a handful of times when converting the original private test 
suite into the public one.  There were a few tests that were large 
enough that I was concerned about the privacy issues.  Being able to 
find that they'd been gathered through public postings to a news group 
or bugzilla was enough to allow them to be made public.

Good stuff..
Brad


More information about the dmd-internals mailing list