Re: [hackers] [slock] [PATCH] Refactor main()

From: Ali H. Fardan <raiz_AT_firemail.cc>
Date: Mon, 22 Aug 2016 19:34:24 +0300

On 2016-08-22 10:37, FRIGN wrote:
> It saves one line to leave the brackets out, but it has happened to me
> more than once that people add a line to the if clause not paying
> attention and effectively introducing a bug. And FWIW, in my opinion,
> having brackets improves readability a bit.
> But again, if it is your explicit desire, I can create a version of the
> patch with the brackets removed.
>
> Cheers
>
> FRIGN

It's their fault, and if it was a mistake, people will notice it in the
patch before anyone proceeds to applying it.

Raiz
Received on Mon Aug 22 2016 - 18:34:24 CEST