Performance measurement with
had its value at some point in the past, but today it simply is no match against CPAN:Devel::NYTProf
I suggested to remove our own monitor in TWiki:Codev/EliminateMonitorDotPm
and such was agreed at TWiki:Codev/KampalaReleaseMeeting2018x02x01
- The speedup of "normal" TWiki operation by eliminating the calls to Monitor is tiny but visible in Devel::NYTProf (about 0.1%).
- If you wonder about the strange name "NYTProf": The work on the initial fork of this profiler from Devel::FastProf has been sponsored by the New York Times, and the name stuck since then.
The two checkins are for the core components and for the default plugins. Within SVN, there are a few non-default extensions which were once prepared to use
, but in all cases it is commented out. So, no action is required for these extensions (and IMO the deletion of comments doesn't justify a new version). We should be done right now.
These are the extensions with inactive uses of Monitor:
...needed another commit, I forgot two core files.
... and now Monitor.pm has to go, finally