Thank you all. I just pushed.

I set David as contributor since he provided the original patch.

Cheers, Thomas
On Thu, Nov 1, 2018 at 9:11 AM Alan Bateman <alan.bate...@oracle.com> wrote:
>
> On 31/10/2018 13:45, Roger Riggs wrote:
> > Hi Thomas,
> >
> > The webrev looks fine.
> >
> > Please remove the @author tag in the Linux (2nd) test block in Basic.java.
> > Author tags are losing favor and there's no need to repeat it.
> I agree, no need to repeat it here.
>
> The webrev otherwise looks good to me too and I see Roger's issue to
> looking at switching the default.
>
> -Alan

Reply via email to