Patch for MsgBase was [jsword-devel] MsgBase bug

Joe Walker jsword-devel@crosswire.org
Fri, 16 Apr 2004 00:00:30 +0100


DM Smith wrote:
> I have thought about whether having MsgBase be an Enum.
> I don't think that we use it as an Enum (ala apache) but it is very 
> valuable to have it be separate message objects because we can enforce 
> i18n at compile time.
> 
> So I think that we should use the "type safe enumeration pattern". That 
> is, take extends Enum off of MsgBase and leave the rest of the class the 
> same. Is this what you are thinking? Would that be light enough?

Yup. And it is fairly easy to un-Enum it too.
I've not added in a static List to gather together all i18n strings, but 
it would be fairly easy to do.

Joe.