digiKam › digikam-devel

Re: extragear/graphics/digikam/imageplugins/noisereduction

classic Classic list List threaded Threaded
♦
♦
 2 messages Options
Bugzilla from andi.clemens@gmx.net
Reply | Threaded
Open this post in threaded view
♦
♦
|

Re: extragear/graphics/digikam/imageplugins/noisereduction

Bugzilla from andi.clemens@gmx.net
Hi,

this patch is actually redundant code. Shouldn't this be handled by the
EditorTool class?
It could simply disable the plainWidget when hitting "Try", slotEffect() or
slotFinal().

Andi

On Monday 18 May 2009 14:28:17 Andi Clemens wrote:

> SVN commit 969463 by aclemens:
>
> make DExpanderBox globally available
> Now disabling of the control widgets can be simplified by just disabling
> the m_expanderBox widget.
>
>  M  +8 -71     noisereductiontool.cpp
>  M  +3 -0      noisereductiontool.h
>
>
> ---
> trunk/extragear/graphics/digikam/imageplugins/noisereduction/noisereduction
>tool.cpp #969462:969463 @@ -264,7 +264,7 @@
>      grid2->setMargin(m_gboxSettings->spacingHint());
>      grid2->setSpacing(m_gboxSettings->spacingHint());
>
> -    DExpanderBox * m_expanderBox = new DExpanderBox;
> +    m_expanderBox = new DExpanderBox;
>      m_expanderBox->addItem(firstPage, SmallIcon("noisereduction"),
> i18n("Details"), QString("DetailsContainer"), true);
>      m_expanderBox->addItem(secondPage, SmallIcon("noisereduction"),
> i18n("Advanced settings"), @@ -290,16 +290,7 @@
>
>  void NoiseReductionTool::renderingFinished()
>  {
> -    m_radiusInput->setEnabled(true);
> -    m_lumToleranceInput->setEnabled(true);
> -    m_thresholdInput->setEnabled(true);
> -    m_textureInput->setEnabled(true);
> -    m_sharpnessInput->setEnabled(true);
> -    m_csmoothInput->setEnabled(true);
> -    m_lookaheadInput->setEnabled(true);
> -    m_gammaInput->setEnabled(true);
> -    m_dampingInput->setEnabled(true);
> -    m_phaseInput->setEnabled(true);
> +    m_expanderBox->setEnabled(true);
>  }
>
>  void NoiseReductionTool::readSettings()
> @@ -307,16 +298,7 @@
>      KSharedConfig::Ptr config = KGlobal::config();
>      KConfigGroup group        = config->group("noisereduction Tool");
>
> -    m_csmoothInput->setEnabled(false);
> -    m_dampingInput->setEnabled(false);
> -    m_gammaInput->setEnabled(false);
> -    m_lookaheadInput->setEnabled(false);
> -    m_lumToleranceInput->setEnabled(false);
> -    m_phaseInput->setEnabled(false);
> -    m_radiusInput->setEnabled(false);
> -    m_sharpnessInput->setEnabled(false);
> -    m_textureInput->setEnabled(false);
> -    m_thresholdInput->setEnabled(false);
> +    m_expanderBox->setEnabled(false);
>
>      m_csmoothInput->setValue(group.readEntry("CsmoothAdjustment",
> m_csmoothInput->defaultValue()));
> m_dampingInput->setValue(group.readEntry("DampingAdjustment",
> m_dampingInput->defaultValue())); @@ -329,16 +311,7 @@
>      m_textureInput->setValue(group.readEntry("TextureAdjustment",
> m_textureInput->defaultValue()));
> m_thresholdInput->setValue(group.readEntry("ThresholdAdjustment",
> m_thresholdInput->defaultValue()));
>
> -    m_csmoothInput->setEnabled(true);
> -    m_dampingInput->setEnabled(true);
> -    m_gammaInput->setEnabled(true);
> -    m_lookaheadInput->setEnabled(true);
> -    m_lumToleranceInput->setEnabled(true);
> -    m_phaseInput->setEnabled(true);
> -    m_radiusInput->setEnabled(true);
> -    m_sharpnessInput->setEnabled(true);
> -    m_textureInput->setEnabled(true);
> -    m_thresholdInput->setEnabled(true);
> +    m_expanderBox->setEnabled(true);
>  }
>
>  void NoiseReductionTool::writeSettings()
> @@ -361,16 +334,7 @@
>
>  void NoiseReductionTool::slotResetSettings()
>  {
> -    m_csmoothInput->setEnabled(false);
> -    m_dampingInput->setEnabled(false);
> -    m_gammaInput->setEnabled(false);
> -    m_lookaheadInput->setEnabled(false);
> -    m_lumToleranceInput->setEnabled(false);
> -    m_phaseInput->setEnabled(false);
> -    m_radiusInput->setEnabled(false);
> -    m_sharpnessInput->setEnabled(false);
> -    m_textureInput->setEnabled(false);
> -    m_thresholdInput->setEnabled(false);
> +    m_expanderBox->setEnabled(false);
>
>      m_csmoothInput->slotReset();
>      m_dampingInput->slotReset();
> @@ -383,30 +347,12 @@
>      m_textureInput->slotReset();
>      m_thresholdInput->slotReset();
>
> -    m_csmoothInput->setEnabled(true);
> -    m_dampingInput->setEnabled(true);
> -    m_gammaInput->setEnabled(true);
> -    m_lookaheadInput->setEnabled(true);
> -    m_lumToleranceInput->setEnabled(true);
> -    m_phaseInput->setEnabled(true);
> -    m_radiusInput->setEnabled(true);
> -    m_sharpnessInput->setEnabled(true);
> -    m_textureInput->setEnabled(true);
> -    m_thresholdInput->setEnabled(true);
> +    m_expanderBox->setEnabled(true);
>  }
>
>  void NoiseReductionTool::prepareEffect()
>  {
> -    m_radiusInput->setEnabled(false);
> -    m_lumToleranceInput->setEnabled(false);
> -    m_thresholdInput->setEnabled(false);
> -    m_textureInput->setEnabled(false);
> -    m_sharpnessInput->setEnabled(false);
> -    m_csmoothInput->setEnabled(false);
> -    m_lookaheadInput->setEnabled(false);
> -    m_gammaInput->setEnabled(false);
> -    m_dampingInput->setEnabled(false);
> -    m_phaseInput->setEnabled(false);
> +    m_expanderBox->setEnabled(false);
>
>      double r   = m_radiusInput->value();
>      double l   = m_lumToleranceInput->value();
> @@ -425,16 +371,7 @@
>
>  void NoiseReductionTool::prepareFinal()
>  {
> -    m_radiusInput->setEnabled(false);
> -    m_lumToleranceInput->setEnabled(false);
> -    m_thresholdInput->setEnabled(false);
> -    m_textureInput->setEnabled(false);
> -    m_sharpnessInput->setEnabled(false);
> -    m_csmoothInput->setEnabled(false);
> -    m_lookaheadInput->setEnabled(false);
> -    m_gammaInput->setEnabled(false);
> -    m_dampingInput->setEnabled(false);
> -    m_phaseInput->setEnabled(false);
> +    m_expanderBox->setEnabled(false);
>
>      double r  = m_radiusInput->value();
>      double l  = m_lumToleranceInput->value();
> ---
> trunk/extragear/graphics/digikam/imageplugins/noisereduction/noisereduction
>tool.h #969462:969463 @@ -35,6 +35,7 @@
>
>  namespace Digikam
>  {
> +class DExpanderBox;
>  class EditorToolSettings;
>  class ImagePanelWidget;
>  }
> @@ -84,6 +85,8 @@
>
>      Digikam::ImagePanelWidget    *m_previewWidget;
>
> +    Digikam::DExpanderBox        *m_expanderBox;
> +
>      Digikam::EditorToolSettings  *m_gboxSettings;
>  };

_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Gilles Caulier-4
Reply | Threaded
Open this post in threaded view
♦
♦
|

Re: extragear/graphics/digikam/imageplugins/noisereduction

Gilles Caulier-4
2009/5/18 Andi Clemens <[hidden email]>:
> Hi,
>
> this patch is actually redundant code. Shouldn't this be handled by the
> EditorTool class?

I just thinking the same (:=)))

