[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v4 04/11] xsm: apply coding style


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: "Daniel P. Smith" <dpsmith@xxxxxxxxxxxxxxxxxxxx>
  • Date: Tue, 7 Sep 2021 10:09:12 -0400
  • Arc-authentication-results: i=1; mx.zohomail.com; dkim=pass header.i=apertussolutions.com; spf=pass smtp.mailfrom=dpsmith@xxxxxxxxxxxxxxxxxxxx; dmarc=pass header.from=<dpsmith@xxxxxxxxxxxxxxxxxxxx>
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1631023862; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:To; bh=NGknd/JHTSfyOh9tzQSO+4h9Q2/QFArTITRMaNAFEqs=; b=juQWA93l5PqfG068ROI0otQkMXrRjgB8gNMUnU4plSFOI4dXmW56aGY0hvNbzE0wS/nrJFbdyX7Vt6+Aw8zmt2TYypHCDpn4ObYFj+Ix1GgQNuSZl8E+NErq8UMTHcGgThucIUq71++r2ukwk8JUaj6RvLeiYbcTCTewYAug3Ng=
  • Arc-seal: i=1; a=rsa-sha256; t=1631023862; cv=none; d=zohomail.com; s=zohoarc; b=ZIL3ngtVoMCpENgJBqG6qOYrPmBTvfSmYGYT8jlTyBVe8U2ePx8VpgTasGI+kphTyo4ro3Il8mVA8m6ez/s80TvKrK614WL/ZUI/HKh2S85c4HvHwoMOzUSpPQhNzABkd/WhlXwatH46GTnzC3eWVUO6e/GsThtIq1msUG070ZI=
  • Cc: Daniel De Graaf <dgdegra@xxxxxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
  • Delivery-date: Tue, 07 Sep 2021 14:11:10 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 9/7/21 9:50 AM, Jan Beulich wrote:
On 07.09.2021 15:41, Daniel P. Smith wrote:
On 9/6/21 2:17 PM, Andrew Cooper wrote:
On 03/09/2021 20:06, Daniel P. Smith wrote:
--- a/xen/include/xsm/dummy.h
+++ b/xen/include/xsm/dummy.h
@@ -69,8 +69,9 @@ void __xsm_action_mismatch_detected(void);
#endif /* CONFIG_XSM */ -static always_inline int xsm_default_action(
-    xsm_default_t action, struct domain *src, struct domain *target)
+static always_inline int xsm_default_action(xsm_default_t action,
+                                            struct domain *src,
+                                            struct domain *target)

The old code is correct.  We have plenty of examples of this in Xen, and
I have been adding new ones when appropriate.

It avoids squashing everything on the RHS and ballooning the line count
to compensate.  (This isn't a particularly bad example, but we've had
worse cases in the past).

Based on the past discussions I understood either is acceptable and find
this version much easier to visually parse myself. With that said, if
the "next line single indent" really is the preferred style by the
maintainers/community, then I can convert all of these over.

I guess neither is the "preferred" style; as Andrew says, both are
acceptable and both are in active use. I guess the rule of thumb is:
The longer what's left of the function name, the more you should
consider the style that you change away from.

Anyway, in the end I guess the request for style adjustments was
mainly to purge bad style, not to convert one acceptable form to
another. Converting the entire file to the same style is of course
fine (for producing a consistent result), but then - as per above -
here it would more likely be the one that in this case was already
there.

Understood, I will respin with all the function defs to align with the "next line single indent" style, though it would be helpful for clarification on this style exactly. Do you always wrap all args if one extends past 80 col or is there a rule for when some should remain on the first line (function def line)?

v/r,
dps



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.