[rfc][icedtea-web] allow access to jars' attributes from jnlpfile class
Andrew Azores
aazores at redhat.com
Fri Nov 8 06:32:16 PST 2013
On 11/08/2013 07:24 AM, Jiri Vanek wrote:
> Above fixed, also littl ebut improved (some logic moved from
> pluginBridge to jnlpfile)
>
> As this should go to 1.4 , please double check before aprove.
>
>
> J.
Few nits again:
JNLPFile:
- instead of named String constants APP_NAME etc., maybe use an enum
here? If not, at least make them final, especially since they're public.
IMO enum is better though.
- #getTitle() - last return statement has a typo. "Unnown" should be
"Unknown"
- #getTitleFromManifest() - typo, "manifes" is missing its 't'. Maybe
"Check with application vendor" rather than "Check vendor of application" ?
ManifestAttributes:
- #getApplicationName() uses APP_NAME named constant - why not for the
other two similar calls? You just have a hardcoded string in those.
Thanks,
--
Andrew A
More information about the distro-pkg-dev
mailing list