[Kst] kdeextragear-2/kst/kst

[George Staikos]

On Wednesday 15 September 2004 12:59, Andrew Walker wrote:
> +    QString strRange;
>
> -    // FIXME: i18n completely broken here for RTL.
> -    str = i18n("Event Monitor: %1: ").arg( str );
>      for( i=0; i<iSize; i++ ) {
>        iIndex = *(_indexArray.at(i));
>        if( i == 0 ) {
> -        str += i18n("%1").arg( iIndex );
> +        strRange += i18n("%1").arg( iIndex );

   This isn't fixed.  If you use += with i18n() you almost certainly are 
breaking support for RTL languages.  Also i18n("%1") is a no-op generally, 
and definitely in this case.  It just adds an extra step for the translators.

-- 
George Staikos
KDE Developer				http://www.kde.org/
Staikos Computing Services Inc.		http://www.staikos.net/