[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