RFR for JDK-8028711: TEST_BUG: Tests should pass through VM options: corelibs tests
Alan.Bateman at oracle.com
Wed Feb 12 18:35:17 UTC 2014
On 12/02/2014 12:59, michael cui wrote:
> On 02/10/2014 05:36 PM, Alan Bateman wrote:
>> A minor comment on test/java/net/URLPermission/nstest/lookup.sh is
>> that the resulting line length is 182 characters and this will likely
>> be annoying for future side-by-side views. So I think I'd split this
>> while you are there.
> Please review the updated version at
> Changes includes :
> 1. split line if it longer than 80 characters.
> 2. merge the fix of JDK-8033897
> 3. add few missed scripts.
> If no further changes need to be made, I would like to find sponsor to
> push this fix.
The changes look okay to me. I see you decided to ignore the javac
usages but that is okay and can be done another time.
Also thanks for fixing lookup.sh. I don't personally mind lines > 80 but
that one was >180 which make it difficult to look at side-by-side changes.
More information about the core-libs-dev