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