On Fri, 2002-03-01 at 13:40, Petter Reinholdtsen wrote:
> - Avoid code duplication in main.c, and standardize how the messages
> catalog file is loaded. Allow the language code string to be a colon
> separated list.
> - Modify boxChooseLanguageVariant() accept a list of language codes
> - Change release_notes.c to use the same code path with and without
> the language chooser. Make it accept a list of language codes.
> - In util.c, allow a language list when choosing download host.
> - in norwegian.src, demonstrate how this new list should be used for
> Norwegian bokm�l and nynorsk.
In principle this is fine, but the changes to main.c and release_notes.c
are sufficiently inscrutable at a casual reading to make me a bit
nervous.
Would you mind sending a patch that contains only the changes necessary
for multiple language code handling, without any reorganisation of the
rest of the code? I think that would make it easier for people to
review.
+ if ( LOAD_TRMFILE("/etc/messages.en") != 0 ||
+ LOAD_TRMFILE("/etc/messages.trm") != 0 ||
+ LOAD_TRMFILE(TRMBACKUP) != 0 )
That doesn't look quite right to me.
p.
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
