[phobos] phobos commit, revision 1919

Andrei Alexandrescu andrei at erdani.com
Fri Aug 27 10:25:27 PDT 2010


I agree that AutoStart should be a module-level enum.

Andrei

On 8/27/10 10:18 PDT, SHOO wrote:
> (2010/08/27 22:21), David Simcha wrote:
>> Here's one small cleanup request I noticed when actually using
>> StopWatch. The following is absurdly verbose and makes me feel like I'm
>> programming in Java:
>>
>> auto sw = new StopWatch(StopWatch.AutoStart.yes);
>>
>> Could we move the AutoStart enum into the module-level namespace instead
>> of the StopWatch class namespace, and maybe give it a default value of
>> yes, just to make instantiating StopWatch less verbose?
>>
>> On 8/22/2010 11:11 AM, dsource.org wrote:
>>> phobos commit, revision 1919
>>>
>>>
>>> user: SHOO
>>>
>>> msg:
>>> Added stopwatch module
>>>
>>> Added StopWatch, Ticks, systime, apptime, comparingBenchmark,
>>> measureTime.
>>>
>>> http://www.dsource.org/projects/phobos/changeset/1919
>>>
>>> _______________________________________________
>>> phobos mailing list
>>> phobos at puremagic.com
>>> http://lists.puremagic.com/mailman/listinfo/phobos
>>>
>
> You can use:
> auto sw = new StopWatch(autoStart);
>
> AutoStart.yes is redundant expression.
> And, it is impossible to make default behavior of the StopWatch
> AutoStart.yes.
>
> This comes from that StopWatch is a struct.
> struct cannot run default behavior.
> _______________________________________________
> phobos mailing list
> phobos at puremagic.com
> http://lists.puremagic.com/mailman/listinfo/phobos


More information about the phobos mailing list