[amarok] src/dynamic: fix more i18n Krazy issues

Matěj Laitl matej at laitl.cz
Tue Apr 2 14:08:27 UTC 2013


On 2. 4. 2013 Myriam Schweingruber wrote:
> Git commit 601be3102ccb8ca78e3629f9d1b7696c2a535fa4 by Myriam Schweingruber.
> Committed on 02/04/2013 at 15:45.
> Pushed by schweingruber into branch 'master'.
> 
> fix more i18n Krazy issues
> diff --git a/src/dynamic/BiasFactory.cpp b/src/dynamic/BiasFactory.cpp
> index d0cdb0d..77d8d76 100644
> --- a/src/dynamic/BiasFactory.cpp
> +++ b/src/dynamic/BiasFactory.cpp
>  QWidget*
>  Dynamic::ReplacementBias::widget( QWidget* parent )
>  {
> -    QLabel *label = new QLabel( i18n( "Replacement for bias %1" ).arg(
> m_name ), parent );
> +    QLabel *label = new QLabel( i18n( "Replacement for
> bias %1", m_name, parent ) );

This is incorrect. parent should be an argument of QLabel, not i18n(). This 
change thus introduces a memory leak. Please fix this in a follow-up commit.

	Matěj


More information about the Amarok-devel mailing list