Description: ------------ Hi, I'm running Benchmark 1.2.2. When instantiating without automatic profiling : $this->_profiler =& new Benchmark_Profiler(); I get some output even If do not call display(). This comes from Benchmark/Profiler.php, line 161, in the destructor : function _Benchmark_Profiler() { if (isset($this->auto)) { $this->stop(); $this->display(); } } The isset() test evaluates to true even if automatic profiling is disabled, since $this->auto is always set, either to true or false. This issue gets fixed by replacing the isset() test with : if ($this->auto)

I just realized that this bug has another effect : In manual mode, the Profiler outputs : "reached end of section Global but expecting end of" While everything is set up correctly : I call stop() and display the data by myself. The fact that the destructor calls stop() and display() again seems to trigger this error. Fixing the isset() call, as previously described, fixes this junk output as well.

Yes, probably (isset($this->auto)) should be changed to (isset($this->auto) && $this->auto).

This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better.

Matthias, will there be new release of Benchmark any time soon? Living with manually-fixed pear-package makes me nervous :-/

What do you mean with "manually-fixed pear-package" ? Do you have some other features you need to patch for your uses ? Indeed, I will soon make a bug-fix release. Thank you for your feedback.