On Tue, 24 May 2022 04:50:58 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
>> Christian Stein has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Update src/java.base/share/classes/java/util/spi/ToolProvider.java >> >> Co-authored-by: Anthony Vanelverdinghe <d...@anthonyv.be> > > src/java.base/share/classes/java/util/spi/ToolProvider.java line 59: > >> 57: * a tool may be the target of multiple {@code run} method invocations, >> 58: * and reentrant means that multiple invocations of {@code run} may occur >> 59: * concurrently. > > Hello @sormuras, > >> reentrant means that multiple invocations of {@code run} may occur >> concurrently. > > My understanding of re-entrant was that the same method could be re-invoked > on the same thread while the current method execution is in progress (a > recursion). Whereas "multiple invocations may occur concurrently" sounds more > like multiple threads invoking this concurrently (i.e. thread-safety). Which > of these 2 characteristics are we recommending here? According to https://wikidiff.com/reentrant/reusable "reentrant" includes/spans both characteristics: > (programming) That may be executed more than once at a time either by > different threads, or because of recursion. ------------- PR: https://git.openjdk.java.net/jdk/pull/8833