Locallab tools code cleanup and format

This commit is contained in:
Pandagrapher 2020-04-23 21:09:29 +02:00
parent de2f66ae76
commit 32f66da534
3 changed files with 12 additions and 29 deletions

View File

@ -3167,7 +3167,7 @@ LocallabShadow::LocallabShadow():
// Shadow highlight specific widgets // Shadow highlight specific widgets
shMethod(Gtk::manage(new MyComboBoxText())), shMethod(Gtk::manage(new MyComboBoxText())),
multipliersh([this]() -> std::array<Adjuster *, 5> multipliersh([]() -> std::array<Adjuster *, 5>
{ {
std::array<Adjuster*, 5> res = {}; std::array<Adjuster*, 5> res = {};

View File

@ -1009,7 +1009,6 @@ private:
void updateContrastGUI1(); void updateContrastGUI1();
void updateContrastGUI2(); void updateContrastGUI2();
void updateContrastGUI3(); void updateContrastGUI3();
void updateContrastGUI4();
}; };
/* ==== LocallabCBDL ==== */ /* ==== LocallabCBDL ==== */

View File

@ -2172,6 +2172,7 @@ LocallabContrast::LocallabContrast():
wavedgConn = wavedg->signal_toggled().connect(sigc::mem_fun(*this, &LocallabContrast::wavedgChanged)); wavedgConn = wavedg->signal_toggled().connect(sigc::mem_fun(*this, &LocallabContrast::wavedgChanged));
strengthw->setAdjusterListener(this); strengthw->setAdjusterListener(this);
sigmaed->setAdjusterListener(this); sigmaed->setAdjusterListener(this);
LocalcurveEditorwavedg->setCurveListener(this); LocalcurveEditorwavedg->setCurveListener(this);
@ -2214,6 +2215,7 @@ LocallabContrast::LocallabContrast():
wavblurConn = wavblur->signal_toggled().connect(sigc::mem_fun(*this, &LocallabContrast::wavblurChanged)); wavblurConn = wavblur->signal_toggled().connect(sigc::mem_fun(*this, &LocallabContrast::wavblurChanged));
levelblur->setAdjusterListener(this); levelblur->setAdjusterListener(this);
sigmabl->setAdjusterListener(this); sigmabl->setAdjusterListener(this);
chromablu->setAdjusterListener(this); chromablu->setAdjusterListener(this);
@ -2270,6 +2272,7 @@ LocallabContrast::LocallabContrast():
LocalcurveEditorwavcompre->curveListComplete(); LocalcurveEditorwavcompre->curveListComplete();
sigmadr->setAdjusterListener(this); sigmadr->setAdjusterListener(this);
threswav->setAdjusterListener(this); threswav->setAdjusterListener(this);
residcomp->setAdjusterListener(this); residcomp->setAdjusterListener(this);
@ -2477,7 +2480,7 @@ LocallabContrast::LocallabContrast():
compBox->pack_start(*separatorcomp); compBox->pack_start(*separatorcomp);
compBox->pack_start(*fatres); compBox->pack_start(*fatres);
compFrame->add(*compBox); compFrame->add(*compBox);
// blurcontBox2->pack_start(*compFrame); // blurcontBox2->pack_start(*compFrame);
expcontrastpyr2->add(*blurcontBox2, false); expcontrastpyr2->add(*blurcontBox2, false);
pack_start(*expcontrastpyr2); pack_start(*expcontrastpyr2);
@ -2688,14 +2691,11 @@ void LocallabContrast::read(const rtengine::procparams::ProcParams* pp, const Pa
// Update Local contrast GUI according to localcontMethod combobox value // Update Local contrast GUI according to localcontMethod combobox value
updateContrastGUI1(); updateContrastGUI1();
// Update Local contrast GUI according to wavedg button state // Update Local contrast GUI according to waveshow button state
updateContrastGUI2(); updateContrastGUI2();
// Update Local contrast GUI according to waveshow button state
updateContrastGUI3();
// Update Local contrast GUI according to fftwlc button state // Update Local contrast GUI according to fftwlc button state
updateContrastGUI4(); updateContrastGUI3();
// Note: No need to manage pedited as batch mode is deactivated for Locallab // Note: No need to manage pedited as batch mode is deactivated for Locallab
} }
@ -3282,9 +3282,6 @@ void LocallabContrast::wavgradlChanged()
void LocallabContrast::wavedgChanged() void LocallabContrast::wavedgChanged()
{ {
// Update Local contrast GUI according to wavedg button state
updateContrastGUI2();
if (isLocActivated && exp->getEnabled()) { if (isLocActivated && exp->getEnabled()) {
if (listener) { if (listener) {
if (wavedg->get_active()) { if (wavedg->get_active()) {
@ -3311,7 +3308,7 @@ void LocallabContrast::localedgMethodChanged()
void LocallabContrast::waveshowChanged() void LocallabContrast::waveshowChanged()
{ {
// Update Local contrast GUI according to waveshow button state // Update Local contrast GUI according to waveshow button state
updateContrastGUI3(); updateContrastGUI2();
if (isLocActivated && exp->getEnabled()) { if (isLocActivated && exp->getEnabled()) {
if (listener) { if (listener) {
@ -3399,7 +3396,6 @@ void LocallabContrast::wavcompreChanged()
void LocallabContrast::wavcompChanged() void LocallabContrast::wavcompChanged()
{ {
if (isLocActivated && exp->getEnabled()) { if (isLocActivated && exp->getEnabled()) {
wavcomp->set_active(false);//always disabled
if (listener) { if (listener) {
if (wavcomp->get_active()) { if (wavcomp->get_active()) {
listener->panelChanged(Evlocallabwavcomp, listener->panelChanged(Evlocallabwavcomp,
@ -3415,7 +3411,7 @@ void LocallabContrast::wavcompChanged()
void LocallabContrast::fftwlcChanged() void LocallabContrast::fftwlcChanged()
{ {
// Update Local contrast GUI according to fftwlc button state // Update Local contrast GUI according to fftwlc button state
updateContrastGUI4(); updateContrastGUI3();
if (isLocActivated && exp->getEnabled()) { if (isLocActivated && exp->getEnabled()) {
if (listener) { if (listener) {
@ -3546,16 +3542,6 @@ void LocallabContrast::updateContrastGUI1()
} }
void LocallabContrast::updateContrastGUI2() void LocallabContrast::updateContrastGUI2()
{
// Update Local contrast GUI according to wavedg button state
if (wavedg->get_active()) {
edgsBox->show();
} else {
edgsBox->show();//always show
}
}
void LocallabContrast::updateContrastGUI3()
{ {
// Update Local contrast GUI according to waveshow button state // Update Local contrast GUI according to waveshow button state
if (waveshow->get_active()) { if (waveshow->get_active()) {
@ -3565,7 +3551,7 @@ void LocallabContrast::updateContrastGUI3()
} }
} }
void LocallabContrast::updateContrastGUI4() void LocallabContrast::updateContrastGUI3()
{ {
// Update Local contrast GUI according to fftwlc button state // Update Local contrast GUI according to fftwlc button state
const double temp = lcradius->getValue(); const double temp = lcradius->getValue();
@ -3584,8 +3570,7 @@ LocallabCBDL::LocallabCBDL():
LocallabTool(this, M("TP_LOCALLAB_CBDL_TOOLNAME"), M("TP_LOCALLAB_CBDL"), true), LocallabTool(this, M("TP_LOCALLAB_CBDL_TOOLNAME"), M("TP_LOCALLAB_CBDL"), true),
// CBDL specific widgets // CBDL specific widgets
multiplier( multiplier([]() -> std::array<Adjuster *, 6>
[this]() -> std::array<Adjuster *, 6>
{ {
std::array<Adjuster*, 6> res = {}; std::array<Adjuster*, 6> res = {};
@ -3603,8 +3588,7 @@ LocallabCBDL::LocallabCBDL():
return res; return res;
} }
() ()),
),
chromacbdl(Gtk::manage(new Adjuster(M("TP_LOCALLAB_CHROMACBDL"), 0., 1.5, 0.01, 0.))), chromacbdl(Gtk::manage(new Adjuster(M("TP_LOCALLAB_CHROMACBDL"), 0., 1.5, 0.01, 0.))),
threshold(Gtk::manage(new Adjuster(M("TP_DIRPYREQUALIZER_THRESHOLD"), 0, 1., 0.01, 0.2))), threshold(Gtk::manage(new Adjuster(M("TP_DIRPYREQUALIZER_THRESHOLD"), 0, 1., 0.01, 0.2))),
blurcbdl(Gtk::manage(new Adjuster(M("TP_LOCALLAB_BLURCBDL"), 0., 100., 0.1, 0.))), blurcbdl(Gtk::manage(new Adjuster(M("TP_LOCALLAB_BLURCBDL"), 0., 100., 0.1, 0.))),