[rfc][icedtea-web] Trusted-only manifest attribute

Omair Majid omajid at redhat.com
Fri Mar 21 14:56:39 UTC 2014


* Andrew Azores <aazores at redhat.com> [2014-03-20 11:35]:
> +++ b/netx/net/sourceforge/jnlp/runtime/ManifestsAttributesValidator.java

> +    void checkTrustedOnlyAttribute() throws LaunchException {

A stylistic nit: if you ask me to use a class named *Validator, I will
look for the validate* method as the first thing. If the class is named
*Checker, then maybe a check* method makes more sense.

What do you think?

Thanks,
Omair

-- 
PGP Key: 66484681 (http://pgp.mit.edu/)
Fingerprint = F072 555B 0A17 3957 4E95  0056 F286 F14F 6648 4681


More information about the distro-pkg-dev mailing list