[libvirt] [PATCH 29/29] Require space after cast

Ján Tomko jtomko at redhat.com
Mon Apr 23 13:04:07 UTC 2018


On Mon, Apr 23, 2018 at 02:44:57PM +0200, Martin Kletzander wrote:
>Let's make a rule out of it and document it.  This is based on few sources:
>
>1) Most of the code [1] used spaces after casts, so the patch to change it this
>   way rather than the other way around is smaller
>

Just because it's popular does not mean it's right.

>2) I asked the first libvirt developer on my left when deciding, they preferred
>   spaces

Selection bias.

>
>3) My own preference.
>

De gustibus non est disputandum.

>4) The fact that this is clearly the superior way of casting =D
>

[citation needed]

>[1] 54.85% is more than 50%, plus it is increasing as it was 52.96% during the
>    first draft of this clean-up.
>
>Signed-off-by: Martin Kletzander <mkletzan at redhat.com>

ACK if you forbid spaces after casts instead.

Jano

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20180423/204fda8f/attachment-0001.sig>


More information about the libvir-list mailing list