Hello,
Indeed, I think I think the "else if" should have been removed (even if it was very long ago and I don't completely remember)

Thanks
Aldric

Le 18/12/2018 à 17:35, Matthieu Moy a écrit :
----- Original Message -----
From: "Heiko Bauke" <heiko.ba...@mail.de>
else if(p->k_type == 0)
{
d->all_off = 1;
d->k_apply = 0;
}
else
{
d->all_off = 1;
d->k_apply = 0;
}
It was introduced in c53168bc5 (direct keystoning system, 2012-11-27) with a bit more 
code in the 'else' branch. The extra code was removed in e298f3901 (clipping : cleanup 
old code, 2013-01-05). Probably that last commit should have removed the "else 
if" branch and kept only the else indeed. Cc-ing the author, in case.

Regards,

___________________________________________________________________________
darktable developer mailing list
to unsubscribe send a mail to darktable-dev+unsubscr...@lists.darktable.org

Reply via email to