I think this is a useful addition, but I wonder if these new APIs are misnamed: StopWatch::test doesn't actually test, nor does StopWatch::accept consume. Instead, these all wrap lambdas in a timed version of the same type. (There's probably some lambda calculus terminology for this.)
I'd propose renaming all of the functional interface-wrapping functions to use a single, overloaded name, e.g. StopWatch::timed(...) or StopWatch::withTiming(...) /peter On Fri, Aug 16, 2024, 10:50 PM Gary Gregory <garydgreg...@gmail.com> wrote: > Hi all, > > Please provide any feedback on > - LANG-1747: Measure the execution time of functional interfaces #1254 > - https://github.com/apache/commons-lang/pull/1254/ > TY! > Gary > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > >
