The new std.process is ready for review

Vladimir Panteleev vladimir at thecybershadow.net
Sun Feb 24 15:47:36 PST 2013


On Sunday, 24 February 2013 at 21:40:24 UTC, Andrej Mitrovic 
wrote:
> On 2/24/13, Vladimir Panteleev <vladimir at thecybershadow.net> 
> wrote:
>> a) they come with a very thorough unit test
>
> Unfortunately those unittests are never being run in the Phobos
> test-suite and as a result a regression was missed (which was 
> fixed by
> now):
> http://d.puremagic.com/issues/show_bug.cgi?id=9309
>
> The report for the unittests:
> http://d.puremagic.com/issues/show_bug.cgi?id=9310

Hmm, that's unfortunate. Some of that function's callees are 
tested, but not the function itself.

The reason why unittest_burnin is not set in Phobos is that 1) it 
requires a helper program to be compiled beforehand, and 2) it 
runs indefinitely with random variations (hence its name).

I suppose a simpler additional unit test would be appropriate.
https://github.com/D-Programming-Language/phobos/pull/1161


More information about the Digitalmars-d mailing list