<div>On Tue, 17 Aug 2010 19:43:41 +0900, SHOO wrote:</div><div><br></div><div>> Here is the module to suggest this time:</div><div>> <a href="http://ideone.com/TVw1P" target="_blank">http://ideone.com/TVw1P</a></div>
<div><br></div>
<div>Very nicely coded. I think much better than the messing std.perf. Few </div><div>things...</div><div><br></div><div>1. I didn't found initialization of APPORIGIN. Is it meant to be </div><div>initialized in runtime, or else module constructor comes to my mind...</div>
<div><br></div><div>2. As a user of this module, I would much prefer to have an exception </div><div>throw when I call start or stop function out of order, instead of silent </div><div>return. If the only returns, there semantic requirement for having stop </div>
<div>function - it is has then same meaning as peek.</div><div><br></div><div>3. in std.perf the mili and microseconds use some special casing for </div><div>computation. I suppose it is to increase floating point accuracy. Does </div>
<div>this casing applies to your module and can it be used to increase </div><div>accuracy ?</div><div><br></div><div>and some much less important things...</div><div><br></div><div>4. I personally would prefer names like sec, usec, msec instead the </div>
<div>longer you use.</div><div><br></div><div>5. m_TimeObject is not an instance of any class. Better name could be </div><div>probably "m_TimeInit" "m_TimeInstanceInit"</div><div><br></div><div>6. Wouldn't be better to name the variable "m_FlagObserving" or </div>
<div>"m_FlagStarted" ?</div><div><br></div><div>7. ovserving is misspelled. </div><div><br></div>