From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: redhat.com, ip: 209.132.183.28, mailfrom: lersek@redhat.com) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by groups.io with SMTP; Thu, 05 Sep 2019 11:38:24 -0700 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 82509190C006; Thu, 5 Sep 2019 18:38:23 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-116-133.ams2.redhat.com [10.36.116.133]) by smtp.corp.redhat.com (Postfix) with ESMTP id 157465D712; Thu, 5 Sep 2019 18:38:21 +0000 (UTC) From: "Laszlo Ersek" To: edk2-devel-groups-io Cc: Andrew Fish , Leif Lindholm , Michael D Kinney , Rebecca Cran Subject: [PATCH edk2-CCSS 0/3] Coding Standards: add rule for documenting spurious variable assignments Date: Thu, 5 Sep 2019 20:38:17 +0200 Message-Id: <20190905183820.10312-1-lersek@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.6.2 (mx1.redhat.com [10.5.110.70]); Thu, 05 Sep 2019 18:38:23 +0000 (UTC) Content-Transfer-Encoding: quoted-printable Repo: https://github.com/lersek/edk2-CCodingStandardsSpecification.git Branch: spurious_assign_bz_607 HTML-rendered views of the modified pages: - https://lersek.gitbooks.io/laszlo-s-fork-of-the-edk-ii-c-coding-standar= ds-sp/content/v/spurious_assign_bz_607 - https://lersek.gitbooks.io/laszlo-s-fork-of-the-edk-ii-c-coding-standar= ds-sp/content/v/spurious_assign_bz_607/6_documenting_software/62_comments= .html - https://lersek.gitbooks.io/laszlo-s-fork-of-the-edk-ii-c-coding-standar= ds-sp/content/v/spurious_assign_bz_607/6_documenting_software/64_what_you= _must_comment.html The first two patches are cleanups for things that popped up in the discussion in . The third patch is the one fixing the BZ. Thanks, Laszlo Cc: Andrew Fish Cc: Leif Lindholm Cc: Michael D Kinney Cc: Rebecca Cran Laszlo Ersek (3): comments: remove "Horror Vacui" rule comments: restrict and clarify applicability of "/*" comments must comment: add rule for documenting spurious variable assignments 6_documenting_software/62_comments.md | 20 +--------- 6_documenting_software/64_what_you_must_comment.md | 39 ++++++++++++++++= ++++ README.md | 1 + 3 files changed, 42 insertions(+), 18 deletions(-) --=20 2.19.1.3.g30247aa5d201