Patch was RE: [jsword-devel] ActionFactory and ResourceBundles

[DM Smith]

Line 307 should be removed. My goal was to eliminate private members where 
they were not needed outside of creating the layout. At this point and time 
there is no need for it to be a member var, but there is unused code that 
would show or hide the tool bar.

>From: Joe Walker <joe@eireneh.com>
>Reply-To: jsword-devel@crosswire.org
>To: jsword-devel@crosswire.org
>Subject: Re: Patch was RE: [jsword-devel] ActionFactory and ResourceBundles
>Date: Mon, 05 Apr 2004 20:30:27 +0100
>
>DM Smith wrote:
>>I am resending the patch that I sent before with a few changes. I have 
>>added actions for menus, fixed a bug in the CWClassLoader, which I found 
>>while testing under jnlp (and is not present otherwise).
>...
>
>I forgot, Desktop.java line 307 (or at least it is in mine!)
>         JToolBar pnlTbar = new JToolBar();
>
>Is there a reason for this? It just hides the following, line 861:
>     private JToolBar pnlTbar;
>
>For the moment I've commented it 307 out and it seems to work fine.
>
>?
>
>Joe.
>
>
>_______________________________________________
>jsword-devel mailing list
>jsword-devel@crosswire.org
>http://www.crosswire.org/mailman/listinfo/jsword-devel

_________________________________________________________________
Get rid of annoying pop-up ads with the new MSN Toolbar – FREE! 
http://toolbar.msn.com/go/onm00200414ave/direct/01/