StopWatch does not need to be Abstract

[Tibor17]

The StopWatch is abstract without any abstract method. I am forced to extend this class even if not necessary.
Can we remove the modifier?

[Tibor17]

@kcooney
@dsaff
@marcphilipp
@stefanbirkner
Any thoughts?

[kcooney]

I don't have any objections.

BTW, is there a reason why you needed a quick response? You created this issue less than a day ago

[marcphilipp]

Is there a usecase when it's not necessary to extend this class?

[Tibor17]

We can commit in 4.13, no hurry; but usully you are fast to response so it looked like you did forget this.
I created StopWatch in JUnit and now I am using it in my company and wondering about the modifier. I will replace the Stopwatch in surefire with this one but new StopWatch() {} isn't so nice in the code.

[Tibor17]

If measuring exec time, I don't need other notifications, only reading the runtime.

[kcooney]

@Tibor17 although we often respond to issues quickly, we all have day jobs :-) If we don't respond to issues within a few weeks, feel free to ping the issues.

[marcphilipp]

I forgot about runtime being public, never mind. No objections from me either.

[Tibor17]

The PR #1111 was created.