Looks fine to me except a minor comment. PolicyParser.java ================= 1366 return i18nMessage != null ? i18nMessage : form.format(source);
May be nice to set the i18nMessage if it is not set yet. Thanks, Xuelei On 5/7/2016 2:43 AM, Sean Mullan wrote: > Please review this fix for > https://bugs.openjdk.java.net/browse/JDK-8150468: > > http://cr.openjdk.java.net/~mullan/webrevs/8150468/webrev.00/ > > The fix is to record bad policy files as they are parsed and ignore them > during any subsequent permission checks. > > Thanks, > Sean
