[RFC][Icedtea-web] Add cache viewer for control panel.
omajid at redhat.com
Fri Dec 17 08:09:57 PST 2010
On 12/15/2010 03:30 PM, Andrew Su wrote:
>> All these changes have now been added + fixed in this updated patch.
>> Added a cache viewer for the control panel.
>> (addComponents): Changed buttons to open cache viewer.
>> * netx/net/sourceforge/jnlp/resources/Messages.properties: Added text
>> used by the cache viewer
>> * netx/net/sourceforge/jnlp/cache/CacheDirectory.java,
>> netx/net/sourceforge/jnlp/controlpanel/CacheViewer.java: New classes.
>> Questions? Comments? Concerns?
Overall it looks fine to me. I only have two concerns:
Please fix the license in files that you are adding to
net.sourceforge.jnlp.cache.*. It should be LGPL, not GPL.
Please avoid adding in extra calls to UIManager.setLookAndFeel().
Calling it multiple times doesnt make it any more system-like :) This
call should be made just once - preferably in ControLPanel's main method
- before any GUI components have been created.
Other than those issues above, the patch looks fine to me. It would be
nice to get rid of extra whitespace characters after the end of lines
too, but I dont think that's a big deal (ctrl-shift+f in eclipse should
More information about the distro-pkg-dev