ad "protected non-final fields": I don't like it either, including this inconsistency, but this was part of the original forked code already. And in this concern I also wrote in the conversation/description of this issue:

> (3) to allow access to all fields of Timeout (i.e. fTimeout, fTimeUnit and fLookForStuckThread) either via changing its visibility from private to e.g. protected, or via a getter (with e.g. protected visibility) => for now with changed visibility for the sake of less change, what do you prefer?

So maybe the original code should be refactored first?