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

Reply via email to