Looking at the change in context, it might be a slight improvement in readability, using a positive vs a double negative, it's still mostly a no-op though. that logic is so simple it shouldn't be confusing either way when read in context.
On Wed, Apr 13, 2016 at 9:53 AM, Pereira, Patrick <patrick.pere...@intel.com> wrote: > Ooooops, sorry for my inattention. > > Correction in attachment. > > ________________________________________ > De: Kicad-developers > [kicad-developers-bounces+patrick.pereira=intel....@lists.launchpad.net] em > nome de Lorenzo Marcantonio [l.marcanto...@cz-dynamic.it] > Enviado: quarta-feira, 13 de abril de 2016 3:19 > Para: Kicad Developers > Assunto: Re: [Kicad-developers] [PATCH] Logic reformulation in > Plot_Edges_Modules > > On Tue, Apr 12, 2016 at 09:16:00PM +0000, Pereira, Patrick wrote: >> Hi, >> >> Corrections to avoid unnecessary boolean operations >> !(!a || !b) = a || b > > You just insulted DeMorgan :D Hope the code is better > > -- > Lorenzo Marcantonio > CZ Srl - Parma > > _______________________________________________ > Mailing list: https://launchpad.net/~kicad-developers > Post to : kicad-developers@lists.launchpad.net > Unsubscribe : https://launchpad.net/~kicad-developers > More help : https://help.launchpad.net/ListHelp > _______________________________________________ Mailing list: https://launchpad.net/~kicad-developers Post to : kicad-developers@lists.launchpad.net Unsubscribe : https://launchpad.net/~kicad-developers More help : https://help.launchpad.net/ListHelp