On Thu, Sep 12, 2019 at 1:11 PM Thomas Vandahl <t...@apache.org> wrote:
> On 11.09.19 15:53, Gary Gregory wrote:
> > public static final ConfigurationConsumer<ConfigurationException>
> > DEFAULT_INCLUDE_LISTENER = e -> { throw e; };
> >
> > public static final ConfigurationConsumer<ConfigurationException>
> > NOOP_INCLUDE_LISTENER = e -> { /* noop */ };
>
> +1 for these.
>
> > I propose a reformatting to use the "{ on the same line" which means that
> > blocks go from:
> >
> > if (test)
> > {
> > // this
> > }
> > else
> > {
> > // that
> > }
> >
> > to:
> >
> > if (test) {
> > // this
> > } else {
> > // that
> > }
>
> -1 here. At least not for all components.
>
I was only talking about Configuration here.
Gary
>
> Bye, Thomas
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>
>
