From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) (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 F126F1A1DEB for ; Mon, 12 Sep 2016 01:36:29 -0700 (PDT) Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 8A88E4E4CE; Mon, 12 Sep 2016 08:36:29 +0000 (UTC) Received: from [10.36.116.40] (ovpn-116-40.ams2.redhat.com [10.36.116.40]) by int-mx11.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id u8C8aQmI011253 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Mon, 12 Sep 2016 04:36:28 -0400 To: "Gao, Liming" , "edk2-devel@ml01.01.org" References: <1473453189-22983-1-git-send-email-thuth@redhat.com> <4A89E2EF3DFEDB4C8BFDE51014F606A14B3E6C26@shsmsx102.ccr.corp.intel.com> Cc: "Kinney, Michael D" From: Thomas Huth Message-ID: <5b711a30-5e2c-0833-4e24-a2f0ff59b6a1@redhat.com> Date: Mon, 12 Sep 2016 10:36:21 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2 MIME-Version: 1.0 In-Reply-To: <4A89E2EF3DFEDB4C8BFDE51014F606A14B3E6C26@shsmsx102.ccr.corp.intel.com> X-Scanned-By: MIMEDefang 2.68 on 10.5.11.24 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.38]); Mon, 12 Sep 2016 08:36:29 +0000 (UTC) X-Content-Filtered-By: Mailman/MimeDel 2.1.21 Subject: Re: [PATCH] MdePkg: Fix some typing errors 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: Mon, 12 Sep 2016 08:36:30 -0000 X-Groupsio-MsgNum: 1701 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="Xj1QPTnvAXMU0p9GfxENHQgW2lTvsPQtR" --Xj1QPTnvAXMU0p9GfxENHQgW2lTvsPQtR From: Thomas Huth To: "Gao, Liming" , "edk2-devel@ml01.01.org" Cc: "Kinney, Michael D" Message-ID: <5b711a30-5e2c-0833-4e24-a2f0ff59b6a1@redhat.com> Subject: Re: [PATCH] MdePkg: Fix some typing errors References: <1473453189-22983-1-git-send-email-thuth@redhat.com> <4A89E2EF3DFEDB4C8BFDE51014F606A14B3E6C26@shsmsx102.ccr.corp.intel.com> In-Reply-To: <4A89E2EF3DFEDB4C8BFDE51014F606A14B3E6C26@shsmsx102.ccr.corp.intel.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 12.09.2016 03:26, Gao, Liming wrote: > Thomas: > Thanks for your catching. Some similar issues also exist in header fi= les in MdePkg. Could you help fix them also? Sure, I just send a patch. Regards, Thomas > Thanks > Liming >> -----Original Message----- >> From: Thomas Huth [mailto:thuth@redhat.com] >> Sent: Saturday, September 10, 2016 4:33 AM >> To: edk2-devel@ml01.01.org >> Cc: Kinney, Michael D ; Gao, Liming >> >> Subject: [PATCH] MdePkg: Fix some typing errors >> >> Correct the typos in some files of MdePkg. >> >> Contributed-under: TianoCore Contribution Agreement 1.0 >> Signed-off-by: Thomas Huth >> --- >> .../BaseExtractGuidedSectionLib.c | 6 +++--- >> MdePkg/Library/BaseLib/String.c | 2 +- >> MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c | 20 ++++++++++-= --- >> ------ >> MdePkg/Library/UefiUsbLib/UsbDxeLib.c | 2 +- >> 4 files changed, 15 insertions(+), 15 deletions(-) >> >> diff --git >> a/MdePkg/Library/BaseExtractGuidedSectionLib/BaseExtractGuidedSectionL= >> ib.c >> b/MdePkg/Library/BaseExtractGuidedSectionLib/BaseExtractGuidedSectionL= >> ib.c >> index 3a12cb1..28996f3 100644 >> --- >> a/MdePkg/Library/BaseExtractGuidedSectionLib/BaseExtractGuidedSectionL= >> ib.c >> +++ >> b/MdePkg/Library/BaseExtractGuidedSectionLib/BaseExtractGuidedSectionL= >> ib.c >> @@ -175,7 +175,7 @@ ExtractGuidedSectionRegisterHandlers ( >> EXTRACT_GUIDED_SECTION_HANDLER_INFO *HandlerInfo; >> >> // >> - // Check input paramter >> + // Check input parameter >> // >> ASSERT (SectionGuid !=3D NULL); >> ASSERT (GetInfoHandler !=3D NULL); >> @@ -270,7 +270,7 @@ ExtractGuidedSectionGetInfo ( >> EFI_GUID *SectionDefinitionGuid; >> >> // >> - // Check input paramter >> + // Check input parameter >> // >> ASSERT (InputSection !=3D NULL); >> ASSERT (OutputBufferSize !=3D NULL); >> @@ -449,7 +449,7 @@ ExtractGuidedSectionGetHandlers ( >> EXTRACT_GUIDED_SECTION_HANDLER_INFO *HandlerInfo; >> >> // >> - // Check input paramter >> + // Check input parameter >> // >> ASSERT (SectionGuid !=3D NULL); >> >> diff --git a/MdePkg/Library/BaseLib/String.c >> b/MdePkg/Library/BaseLib/String.c >> index 07c0562..25962f8 100644 >> --- a/MdePkg/Library/BaseLib/String.c >> +++ b/MdePkg/Library/BaseLib/String.c >> @@ -1,5 +1,5 @@ >> /** @file >> - Unicode and ASCII string primatives. >> + Unicode and ASCII string primitives. >> >> Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<= BR> >> This program and the accompanying materials >> diff --git a/MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c >> b/MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c >> index 8f3cde0..daed0f4 100644 >> --- a/MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c >> +++ b/MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c >> @@ -231,7 +231,7 @@ FileHandleWrite( >> >> @param FileHandle the file handle to close. >> >> -@retval EFI_SUCCESS the file handle was closed sucessfull= y. >> +@retval EFI_SUCCESS the file handle was closed successful= ly. >> **/ >> EFI_STATUS >> EFIAPI >> @@ -261,7 +261,7 @@ FileHandleClose ( >> >> @param FileHandle the file handle to delete >> >> - @retval EFI_SUCCESS the file was closed sucessfully >> + @retval EFI_SUCCESS the file was closed successfully >> @retval EFI_WARN_DELETE_FAILURE the handle was closed, but the file= >> was not >> deleted >> @retval INVALID_PARAMETER One of the parameters has an invalid = value. >> @@ -297,9 +297,9 @@ FileHandleDelete ( >> has the effect of starting the read process of the directory entrie= s over. >> >> @param FileHandle The file handle on which the position= is being set >> - @param Position Byte position from begining of file >> + @param Position Byte position from beginning of file >> >> - @retval EFI_SUCCESS Operation completed sucessfully. >> + @retval EFI_SUCCESS Operation completed successfully. >> @retval EFI_UNSUPPORTED the seek request for non-zero is not = valid >> on >> directories. >> @retval INVALID_PARAMETER One of the parameters has an invalid = value. >> @@ -330,9 +330,9 @@ FileHandleSetPosition ( >> if FileHandle is a directory. >> >> @param FileHandle The open file handle on which to get = the position. >> - @param Position Byte position from begining of file. >> + @param Position Byte position from beginning of file.= >> >> - @retval EFI_SUCCESS the operation completed sucessfully. >> + @retval EFI_SUCCESS the operation completed successfully.= >> @retval INVALID_PARAMETER One of the parameters has an invalid = value. >> @retval EFI_UNSUPPORTED the request is not valid on directori= es. >> **/ >> @@ -489,7 +489,7 @@ FileHandleFindFirstFile ( >> } >> >> // >> - // reset to the begining of the directory >> + // reset to the beginning of the directory >> // >> Status =3D FileHandleSetPosition(DirHandle, 0); >> if (EFI_ERROR(Status)) { >> @@ -581,7 +581,7 @@ FileHandleFindNextFile( >> @param[in] FileHandle The file handle from which size is re= trieved. >> @param[out] Size The pointer to size. >> >> - @retval EFI_SUCCESS Operation was completed sucessfully. >> + @retval EFI_SUCCESS Operation was completed successfully.= >> @retval EFI_DEVICE_ERROR Cannot access the file. >> @retval EFI_INVALID_PARAMETER FileHandle is NULL. >> Size is NULL. >> @@ -821,7 +821,7 @@ FileHandleGetFileName ( >> break; >> } else { >> // >> - // We got info... do we have a name? if yes preceed the curre= nt path >> with it... >> + // We got info... do we have a name? if yes precede the curre= nt path >> with it... >> // >> if (StrLen (FileInfo->FileName) =3D=3D 0) { >> if (*FullFileName =3D=3D NULL) { >> @@ -1181,7 +1181,7 @@ FileHandleWriteLine( >> @param[in] Format the format argument (see printlib for format sp= ecifier) >> @param[in] ... the variable arguments for the format >> >> - @retval EFI_SUCCESS the operation was sucessful >> + @retval EFI_SUCCESS the operation was successful >> @return other a return value from FileHandleWriteLine >> >> @sa FileHandleWriteLine >> diff --git a/MdePkg/Library/UefiUsbLib/UsbDxeLib.c >> b/MdePkg/Library/UefiUsbLib/UsbDxeLib.c >> index b22a829..b1fcbc5 100644 >> --- a/MdePkg/Library/UefiUsbLib/UsbDxeLib.c >> +++ b/MdePkg/Library/UefiUsbLib/UsbDxeLib.c >> @@ -628,7 +628,7 @@ UsbClearEndpointHalt ( >> >> ZeroMem (&EndpointDescriptor, sizeof >> (EFI_USB_ENDPOINT_DESCRIPTOR)); >> // >> - // First seach the endpoint descriptor for that endpoint addr >> + // First search the endpoint descriptor for that endpoint addr >> // >> Result =3D UsbIo->UsbGetInterfaceDescriptor ( >> UsbIo, >> -- >> 1.8.3.1 >=20 --Xj1QPTnvAXMU0p9GfxENHQgW2lTvsPQtR Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (GNU/Linux) iQIcBAEBAgAGBQJX1mkKAAoJEC7Z13T+cC219LIP/2JTn3Wl/154lFRQsxcF/OJx C64obewv88HnSjFbL95qECwKWKp2jes1EbcsCoVtX5Sq3PFXLQgKYtaczMyA0d1Y 879+vDFCu6IoBWBERacxTP7oCqHGNPupA0skf2N6/evvUa3TMlkSS/pFVyWgQc9L zaL++bMAB+z4Gsc36MHLuL+VwE6MH0ioC8oBhc+WZw6l4Pmzz8QezCejKAZ3c2qO nfyV8QbjWmpMZ6r5LjxdHPms4DMTiG7M8exCewvuAaxuDaqM95bjl6bgshqeoWMj XguZlJDc0nOtwfjBmv/GZktThL0bp7TN5UruL7mis1pcauuQtXF+WgUFgEeGR/ph yELJcAhxcv80ZCiDcm2mlkM0UXPmW4BaiXRHicYDllqugCV2Bv4z89GGRkcxxok8 4fn8PmRbUqOmAFDC1yNHVXO3ok2aylq2MN6JJjjg9R+wVU45kYtXu8XcYRo2m5Yp TlcrWvFkelJE5yCXXksQQUh1tiRzOEm2LrQN1FcRvQzlD1Ssyf3flMRGR9HWB2Hq sDrRfKQmtyj0wU/M9WFv97FCnXdPQPFtopdvIgrxNCYEtb+/3Eveenb9aNPzfcvk uEhf4VpXxy3X5WzOiPZLHUtxzxKYrAUNgOQHtPZB0jdbkyNVyH+IG6tK75E+Z1na 6WVF5E/wbIEj9X2/enue =/fTF -----END PGP SIGNATURE----- --Xj1QPTnvAXMU0p9GfxENHQgW2lTvsPQtR--