[jsword-devel] updates
DM Smith
jsword-devel@crosswire.org
Sun, 18 Apr 2004 14:38:04 -0400
Joe,
The assert stuff looks solid! I had to make a trivial change to get the
compiler to accept it. Did you set it to build 1.2, 1.3 or 1.4 compatible
classes? I have set mine for 1.4. I want to make sure that we are debugging
the same built code. I guess that we did not need to require 1.4 as a
runtime but just as a compile time.
The change to MsgBase to remove dependency on Enum looks good.
So does the umpteen thousand other changes you made to externalize strings.
I am working on the externalizing of Strings in .../view/swing/book,
starting with DisplaySelectPane.
Mostly modeling it after what I did in .../view/swing/desktop.
DM
>From: Joe Walker <joe@eireneh.com>
>Reply-To: jsword-devel@crosswire.org
>To: jsword-devel@crosswire.org
>Subject: [jsword-devel] updates
>Date: Sun, 11 Apr 2004 12:59:24 +0100
>
>
>Hi,
>
>I'm down to 87 i18n strings to go. These however are the gnarly ones. These
>are mostly ones with mnemonics, so just blindly putting them into Msg won't
>work. The options that I can see are:
>- take off the mnemonics
>- move the mnemonics into the Msg file as well
>- turn these instances into Actions
>
>Any opinions?
>
>I've also removed LogicError and replaced with assert everywhere. I'd like
>to know if I've done it wrong anywhere. I've tried to make it so that
>generally speaking if asserts are turned off (the default) then something
>sensible happens.
>
>I've also edited the launchers to add a -ea to the command line to enable
>assertions. My understanding is that this only enables assertions in
>non-system classes. Does anyone know how to enable them for system classes
>too?
>
>We also need to fix the nightly build - I need to add some assertion
>switches, however the urgency for this is behind the urgency to fix the
>bizarre startup error that I'm getting at the moment!
>
>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/