public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Bob Feng" <bob.c.feng@intel.com>
To: Leif Lindholm <leif.lindholm@linaro.org>,devel@edk2.groups.io
Subject: Re: [edk2-devel] [PATCH] BaseTools: Fix various typos
Date: Thu, 04 Jul 2019 17:29:43 -0700	[thread overview]
Message-ID: <28803.1562286583784879484@groups.io> (raw)
In-Reply-To: <20190704130331.3muurxmqc6qpnh3f@bivouac.eciton.net>

[-- Attachment #1: Type: text/plain, Size: 16015 bytes --]

The attached patch looks fine.
Reviewed-by: Bob Feng <bob.c.feng@intel.com>

On Thu, Jul 4, 2019 at 09:03 PM, Leif Lindholm wrote:

> 
> +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)
>> 
>> 
>> 
> 
>

[-- Attachment #2: Type: text/html, Size: 17596 bytes --]

      reply	other threads:[~2019-07-05  0:29 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
2019-07-05  0:29   ` Bob Feng [this message]

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=28803.1562286583784879484@groups.io \
    --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