LVS
lvs-devel
Google
 
Web LinuxVirtualServer.org

Re: [v2 PATCH 3/4] IPVS: Backup, Adding Version 1 receive capability

To: Simon Horman <horms@xxxxxxxxxxxx>
Subject: Re: [v2 PATCH 3/4] IPVS: Backup, Adding Version 1 receive capability
Cc: Hans Schillstrom <hans.schillstrom@xxxxxxxxxxxx>, lvs-devel@xxxxxxxxxxxxxxx, wensong@xxxxxxxxxxxx, daniel.lezcano@xxxxxxx
From: Julian Anastasov <ja@xxxxxx>
Date: Sat, 30 Oct 2010 13:59:34 +0300 (EEST)

        Hello,

On Sat, 30 Oct 2010, Simon Horman wrote:

I realise that the commenting syntax used throughout IPVS
is a) inconsistent and b) for the most part does not comply with
any variant of the style guidelines. However, for the record
I believe that the preferred commenting style for code in net/
and thus IPVS is:

/* This is a one line comment */

/* This is a
* multi-line comment
*/

        Documentation/CodingStyle has rule for multi-line:

/*
 * This is a
 * multi-line comment
 */

+               ip_vs_conn_fill_param(af, sc->v6.protocol,
+                                  (const union nf_inet_addr *)&sc->v6.caddr,
+                                  sc->v6.cport,
+                                  (const union nf_inet_addr *)&sc->v6.vaddr,
+                                  sc->v6.vport, p);

I'm ok with not aligning to the opening ( in order to avoid
going over 80 columns here. Though I suspect others would disagree.

        Agreed, in such cases I prefer one tab after parent,
especially, if there is no other code around and when not
part of some complex 'if' expression.

        I'm now sure Hans will read Documentation/CodingStyle again :)

Regards

--
Julian Anastasov <ja@xxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

<Prev in Thread] Current Thread [Next in Thread>