Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20180111091229.3482569b@inria.fr>
Date: Thu, 11 Jan 2018 09:12:29 +0100
From: Jens Gustedt <jens.gustedt@...ia.fr>
Cc: musl@...ts.openwall.com
Subject: Re: alternative form flag with zero octal value

Hello,
seeing this patch, I am somewhat horrified by the coding style that is
applied in that file.

Comma operator, seriously? Conditionals without proper indentation?
"else" at the end of a line and the depending statement in the next
case?

If have nothing against clever use of switch cases for such
complicated case analysis, but the coding style should prominently
make this clear and not obfuscate.


Thanks
Jens

-- 
:: INRIA Nancy Grand Est ::: Camus ::::::: ICube/ICPS :::
:: ::::::::::::::: office Strasbourg : +33 368854536   ::
:: :::::::::::::::::::::: gsm France : +33 651400183   ::
:: ::::::::::::::: gsm international : +49 15737185122 ::
:: http://icube-icps.unistra.fr/index.php/Jens_Gustedt ::

Content of type "application/pgp-signature" skipped

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.