Gilles


> It could simply disable the plainWidget when hitting "Try", slotEffect() or
> slotFinal().
>
> Andi
>
> On Monday 18 May 2009 14:28:17 Andi Clemens wrote:
>> SVN commit 969463 by aclemens:
>>
>> make DExpanderBox globally available
>> Now disabling of the control widgets can be simplified by just disabling
>> the m_expanderBox widget.
>>
>>  M  +8 -71     noisereductiontool.cpp
>>  M  +3 -0      noisereductiontool.h
>>
>>
>> ---
>> trunk/extragear/graphics/digikam/imageplugins/noisereduction/noisereduction
>>tool.cpp #969462:969463 @@ -264,7 +264,7 @@
>>      grid2->setMargin(m_gboxSettings->spacingHint());
>>      grid2->setSpacing(m_gboxSettings->spacingHint());
>>
>> -    DExpanderBox * m_expanderBox = new DExpanderBox;
>> +    m_expanderBox = new DExpanderBox;
>>      m_expanderBox->addItem(firstPage, SmallIcon("noisereduction"),
>> i18n("Details"), QString("DetailsContainer"), true);
>>      m_expanderBox->addItem(secondPage, SmallIcon("noisereduction"),
>> i18n("Advanced settings"), @@ -290,16 +290,7 @@
>>
>>  void NoiseReductionTool::renderingFinished()
>>  {
>> -    m_radiusInput->setEnabled(true);
>> -    m_lumToleranceInput->setEnabled(true);
>> -    m_thresholdInput->setEnabled(true);
>> -    m_textureInput->setEnabled(true);
>> -    m_sharpnessInput->setEnabled(true);
>> -    m_csmoothInput->setEnabled(true);
>> -    m_lookaheadInput->setEnabled(true);
>> -    m_gammaInput->setEnabled(true);
>> -    m_dampingInput->setEnabled(true);
>> -    m_phaseInput->setEnabled(true);
>> +    m_expanderBox->setEnabled(true);
>>  }
>>
>>  void NoiseReductionTool::readSettings()
>> @@ -307,16 +298,7 @@
>>      KSharedConfig::Ptr config = KGlobal::config();
>>      KConfigGroup group        = config->group("noisereduction Tool");
>>
>> -    m_csmoothInput->setEnabled(false);
>> -    m_dampingInput->setEnabled(false);
>> -    m_gammaInput->setEnabled(false);
>> -    m_lookaheadInput->setEnabled(false);
>> -    m_lumToleranceInput->setEnabled(false);
>> -    m_phaseInput->setEnabled(false);
>> -    m_radiusInput->setEnabled(false);
>> -    m_sharpnessInput->setEnabled(false);
>> -    m_textureInput->setEnabled(false);
>> -    m_thresholdInput->setEnabled(false);
>> +    m_expanderBox->setEnabled(false);
>>
>>      m_csmoothInput->setValue(group.readEntry("CsmoothAdjustment",
>> m_csmoothInput->defaultValue()));
>> m_dampingInput->setValue(group.readEntry("DampingAdjustment",
>> m_dampingInput->defaultValue())); @@ -329,16 +311,7 @@
>>      m_textureInput->setValue(group.readEntry("TextureAdjustment",
>> m_textureInput->defaultValue()));
>> m_thresholdInput->setValue(group.readEntry("ThresholdAdjustment",
>> m_thresholdInput->defaultValue()));
>>
>> -    m_csmoothInput->setEnabled(true);
>> -    m_dampingInput->setEnabled(true);
>> -    m_gammaInput->setEnabled(true);
>> -    m_lookaheadInput->setEnabled(true);
>> -    m_lumToleranceInput->setEnabled(true);
>> -    m_phaseInput->setEnabled(true);
>> -    m_radiusInput->setEnabled(true);
>> -    m_sharpnessInput->setEnabled(true);
>> -    m_textureInput->setEnabled(true);
>> -    m_thresholdInput->setEnabled(true);
>> +    m_expanderBox->setEnabled(true);
>>  }
>>
>>  void NoiseReductionTool::writeSettings()
>> @@ -361,16 +334,7 @@
>>
>>  void NoiseReductionTool::slotResetSettings()
>>  {
>> -    m_csmoothInput->setEnabled(false);
>> -    m_dampingInput->setEnabled(false);
>> -    m_gammaInput->setEnabled(false);
>> -    m_lookaheadInput->setEnabled(false);
>> -    m_lumToleranceInput->setEnabled(false);
>> -    m_phaseInput->setEnabled(false);
>> -    m_radiusInput->setEnabled(false);
>> -    m_sharpnessInput->setEnabled(false);
>> -    m_textureInput->setEnabled(false);
>> -    m_thresholdInput->setEnabled(false);
>> +    m_expanderBox->setEnabled(false);
>>
>>      m_csmoothInput->slotReset();
>>      m_dampingInput->slotReset();
>> @@ -383,30 +347,12 @@
>>      m_textureInput->slotReset();
>>      m_thresholdInput->slotReset();
>>
>> -    m_csmoothInput->setEnabled(true);
>> -    m_dampingInput->setEnabled(true);
>> -    m_gammaInput->setEnabled(true);
>> -    m_lookaheadInput->setEnabled(true);
>> -    m_lumToleranceInput->setEnabled(true);
>> -    m_phaseInput->setEnabled(true);
>> -    m_radiusInput->setEnabled(true);
>> -    m_sharpnessInput->setEnabled(true);
>> -    m_textureInput->setEnabled(true);
>> -    m_thresholdInput->setEnabled(true);
>> +    m_expanderBox->setEnabled(true);
>>  }
>>
>>  void NoiseReductionTool::prepareEffect()
>>  {
>> -    m_radiusInput->setEnabled(false);
>> -    m_lumToleranceInput->setEnabled(false);
>> -    m_thresholdInput->setEnabled(false);
>> -    m_textureInput->setEnabled(false);
>> -    m_sharpnessInput->setEnabled(false);
>> -    m_csmoothInput->setEnabled(false);
>> -    m_lookaheadInput->setEnabled(false);
>> -    m_gammaInput->setEnabled(false);
>> -    m_dampingInput->setEnabled(false);
>> -    m_phaseInput->setEnabled(false);
>> +    m_expanderBox->setEnabled(false);
>>
>>      double r   = m_radiusInput->value();
>>      double l   = m_lumToleranceInput->value();
>> @@ -425,16 +371,7 @@
>>
>>  void NoiseReductionTool::prepareFinal()
>>  {
>> -    m_radiusInput->setEnabled(false);
>> -    m_lumToleranceInput->setEnabled(false);
>> -    m_thresholdInput->setEnabled(false);
>> -    m_textureInput->setEnabled(false);
>> -    m_sharpnessInput->setEnabled(false);
>> -    m_csmoothInput->setEnabled(false);
>> -    m_lookaheadInput->setEnabled(false);
>> -    m_gammaInput->setEnabled(false);
>> -    m_dampingInput->setEnabled(false);
>> -    m_phaseInput->setEnabled(false);
>> +    m_expanderBox->setEnabled(false);
>>
>>      double r  = m_radiusInput->value();
>>      double l  = m_lumToleranceInput->value();
>> ---
>> trunk/extragear/graphics/digikam/imageplugins/noisereduction/noisereduction
>>tool.h #969462:969463 @@ -35,6 +35,7 @@
>>
>>  namespace Digikam
>>  {
>> +class DExpanderBox;
>>  class EditorToolSettings;
>>  class ImagePanelWidget;
>>  }
>> @@ -84,6 +85,8 @@
>>
>>      Digikam::ImagePanelWidget    *m_previewWidget;
>>
>> +    Digikam::DExpanderBox        *m_expanderBox;
>> +
>>      Digikam::EditorToolSettings  *m_gboxSettings;
>>  };
>
> _______________________________________________
> Digikam-devel mailing list
> [hidden email]
> https://mail.kde.org/mailman/listinfo/digikam-devel
>
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Free forum by Nabble Edit this page