> On May 6, 2016, at 11:43 AM, Sean Mullan <sean.mul...@oracle.com> 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.
Looks okay. PolicyFile::init catches ParsingException that always calls ParsingException::getLocalizedMessage and prints the localized message. Is that necessary? We don’t typically localize the exception message if thrown at runtime. Mandy
