From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.nue.novell.com (smtp.nue.novell.com [195.135.221.5]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id A607A1A1E6B for ; Wed, 19 Oct 2016 19:15:30 -0700 (PDT) Received: from nwb-ext-pat.microfocus.com ([10.120.13.103]) by smtp.nue.novell.com with ESMTP (TLS encrypted); Thu, 20 Oct 2016 03:55:27 +0200 Received: from GaryWorkstation (nwb-a10-snat.microfocus.com [10.120.13.202]) by nwb-ext-pat.microfocus.com with ESMTP (TLS encrypted); Thu, 20 Oct 2016 02:55:20 +0100 Date: Thu, 20 Oct 2016 09:55:14 +0800 From: Gary Lin To: "Carsey, Jaben" Cc: "Ni, Ruiyu" , "edk2-devel@lists.01.org" Message-ID: <20161020015514.nmt2zophuw6vwzdy@GaryWorkstation> References: <20161019070138.16424-1-glin@suse.com> <20161019070138.16424-31-glin@suse.com> <734D49CCEBEEF84792F5B80ED585239D58E3885D@SHSMSX104.ccr.corp.intel.com> MIME-Version: 1.0 In-Reply-To: User-Agent: Mutt/1.6.2 (2016-07-01) Subject: Re: [PATCH 30/33] ShellPkg: Fix typos in comments and variables X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 20 Oct 2016 02:15:31 -0000 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Wed, Oct 19, 2016 at 08:29:02PM +0000, Carsey, Jaben wrote: > Is this required to be a patch series? By that I mean that they need to be committed in order... > Every patch is independent. I made them a series for convenience. Thanks, Gary Lin > Reviewed-by: Jaben Carsey > > > -----Original Message----- > > From: Ni, Ruiyu > > Sent: Wednesday, October 19, 2016 12:52 AM > > To: Gary Lin ; edk2-devel@lists.01.org > > Cc: Carsey, Jaben > > Subject: RE: [PATCH 30/33] ShellPkg: Fix typos in comments and variables > > Importance: High > > > > > > > > Reviewed-by: Ruiyu Ni > > > -----Original Message----- > > > From: Gary Lin [mailto:glin@suse.com] > > > Sent: Wednesday, October 19, 2016 3:02 PM > > > To: edk2-devel@lists.01.org > > > Cc: Carsey, Jaben ; Ni, Ruiyu > > > > > Subject: [PATCH 30/33] ShellPkg: Fix typos in comments and variables > > > > > > - funciton -> function > > > - paramter -> parameter > > > - supresses -> suppresses > > > - Seperator -> Separator > > > - permenant -> permanent > > > > > > Cc: Jaben Carsey > > > Cc: Ruiyu Ni > > > Contributed-under: TianoCore Contribution Agreement 1.0 > > > Signed-off-by: Gary Lin > > > --- > > > ShellPkg/Include/Library/ShellCEntryLib.h | 2 +- > > > ShellPkg/Include/Library/ShellCommandLib.h | 2 +- > > > ShellPkg/Include/Protocol/EfiShellEnvironment2.h | 2 +- > > > ShellPkg/Library/UefiShellCEntryLib/UefiShellCEntryLib.c | 2 +- > > > ShellPkg/Library/UefiShellDriver1CommandsLib/Dh.c | 10 > > +++++- > > > ---- > > > > > > > > ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2CommandsLib > > > .c | 2 +- > > > ShellPkg/Library/UefiShellNetwork1CommandsLib/Ping.c | 8 > > > ++++---- > > > ShellPkg/Library/UefiShellNetwork2CommandsLib/Ping6.c | 8 > > > ++++---- > > > 8 files changed, 18 insertions(+), 18 deletions(-) > > > > > > diff --git a/ShellPkg/Include/Library/ShellCEntryLib.h > > > b/ShellPkg/Include/Library/ShellCEntryLib.h > > > index dfbb16f..e4983f8 100644 > > > --- a/ShellPkg/Include/Library/ShellCEntryLib.h > > > +++ b/ShellPkg/Include/Library/ShellCEntryLib.h > > > @@ -1,5 +1,5 @@ > > > /** @file > > > - Provides application point extension for "C" style main funciton. > > > + Provides application point extension for "C" style main function. > > > > > > Copyright (c) 2006 - 2011, Intel Corporation. All rights reserved.
> > > This program and the accompanying materials diff --git > > > a/ShellPkg/Include/Library/ShellCommandLib.h > > > b/ShellPkg/Include/Library/ShellCommandLib.h > > > index 3ee8200..5ff1e4f 100644 > > > --- a/ShellPkg/Include/Library/ShellCommandLib.h > > > +++ b/ShellPkg/Include/Library/ShellCommandLib.h > > > @@ -2,7 +2,7 @@ > > > Provides interface to shell internal functions for shell commands. > > > > > > This library is for use ONLY by shell commands linked into the shell > > > application. > > > - This library will not funciton if it is used for UEFI Shell 2.0 Applications. > > > + This library will not function if it is used for UEFI Shell 2.0 Applications. > > > > > > Copyright (c) 2009 - 2016, Intel Corporation. All rights reserved.
> > > (C) Copyright 2016 Hewlett Packard Enterprise Development LP
diff - > > - > > > git a/ShellPkg/Include/Protocol/EfiShellEnvironment2.h > > > b/ShellPkg/Include/Protocol/EfiShellEnvironment2.h > > > index 1aa1b7e..ac6c6a0 100644 > > > --- a/ShellPkg/Include/Protocol/EfiShellEnvironment2.h > > > +++ b/ShellPkg/Include/Protocol/EfiShellEnvironment2.h > > > @@ -124,7 +124,7 @@ GUID for the shell environment2 extension (main > > > GUID above). > > > @param[in] ParentImageHandle Handle of the image executing this > > > operation. > > > @param[in] CommandLine The string command line to execute. > > > @param[in] DebugOutput TRUE indicates that errors should be > > printed > > > directly. > > > - FALSE supresses error messages. > > > + FALSE suppresses error messages. > > > > > > @retval EFI_SUCCESS The command line executed and completed. > > > @retval EFI_ABORTED The operation aborted. > > > diff --git a/ShellPkg/Library/UefiShellCEntryLib/UefiShellCEntryLib.c > > > b/ShellPkg/Library/UefiShellCEntryLib/UefiShellCEntryLib.c > > > index 525c1f1..faed230 100644 > > > --- a/ShellPkg/Library/UefiShellCEntryLib/UefiShellCEntryLib.c > > > +++ b/ShellPkg/Library/UefiShellCEntryLib/UefiShellCEntryLib.c > > > @@ -1,5 +1,5 @@ > > > /** @file > > > - Provides application point extension for "C" style main funciton > > > + Provides application point extension for "C" style main function > > > > > > Copyright (c) 2009 - 2013, Intel Corporation. All rights reserved.
> > > This program and the accompanying materials diff --git > > > a/ShellPkg/Library/UefiShellDriver1CommandsLib/Dh.c > > > b/ShellPkg/Library/UefiShellDriver1CommandsLib/Dh.c > > > index 00a4c10..e07304c 100644 > > > --- a/ShellPkg/Library/UefiShellDriver1CommandsLib/Dh.c > > > +++ b/ShellPkg/Library/UefiShellDriver1CommandsLib/Dh.c > > > @@ -130,7 +130,7 @@ IsDriverProt ( > > > > > > @param[in] TheHandle The handles to show info on. > > > @param[in] Language Language string per UEFI specification. > > > - @param[in] Seperator Separator string between information blocks. > > > + @param[in] Separator Separator string between information blocks. > > > @param[in] Verbose TRUE for extra info, FALSE otherwise. > > > @param[in] ExtraInfo TRUE for extra info, FALSE otherwise. > > > > > > @@ -141,7 +141,7 @@ CHAR16* > > > GetProtocolInfoString( > > > IN CONST EFI_HANDLE TheHandle, > > > IN CONST CHAR8 *Language, > > > - IN CONST CHAR16 *Seperator, > > > + IN CONST CHAR16 *Separator, > > > IN CONST BOOLEAN Verbose, > > > IN CONST BOOLEAN ExtraInfo > > > ) > > > @@ -169,7 +169,7 @@ GetProtocolInfoString( > > > if (Temp != NULL) { > > > ASSERT((RetVal == NULL && Size == 0) || (RetVal != NULL)); > > > if (Size != 0) { > > > - StrnCatGrow(&RetVal, &Size, Seperator, 0); > > > + StrnCatGrow(&RetVal, &Size, Separator, 0); > > > } > > > StrnCatGrow(&RetVal, &Size, L"%H", 0); > > > StrnCatGrow(&RetVal, &Size, Temp, 0); @@ -185,7 +185,7 @@ > > > GetProtocolInfoString( > > > StrnCatGrow(&RetVal, &Size, Temp, 0); > > > StrnCatGrow(&RetVal, &Size, L")\r\n", 0); > > > } else { > > > - StrnCatGrow(&RetVal, &Size, Seperator, 0); > > > + StrnCatGrow(&RetVal, &Size, Separator, 0); > > > StrnCatGrow(&RetVal, &Size, Temp, 0); > > > } > > > FreePool(Temp); > > > @@ -201,7 +201,7 @@ GetProtocolInfoString( > > > } > > > > > > ASSERT((RetVal == NULL && Size == 0) || (RetVal != NULL)); > > > - StrnCatGrow(&RetVal, &Size, Seperator, 0); > > > + StrnCatGrow(&RetVal, &Size, Separator, 0); > > > return (RetVal); > > > } > > > > > > diff --git > > > > > a/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Commands > > > Lib.c > > > > > b/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Commands > > > Lib.c > > > index 1491ee9..7948e53 100644 > > > --- > > > > > a/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Commands > > > Lib.c > > > +++ > > > > > b/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Command > > > +++ sLib.c > > > @@ -99,7 +99,7 @@ ShellLevel2CommandsLibConstructor ( > > > ShellCommandRegisterCommandName(L"vol", ShellCommandRunVol > > , > > > ShellCommandGetManFileNameLevel2, 2, L"", TRUE, > > gShellLevel2HiiHandle, > > > STRING_TOKEN(STR_GET_HELP_VOL) ); > > > > > > // > > > - // support for permenant (built in) aliases > > > + // support for permanent (built in) aliases > > > // > > > ShellCommandRegisterAlias(L"rm", L"del"); > > > ShellCommandRegisterAlias(L"ls", L"dir"); diff --git > > > a/ShellPkg/Library/UefiShellNetwork1CommandsLib/Ping.c > > > b/ShellPkg/Library/UefiShellNetwork1CommandsLib/Ping.c > > > index 38625fe..70d9d9a 100644 > > > --- a/ShellPkg/Library/UefiShellNetwork1CommandsLib/Ping.c > > > +++ b/ShellPkg/Library/UefiShellNetwork1CommandsLib/Ping.c > > > @@ -1492,7 +1492,7 @@ ShellCommandRunPing ( > > > } > > > > > > // > > > - // Parse the paramter of count number. > > > + // Parse the parameter of count number. > > > // > > > ValueStr = ShellCommandLineGetValue (ParamPackage, L"-n"); > > > if (ValueStr != NULL) { > > > @@ -1510,7 +1510,7 @@ ShellCommandRunPing ( > > > SendNumber = DEFAULT_SEND_COUNT; > > > } > > > // > > > - // Parse the paramter of buffer size. > > > + // Parse the parameter of buffer size. > > > // > > > ValueStr = ShellCommandLineGetValue (ParamPackage, L"-l"); > > > if (ValueStr != NULL) { > > > @@ -1532,7 +1532,7 @@ ShellCommandRunPing ( > > > ZeroMem (&DstAddress, sizeof (EFI_IPv6_ADDRESS)); > > > > > > // > > > - // Parse the paramter of source ip address. > > > + // Parse the parameter of source ip address. > > > // > > > ValueStr = ShellCommandLineGetValue (ParamPackage, L"-s"); > > > if (ValueStr == NULL) { > > > @@ -1553,7 +1553,7 @@ ShellCommandRunPing ( > > > } > > > } > > > // > > > - // Parse the paramter of destination ip address. > > > + // Parse the parameter of destination ip address. > > > // > > > NonOptionCount = ShellCommandLineGetCount(ParamPackage); > > > if (NonOptionCount < 2) { > > > diff --git a/ShellPkg/Library/UefiShellNetwork2CommandsLib/Ping6.c > > > b/ShellPkg/Library/UefiShellNetwork2CommandsLib/Ping6.c > > > index 90a2604..265258f 100644 > > > --- a/ShellPkg/Library/UefiShellNetwork2CommandsLib/Ping6.c > > > +++ b/ShellPkg/Library/UefiShellNetwork2CommandsLib/Ping6.c > > > @@ -1177,7 +1177,7 @@ ShellCommandRunPing6 ( > > > BufferSize = 16; > > > > > > // > > > - // Parse the paramter of count number. > > > + // Parse the parameter of count number. > > > // > > > ValueStr = ShellCommandLineGetValue (ParamPackage, L"-n"); > > > ValueStrPtr = ValueStr; > > > @@ -1194,7 +1194,7 @@ ShellCommandRunPing6 ( > > > } > > > } > > > // > > > - // Parse the paramter of buffer size. > > > + // Parse the parameter of buffer size. > > > // > > > ValueStr = ShellCommandLineGetValue (ParamPackage, L"-l"); > > > ValueStrPtr = ValueStr; > > > @@ -1215,7 +1215,7 @@ ShellCommandRunPing6 ( > > > ZeroMem (&DstAddress, sizeof (EFI_IPv6_ADDRESS)); > > > > > > // > > > - // Parse the paramter of source ip address. > > > + // Parse the parameter of source ip address. > > > // > > > ValueStr = ShellCommandLineGetValue (ParamPackage, L"-s"); > > > ValueStrPtr = ValueStr; > > > @@ -1229,7 +1229,7 @@ ShellCommandRunPing6 ( > > > } > > > } > > > // > > > - // Parse the paramter of destination ip address. > > > + // Parse the parameter of destination ip address. > > > // > > > NonOptionCount = ShellCommandLineGetCount(ParamPackage); > > > ValueStr = ShellCommandLineGetRawValue (ParamPackage, > > > (UINT32)(NonOptionCount-1)); > > > -- > > > 2.10.1 > >