Re: RFR: 8287834: Add SymbolLookup::or method [v2]

2023-05-16 Thread Maurizio Cimadamore
On Tue, 16 May 2023 16:55:21 GMT, Paul Sandoz wrote: >> I was trying to split the sentence so that we could get a compact javadoc >> header for the factory. How much important is that? > > Not very IHMO. I was just trying to say the same thing you said with less > words, which makes it more ame

Re: RFR: 8287834: Add SymbolLookup::or method [v2]

2023-05-16 Thread Maurizio Cimadamore
> This patch adds a simpler method for composing symbol lookups. It is common > for clients to chain multiple symbol lookups together, e.g. to find a symbol > in multiple libraries. > > A new instance method, namely `SymbolLookup::or` is added, which first > searches a symbol in the first looku

Re: RFR: 8287834: Add SymbolLookup::or method [v2]

2023-05-16 Thread Paul Sandoz
On Tue, 16 May 2023 17:06:02 GMT, Maurizio Cimadamore wrote: >> This patch adds a simpler method for composing symbol lookups. It is common >> for clients to chain multiple symbol lookups together, e.g. to find a symbol >> in multiple libraries. >> >> A new instance method, namely `SymbolLook