On Fri, 12 Nov 2021 11:22:19 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
> > > P.S: Slightly unrelated question - is it intentional that the `Builder` > > > interface specifies the visibility modifiers on the interface methods. > > > For example: `public abstract Builder timeout(Duration duration);`, > > > `public Builder headers(String... headers);` and so on? > > > > > > Stylistic preference of the author - I guess (could be me). If this were a > > new class I would advise to remove the redundant qualifiers. > > Understood. I have the habit of using the `public` too sometimes. It was the > presence of `abstract` in one of the methods which made me a bit curious. Oh - maybe it went back and forth from being an abstract class or an interface during prototyping... ------------- PR: https://git.openjdk.java.net/jdk/pull/6348