Fabian Groffen wrote:
> On 16-05-2009 18:26:25 +0000, Martin Kersten wrote:
>> Update of /cvsroot/monetdb/MonetDB5/src/mal
>> In directory 23jxhf1.ch3.sourceforge.com:/tmp/cvs-serv27406
>>
>> Modified Files:
>>      mal_authorize.mx mal_client.mx mal_debugger.mx 
>>      mal_exception.mx mal_factory.mx mal_interpreter.mx 
>>      mal_linker.mx mal_profiler.mx mal_sabaoth.mx mal_scenario.mx 
>>      mal_session.mx 
>> Log Message:
>> Collecting the exception/errors messages in one place leads to easier 
>> detection
>> of duplicates and can be used later for internationalization efforts
> 
> Why not do it the gettext way, in that case?
Yes, i am fully aware of gettext.
It is a multi-step process, first condense.....
and gettext() performs translations based on strings
e.g. the ones now identified.
> 
>> U mal_linker.mx
>> Index: mal_linker.mx
>> ===================================================================
>> RCS file: /cvsroot/monetdb/MonetDB5/src/mal/mal_linker.mx,v
>> retrieving revision 1.110
>> retrieving revision 1.111
>> diff -u -d -r1.110 -r1.111
>> --- mal_linker.mx    7 Jan 2009 14:15:12 -0000       1.110
>> +++ mal_linker.mx    16 May 2009 18:26:23 -0000      1.111
>> @@ -251,7 +251,7 @@
>>  
>>      if (fullname == NULL) {
>>              if( flag)
>> -                    throw(LOADER, "loadLibrary", "%s: file not found", 
>> filename);
>> +                    throw(LOADER, "loadLibrary", RUNTIME_FILE_NOT_FOUND, 
>> filename);
> 
> I think this is prone to errors and mistakes, since you no longer know
> how many %X are in the constant, of what type, etc.
true... see how many are really needed after condensation

> 
> I do think consistency is good, I doubt about translations, but I think
> this approach is a good solution.  Please look into gettext, see
> applications like mutt for an example how to use it.
> 
> ------------------------------------------------------------------------------
> Crystal Reports - New Free Runtime and 30 Day Trial
> Check out the new simplified licensing option that enables 
> unlimited royalty-free distribution of the report engine 
> for externally facing server and web deployment. 
> http://p.sf.net/sfu/businessobjects
> _______________________________________________
> Monetdb-checkins mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/monetdb-checkins


------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables 
unlimited royalty-free distribution of the report engine 
for externally facing server and web deployment. 
http://p.sf.net/sfu/businessobjects
_______________________________________________
Monetdb-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/monetdb-developers

Reply via email to