https://bugzilla.novell.com/show_bug.cgi?id=229368 sh@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED ------- Comment #2 from sh@novell.com 2007-01-17 11:05 MST ------- Well, it's a very valid bug report. That code probably only works because the compiler silently merges identical string literals. IIRC that code was a hot fix done in a hurry: gettext() returns the .POT file header (which translator, translation date, etc.) if called with an empty string -- not what people would expect on the GUI. I should have done the correct checking to begin with. And while I am at it, a check against NULL pointers might be worth while, too. --- include/YQi18n.h (Revision 35285) +++ include/YQi18n.h (Arbeitskopie) @@ -27,7 +27,7 @@ inline QString _( const char * msgid ) { - return msgid == "" ? "" : QString::fromUtf8( gettext(msgid ) ); + return ( !msgid || !*msgid ) ? "" : QString::fromUtf8( gettext(msgid ) ); } Will be available with yast2-qt-2.13.78 (SLES-10 SP1), will be merged back to the 10.3 code base after the SLES-10 SP1 beta phase. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.