From nobody Tue Dec 24 01:25:39 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; spf=none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) smtp.mailfrom=edk2-devel-bounces@lists.01.org Return-Path: Received: from ml01.01.org (ml01.01.org [198.145.21.10]) by mx.zohomail.com with SMTPS id 151497858842082.82544403999088; Wed, 3 Jan 2018 03:23:08 -0800 (PST) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 2EA50222A54F3; Wed, 3 Jan 2018 03:18:00 -0800 (PST) Received: from cam-smtp0.cambridge.arm.com (fw-tnat.cambridge.arm.com [217.140.96.140]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id E46AF222A54E3 for ; Wed, 3 Jan 2018 03:17:56 -0800 (PST) Received: from E111747.Emea.Arm.com (e111747.emea.arm.com [10.1.27.84]) by cam-smtp0.cambridge.arm.com (8.13.8/8.13.8) with ESMTP id w03BMrgo021422; Wed, 3 Jan 2018 11:22:54 GMT X-Original-To: edk2-devel@lists.01.org Received-SPF: none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) client-ip=198.145.21.10; envelope-from=edk2-devel-bounces@lists.01.org; helo=ml01.01.org; Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=217.140.96.140; helo=cam-smtp0.cambridge.arm.com; envelope-from=evan.lloyd@arm.com; receiver=edk2-devel@lists.01.org From: evan.lloyd@arm.com To: edk2-devel@lists.01.org Date: Wed, 3 Jan 2018 11:22:48 +0000 Message-Id: <20180103112248.11880-6-evan.lloyd@arm.com> X-Mailer: git-send-email 2.14.1 In-Reply-To: <20180103112248.11880-1-evan.lloyd@arm.com> References: <20180103112248.11880-1-evan.lloyd@arm.com> MIME-Version: 1.0 Subject: [edk2] [edk2-CCodingStandardsSpecification PATCH 5/5] Fix Chapter 5 Typos X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: "Matteo.Carlini@arm.com"@arm.com, "nd@arm.com"@arm.com, "ard.biesheuvel@linaro.org"@arm.com, "lersek@redhat.com"@arm.com, "jordan.l.justen@intel.com"@arm.com, "liming.gao@intel.com"@arm.com, "leif.lindholm@linaro.org"@arm.com, "michael.d.kinney@intel.com"@arm.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" X-ZohoMail: RSF_4 Z_629925259 SPT_0 From: Evan Lloyd 5.1.1 - Replace "less" with "fewer" (because columns is plural and countable) 5.1.5 - Correct tense. (because the C specification still defines...) Insert full stop. Insert comma. 5.1.8 - Correct "provided" to "proven". 5.1.9 - remove hanging "This." 5.2.3.1 - replace "is comprised of" with "comprises" (comprise means "consists of", so "comprised of" is a solecism. Remove use of tab, as they are forbidden. Remove -- before date in copyright header (None of the edk2 files have it). 5.4 - Add indent to comment text. 5.6.1.2 - Fix copy/paste text (from UEFI spec). Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Evan Lloyd --- 5_source_files/52_spacing.md | 9 +++++---- 5_source_files/54_code_file_structure.md | 4 ++-- 5_source_files/56_declarations_and_types.md | 4 ++-- 5_source_files/README.md | 18 +++++++++--------- 4 files changed, 18 insertions(+), 17 deletions(-) diff --git a/5_source_files/52_spacing.md b/5_source_files/52_spacing.md index ddeabf7753a8713bf04e143d6e2e9bccf881f691..3c79f4e4ee91bcd4035d6cf7d8d= 32f1bb9c7756f 100644 --- a/5_source_files/52_spacing.md +++ b/5_source_files/52_spacing.md @@ -249,7 +249,7 @@ And the comment will end with: **/ ``` =20 -The File Heading comment block is comprised of the following sections: File +The File Heading comment block comprises the following sections: File Description, Copyright, License, and the optional Specification Reference = and Glossary sections. =20 @@ -266,8 +266,9 @@ Glossary sections. **/ ``` =20 -The following example begins each body line with a tab (two spaces). This = is -the preferred indentation, but two tabs (four spaces) is also acceptable. +The following example begins each body line with an indent (two spaces). +This is the preferred indentation, but a double indent (four spaces) is al= so +acceptable. =20 #### Example =20 @@ -278,7 +279,7 @@ the preferred indentation, but two tabs (four spaces) i= s also acceptable. Detailed description of the file=E2=80=99s contents and other useful information for a person viewing the file for the first time. =20 - Copyright (C) --20XX, Acme Corporation. All rights reserved.
+ Copyright (C) 20XX, Acme Corporation. All rights reserved.
This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full diff --git a/5_source_files/54_code_file_structure.md b/5_source_files/54_c= ode_file_structure.md index 8cc9f4f61412b07f765d80d7b680c6dd38b838c1..ac999aae99ae9cfd8b6f97dc483= e51bfbd7c7a0b 100644 --- a/5_source_files/54_code_file_structure.md +++ b/5_source_files/54_code_file_structure.md @@ -68,8 +68,8 @@ these are C files with an extension of "`.c`". =20 /* Function Definitions */ =20 -/* If this is a protocol definition, the -protocol structure is defined and initialized here. +/* If this is a protocol definition, the protocol structure is defined and + initialized here. */ ``` =20 diff --git a/5_source_files/56_declarations_and_types.md b/5_source_files/5= 6_declarations_and_types.md index ec1803d980e1fa808b9dc515cdffbc4b47437435..5c57834fe1195b5487f0f59e8a0= 385e44d91aff4 100644 --- a/5_source_files/56_declarations_and_types.md +++ b/5_source_files/56_declarations_and_types.md @@ -43,9 +43,9 @@ or from common EFI data types. The corresponding EFI types must be used instead. =20 "EFI Data Types" below contains the common data types that are referenced = in -the interface definitions defined by this specification. Per the _UEFI -Specification_, version 2.3.1: +the interface definitions defined by the UEFI specification. =20 +Per the _UEFI Specification_, version 2.3.1: "Unless otherwise specified, all data types are naturally aligned. Structu= res are aligned on boundaries equal to the largest internal datum of the struc= ture, and internal data is implicitly padded to achieve natural alignment." diff --git a/5_source_files/README.md b/5_source_files/README.md index a93492db4f0f17e14d9c2c3c95e57cf0f6cc911e..a443148138f2abaf6b9131f4758= 858a4a5f45fd3 100644 --- a/5_source_files/README.md +++ b/5_source_files/README.md @@ -33,9 +33,9 @@ =20 ## 5.1 General Rules =20 -### 5.1.1 Lines shall be 120 columns, or less +### 5.1.1 Lines shall be 120 columns, or fewer =20 -Preferably, limit line lengths to 80 columns or less. When this doesn't le= ave +Preferably, limit line lengths to 80 columns or fewer. When this doesn't l= eave sufficient space for a good postfix style comment, extend the line to a to= tal of 120 columns. Having some level of uniformity in the expected width of t= he source is useful for viewing and printing the code. @@ -79,9 +79,9 @@ Other than '\0', the only permissible escape sequences ar= e: =20 ### 5.1.5 Octal constants (Base 8) shall not be used. =20 -The C language specification has defined numbers whose first digit is zero= as -octal, so 010 is decimal 8 The use of octal has declined considerably sinc= e C -was first defined but this construct remains for backwards compatibility. = Its +The C language specification defines numbers whose first digit is zero as +octal, so 010 is decimal 8. The use of octal has declined considerably sin= ce C +was first defined, but this construct remains for backwards compatibility.= Its use is prohibited. In particular, do not be tempted to use the zero prefix= in tables of numbers to ensure visual alignment: =20 @@ -107,16 +107,16 @@ Trigraphs are a construct to allow character represen= tations that do not support all ASCII characters to enter the equivalent of the ASCII characte= r. Trigraphs are three characters long (hence the "tri"). The first two chara= cters are "??" while the third character disambiguates the trigraph. Technically -therefore, a[5] could be written a??(5??). Trigraphs have provided both +therefore, a[5] could be written a??(5??). Trigraphs have proven both confusing and unnecessary and are prohibited. =20 ### 5.1.9 In-line assembler shall not be used =20 There are really no reasons for in-line assembler to be used in EDK II cod= e. The only exceptions in this case are largely associated with the lowest le= vel -Architectural Protocols. Using in-line assembly language deviates against = the -Scope rules defined in Section 1.3 "Scope" because it is an extension to -standard C. This. +Architectural Protocols. Using in-line assembly language deviates from the +Scope rules defined in Section 1.3 "Scope", because it is an extension to +standard C. =20 ### 5.1.10 Do not use #pragma, except for #pragma pack (#). =20 --=20 Guid("CE165669-3EF3-493F-B85D-6190EE5B9759") _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel