public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Leif Lindholm" <leif.lindholm@linaro.org>
To: devel@edk2.groups.io, coeur@gmx.fr
Cc: Bob Feng <bob.c.feng@intel.com>, Liming Gao <liming.gao@intel.com>
Subject: Re: [edk2-devel] [PATCH] BaseTools: Fix various typos
Date: Thu, 4 Jul 2019 14:03:31 +0100	[thread overview]
Message-ID: <20190704130331.3muurxmqc6qpnh3f@bivouac.eciton.net> (raw)
In-Reply-To: <18238.1562166539775741502@groups.io>

+Bob, Liming

On Wed, Jul 03, 2019 at 08:08:59AM -0700, Cœur wrote:
> Fix various typos in BaseTools.
> 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Coeur <coeur@gmx.fr>
> ---
> BaseTools/Conf/XMLSchema/DistributionPackage.xsd          | 8 ++++----
> .../plugins/EdkPlugins/edk2/model/doxygengen.py           | 2 +-
> .../plugins/EdkPlugins/edk2/model/doxygengen_spec.py      | 2 +-
> BaseTools/Source/C/Common/FirmwareVolumeBuffer.c          | 4 ++--
> BaseTools/Source/C/Include/Common/BaseTypes.h             | 2 +-
> BaseTools/Source/C/Makefiles/NmakeSubdirs.py              | 2 +-
> BaseTools/Source/C/VfrCompile/Pccts/antlr/lex.c           | 2 +-
> BaseTools/Source/C/VfrCompile/Pccts/h/err.h               | 2 +-
> BaseTools/Source/Python/AutoGen/AutoGen.py                | 4 ++--
> BaseTools/Source/Python/BPDG/GenVpd.py                    | 6 +++---
> BaseTools/Source/Python/Common/ToolDefClassObject.py      | 2 +-
> BaseTools/Source/Python/UPT/Library/DataType.py           | 2 +-
> BaseTools/Source/Python/UPT/Library/StringUtils.py        | 2 +-
> BaseTools/Source/Python/UPT/Logger/StringTable.py         | 2 +-
> .../Python/UPT/Object/Parser/InfBuildOptionObject.py      | 2 +-
> 15 files changed, 22 insertions(+), 22 deletions(-)
> 
> diff --git a/BaseTools/Conf/XMLSchema/DistributionPackage.xsd b/BaseTools/Conf/XMLSchema/DistributionPackage.xsd
> index 7499c325c3..67be9c8f32 100644
> --- a/BaseTools/Conf/XMLSchema/DistributionPackage.xsd
> +++ b/BaseTools/Conf/XMLSchema/DistributionPackage.xsd
> @@ -166,7 +166,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> Package.
> </xs:documentation>
> <xs:documentation xml:lang="en-us">
> -              Each Distribution Package is uniquely identified by it's
> +              Each Distribution Package is uniquely identified by its
> GUID and Version number.
> </xs:documentation>
> </xs:annotation>
> @@ -363,7 +363,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> Modules.
> </xs:documentation>
> <xs:documentation xml:lang="en-us">
> -        Each package is uniquely identified by it's GUID and Version number.
> +        Each package is uniquely identified by its GUID and Version number.
> Backward compatible releases of a package need only change the version number, while non-backward
> compatible changes require the GUID to change (resetting the version number to 1.0 is optional.)
> </xs:documentation>
> @@ -1168,7 +1168,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> <xs:element name="ModuleSurfaceArea">
> <xs:annotation>
> <xs:documentation xml:lang="en-us">
> -        Each module is uniquely identified by it's GUID and Version number.
> +        Each module is uniquely identified by its GUID and Version number.
> Backward compatible releases of a module need only change the version number, while non-backward
> compatible changes require the GUID to change (resetting the version number to 1.0 is optional.)
> </xs:documentation>
> @@ -3092,7 +3092,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> <xs:documentation xml:lang="en-us">
> The Patch PCD is a #define that is set at build time, and that
> can be modified within a binary file. Additional information, such as the offset location of the
> -            value, along with it's length may need to be provided.
> +            value, along with its length may need to be provided.
> </xs:documentation>
> </xs:annotation>
> </xs:enumeration>
> diff --git a/BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/edk2/model/doxygengen.py b/BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/edk2/model/doxygengen.py
> index d0274d6afd..89833043c6 100644
> --- a/BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/edk2/model/doxygengen.py
> +++ b/BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/edk2/model/doxygengen.py
> @@ -792,7 +792,7 @@ class PackageDocumentAction(DoxygenAction):
> Generate page for a module/library.
> @param infObj     INF file object for module/library
> @param configFile doxygen config file object
> -        @param isLib      Whether this module is libary
> +        @param isLib      Whether this module is library
> 
> @param module doxygen page object
> """
> diff --git a/BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/edk2/model/doxygengen_spec.py b/BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/edk2/model/doxygengen_spec.py
> index 3ba4eed55e..e37938c466 100644
> --- a/BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/edk2/model/doxygengen_spec.py
> +++ b/BaseTools/Scripts/PackageDocumentTools/plugins/EdkPlugins/edk2/model/doxygengen_spec.py
> @@ -793,7 +793,7 @@ class PackageDocumentAction(DoxygenAction):
> Generate page for a module/library.
> @param infObj     INF file object for module/library
> @param configFile doxygen config file object
> -        @param isLib      Whether this module is libary
> +        @param isLib      Whether this module is library
> 
> @param module doxygen page object
> """
> diff --git a/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c b/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
> index 6e1914be29..70741c8afc 100644
> --- a/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
> +++ b/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
> @@ -937,7 +937,7 @@ Arguments:
> 
> Fv - Address of the Fv in memory
> Key - Should be 0 to get the first file.  After that, it should be
> -        passed back in without modifying it's contents to retrieve
> +        passed back in without modifying its contents to retrieve
> subsequent files.
> File - Output file pointer
> File == NULL - invalid parameter
> @@ -1323,7 +1323,7 @@ Arguments:
> SectionsStart - Address of the start of the FFS sections array
> TotalSectionsSize - Total size of all the sections
> Key - Should be 0 to get the first section.  After that, it should be
> -        passed back in without modifying it's contents to retrieve
> +        passed back in without modifying its contents to retrieve
> subsequent files.
> Section - Output section pointer
> (Section == NULL) -> invalid parameter
> diff --git a/BaseTools/Source/C/Include/Common/BaseTypes.h b/BaseTools/Source/C/Include/Common/BaseTypes.h
> index 9844d1d514..31d0662085 100644
> --- a/BaseTools/Source/C/Include/Common/BaseTypes.h
> +++ b/BaseTools/Source/C/Include/Common/BaseTypes.h
> @@ -151,7 +151,7 @@ typedef struct {
> 
> ///
> ///  _CR - returns a pointer to the structure
> -///      from one of it's elements.
> +///      from one of its elements.
> ///
> #define _CR(Record, TYPE, Field)  ((TYPE *) ((CHAR8 *) (Record) - (CHAR8 *) &(((TYPE *) 0)->Field)))
> 
> diff --git a/BaseTools/Source/C/Makefiles/NmakeSubdirs.py b/BaseTools/Source/C/Makefiles/NmakeSubdirs.py
> index 30a481613e..356f5aca63 100644
> --- a/BaseTools/Source/C/Makefiles/NmakeSubdirs.py
> +++ b/BaseTools/Source/C/Makefiles/NmakeSubdirs.py
> @@ -104,7 +104,7 @@ class ThreadControl(object):
> while len(self.running) > 0:
> time.sleep(0.1)
> if self.error:
> -            print("subprocess not exit sucessfully")
> +            print("subprocess not exit successfully")
> print(self.errorMsg)
> 
> def startTask(self):
> diff --git a/BaseTools/Source/C/VfrCompile/Pccts/antlr/lex.c b/BaseTools/Source/C/VfrCompile/Pccts/antlr/lex.c
> index 357b8ace01..fddb46bbc3 100644
> --- a/BaseTools/Source/C/VfrCompile/Pccts/antlr/lex.c
> +++ b/BaseTools/Source/C/VfrCompile/Pccts/antlr/lex.c
> @@ -298,7 +298,7 @@ genDefFile( )
> /* look in all lexclasses for the reg expr */
> 
> /* MR10  Derek Pappas                                                */
> -/* MR10     A #tokclass doesn't have associated regular expressiones */
> +/* MR10     A #tokclass doesn't have associated regular expressions */
> /* MR10        so don't warn user about it's omission                */
> 
> p = (TermEntry *) hash_get(Tname, TokenString(i));
> diff --git a/BaseTools/Source/C/VfrCompile/Pccts/h/err.h b/BaseTools/Source/C/VfrCompile/Pccts/h/err.h
> index b2b196bd75..435d48356d 100644
> --- a/BaseTools/Source/C/VfrCompile/Pccts/h/err.h
> +++ b/BaseTools/Source/C/VfrCompile/Pccts/h/err.h
> @@ -649,7 +649,7 @@ _inf_zzgettok()
> /* allocate default size text,token and line arrays;
> * then, read all of the input reallocing the arrays as needed.
> * Once the number of total tokens is known, the LATEXT(i) array (zzinf_text)
> - * is allocated and it's pointers are set to the tokens in zzinf_text_buffer.
> + * is allocated and its pointers are set to the tokens in zzinf_text_buffer.
> */
> void
> #ifdef __USE_PROTOS
> diff --git a/BaseTools/Source/Python/AutoGen/AutoGen.py b/BaseTools/Source/Python/AutoGen/AutoGen.py
> index e8e09dc8a3..a2077c5516 100644
> --- a/BaseTools/Source/Python/AutoGen/AutoGen.py
> +++ b/BaseTools/Source/Python/AutoGen/AutoGen.py
> @@ -3030,7 +3030,7 @@ class ModuleAutoGen(AutoGen):
> IncPathList = [NormPath(Path, self.Macros) for Path in BuildOptIncludeRegEx.findall(FlagOption)]
> else:
> #
> -                # RVCT may specify a list of directory seperated by commas
> +                # RVCT may specify a list of directory separated by commas
> #
> IncPathList = []
> for Path in BuildOptIncludeRegEx.findall(FlagOption):
> @@ -3994,7 +3994,7 @@ class ModuleAutoGen(AutoGen):
> #
> @cached_class_function
> def CreateMakeFile(self, CreateLibraryMakeFile=True, GenFfsList = []):
> -        # nest this function inside it's only caller.
> +        # nest this function inside its only caller.
> def CreateTimeStamp():
> FileSet = {self.MetaFile.Path}
> 
> diff --git a/BaseTools/Source/Python/BPDG/GenVpd.py b/BaseTools/Source/Python/BPDG/GenVpd.py
> index 887a0c4a1f..049c082f40 100644
> --- a/BaseTools/Source/Python/BPDG/GenVpd.py
> +++ b/BaseTools/Source/Python/BPDG/GenVpd.py
> @@ -312,7 +312,7 @@ class GenVPD :
> 
> ##
> # Parser the input file which is generated by the build tool. Convert the value of each pcd's
> -    # from string to it's real format. Also remove the useless line in the input file.
> +    # from string to its real format. Also remove the useless line in the input file.
> #
> def ParserInputFile (self):
> count = 0
> @@ -483,7 +483,7 @@ class GenVPD :
> self.PcdFixedOffsetSizeList.sort(key=lambda x: x.PcdBinOffset)
> 
> #
> -        # Sort the un-fixed pcd's offset by it's size.
> +        # Sort the un-fixed pcd's offset by its size.
> #
> self.PcdUnknownOffsetList.sort(key=lambda x: x.PcdBinSize)
> 
> @@ -605,7 +605,7 @@ class GenVPD :
> # Usually it will not enter into this thunk, if so, means it overlapped.
> else :
> EdkLogger.error("BPDG", BuildToolError.ATTRIBUTE_NOT_AVAILABLE,
> -                                "The offset value definition has overlapped at pcd: %s, it's offset is: %s, in file: %s line: %s" % \
> +                                "The offset value definition has overlapped at pcd: %s, its offset is: %s, in file: %s line: %s" % \
> (eachFixedPcd.PcdCName, eachFixedPcd.PcdOffset, eachFixedPcd.InputFileName, eachFixedPcd.Lineno),
> None)
> FixOffsetSizeListCount += 1
> diff --git a/BaseTools/Source/Python/Common/ToolDefClassObject.py b/BaseTools/Source/Python/Common/ToolDefClassObject.py
> index 5d692ea13c..4fa364942c 100644
> --- a/BaseTools/Source/Python/Common/ToolDefClassObject.py
> +++ b/BaseTools/Source/Python/Common/ToolDefClassObject.py
> @@ -105,7 +105,7 @@ class ToolDefClassObject(object):
> 
> ## IncludeToolDefFile
> #
> -    # Load target.txt file and parse it as if it's contents were inside the main file
> +    # Load target.txt file and parse it as if its contents were inside the main file
> #
> # @param Filename:  Input value for full path of tools_def.txt
> #
> diff --git a/BaseTools/Source/Python/UPT/Library/DataType.py b/BaseTools/Source/Python/UPT/Library/DataType.py
> index 88559bcc43..bd216786a5 100644
> --- a/BaseTools/Source/Python/UPT/Library/DataType.py
> +++ b/BaseTools/Source/Python/UPT/Library/DataType.py
> @@ -210,7 +210,7 @@ ARCH_LIST = ["IA32",
> ##
> # PCD driver type list items
> #
> -PCD_DIRVER_TYPE_LIST = ["PEI_PCD_DRIVER", "DXE_PCD_DRIVER"]
> +PCD_DRIVER_TYPE_LIST = ["PEI_PCD_DRIVER", "DXE_PCD_DRIVER"]
> 
> ##
> # Boot Mode List Items
> diff --git a/BaseTools/Source/Python/UPT/Library/StringUtils.py b/BaseTools/Source/Python/UPT/Library/StringUtils.py
> index e43924fa6c..fbc5177caf 100644
> --- a/BaseTools/Source/Python/UPT/Library/StringUtils.py
> +++ b/BaseTools/Source/Python/UPT/Library/StringUtils.py
> @@ -863,7 +863,7 @@ def ConvertNOTEQToNE(Expr):
> ## SplitPcdEntry
> #
> # Split an PCD entry string to Token.CName and PCD value and FFE.
> -# NOTE: PCD Value and FFE can contain "|" in it's expression. And in INF specification, have below rule.
> +# NOTE: PCD Value and FFE can contain "|" in its expression. And in INF specification, have below rule.
> # When using the characters "|" or "||" in an expression, the expression must be encapsulated in
> # open "(" and close ")" parenthesis.
> #
> diff --git a/BaseTools/Source/Python/UPT/Logger/StringTable.py b/BaseTools/Source/Python/UPT/Logger/StringTable.py
> index 7782d26238..13c015844e 100644
> --- a/BaseTools/Source/Python/UPT/Logger/StringTable.py
> +++ b/BaseTools/Source/Python/UPT/Logger/StringTable.py
> @@ -251,7 +251,7 @@ ERR_ASBUILD_PCD_SECTION_TYPE = _("The AsBuilt INF file contains a PCD section ty
> ERR_ASBUILD_PATCHPCD_FORMAT_INVALID = _("The AsBuilt PatchPcd entry must contain 3 elements: PcdName|Value|Offset")
> ERR_ASBUILD_PCDEX_FORMAT_INVALID = _("The AsBuilt PcdEx entry must contain one element: PcdName")
> ERR_ASBUILD_PCD_VALUE_INVALID = \
> -    _("The AsBuilt PCD value %s is incorrect or not align with it's datum type %s. "
> +    _("The AsBuilt PCD value %s is incorrect or not align with its datum type %s. "
> "It must be corrected before continuing.")
> ERR_ASBUILD_PCD_TOKENSPACE_GUID_VALUE_MISS = _("Package file value could not be retrieved for %s.")
> ERR_ASBUILD_PCD_DECLARITION_MISS = _("PCD Declaration in DEC files could not be found for: %s.")
> diff --git a/BaseTools/Source/Python/UPT/Object/Parser/InfBuildOptionObject.py b/BaseTools/Source/Python/UPT/Object/Parser/InfBuildOptionObject.py
> index 3383dc4b0f..fdba5db983 100644
> --- a/BaseTools/Source/Python/UPT/Object/Parser/InfBuildOptionObject.py
> +++ b/BaseTools/Source/Python/UPT/Object/Parser/InfBuildOptionObject.py
> @@ -49,7 +49,7 @@ class InfBuildOptionsObject(InfSectionCommonDef):
> InfSectionCommonDef.__init__(self)
> ## SetBuildOptions function
> #
> -    # For BuildOptionName, need to validate it's format
> +    # For BuildOptionName, need to validate its format
> # For BuildOptionValue, just ignore it.
> #
> # @param  Arch          Indicated which arch of build options belong to.
> --
> 2.20.1 (Apple Git-117)
> 
> 
> 

  parent reply	other threads:[~2019-07-04 13:03 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-03 15:08 [PATCH] BaseTools: Fix various typos Cœur
2019-07-04 11:16 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-07-04 11:21   ` Philippe Mathieu-Daudé
2019-07-04 11:47     ` Cœur
2019-07-04 13:03 ` Leif Lindholm [this message]
2019-07-05  0:29   ` Bob Feng

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190704130331.3muurxmqc6qpnh3f@bivouac.eciton.net \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox