Patch was RE: [jsword-devel] ActionFactory and ResourceBundles
DM Smith
jsword-devel@crosswire.org
Mon, 05 Apr 2004 15:46:10 -0400
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/