From: "Palmer, Thomas" <thomas.palmer@hpe.com>
To: "Ni, Ruiyu" <ruiyu.ni@intel.com>,
"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>,
"Kinney, Michael D" <michael.d.kinney@intel.com>
Subject: Re: [PATCH 05/11] ShellPkg/UefiShellLevel2CommandsLib: Remove unnecessary EFIAPI
Date: Wed, 9 Aug 2017 23:39:36 +0000 [thread overview]
Message-ID: <CS1PR84MB0151B8CC55E71E765CF4936CED8B0@CS1PR84MB0151.NAMPRD84.PROD.OUTLOOK.COM> (raw)
In-Reply-To: <734D49CCEBEEF84792F5B80ED585239D5B9CE1E7@SHSMSX104.ccr.corp.intel.com>
Ruiyu,
Will you put out the appropriate patch or should I?
Regards,
Thomas Palmer
"I have only made this letter longer because I have not had the time to make it shorter" - Blaise Pascal
-----Original Message-----
From: Ni, Ruiyu [mailto:ruiyu.ni@intel.com]
Sent: Monday, August 7, 2017 12:57 AM
To: Palmer, Thomas <thomas.palmer@hpe.com>; edk2-devel@lists.01.org; Kinney, Michael D <michael.d.kinney@intel.com>
Subject: RE: [edk2] [PATCH 05/11] ShellPkg/UefiShellLevel2CommandsLib: Remove unnecessary EFIAPI
Because an internal function doesn't need to have EFIAPI prefix.
I think a proper fix is to change both BaseLib and UefiShellLevel2CommandsLib to add prefix to the function name.
e.g.: ShellLevel2CommandsLibCharToUpper, BaseLibCharToUpper.
And I am also surprised that BaseLib's version also has the EFIAPI prefix.
---BaseLib/String.c---
CHAR16
EFIAPI
InternalCharToUpper (
IN CHAR16 Char
)
Copying Mike for comments.
Thanks/Ray
> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> Palmer, Thomas
> Sent: Friday, August 4, 2017 3:07 AM
> To: Ni, Ruiyu <ruiyu.ni@intel.com>; edk2-devel@lists.01.org
> Subject: Re: [edk2] [PATCH 05/11] ShellPkg/UefiShellLevel2CommandsLib:
> Remove unnecessary EFIAPI
>
> Ruiyu,
> Sorry for replying to old patch, I was cleaning out some old
> sandboxes when I stumbled onto this issue.
>
> Why is EFIAPI removed from InternalCharToUpper in
> UefiShellLevel2CommandsLib.c? It is present in both BaseLibInternals.h and
> String.c. Without EFIAPI, GCC builds of this function can fail
>
>
> Regards,
>
> Thomas Palmer
>
> "I have only made this letter longer because I have not had the time
> to make it shorter" - Blaise Pascal
>
>
> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> Ruiyu Ni
> Sent: Friday, September 30, 2016 3:18 AM
> To: edk2-devel@lists.01.org
> Cc: Ruiyu Ni <Ruiyu.ni@intel.com>; Jaben Carsey
> <jaben.carsey@intel.com>; Chen A Chen <chen.a.chen@intel.com>
> Subject: [edk2] [PATCH 05/11] ShellPkg/UefiShellLevel2CommandsLib:
> Remove unnecessary EFIAPI
>
> From: Ruiyu Ni <Ruiyu.ni@intel.com>
>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Chen A Chen <chen.a.chen@intel.com>
> Cc: Jaben Carsey <jaben.carsey@intel.com>
> Cc: Ruiyu Ni <Ruiyu.ni@intel.com>
> ---
> ShellPkg/Library/UefiShellLevel2CommandsLib/Cp.c | 4 ----
> ShellPkg/Library/UefiShellLevel2CommandsLib/Load.c | 2 --
> ShellPkg/Library/UefiShellLevel2CommandsLib/Ls.c | 5 -----
> ShellPkg/Library/UefiShellLevel2CommandsLib/Map.c | 12 ------------
> ShellPkg/Library/UefiShellLevel2CommandsLib/Mv.c | 7 -------
> ShellPkg/Library/UefiShellLevel2CommandsLib/Parse.c | 3 ---
> ShellPkg/Library/UefiShellLevel2CommandsLib/Rm.c | 3 ---
> ShellPkg/Library/UefiShellLevel2CommandsLib/Set.c | 1 -
> ShellPkg/Library/UefiShellLevel2CommandsLib/TimeDate.c | 4 ----
> .../UefiShellLevel2CommandsLib/UefiShellLevel2CommandsLib.c | 5 -----
> .../UefiShellLevel2CommandsLib/UefiShellLevel2CommandsLib.h | 6 ------
> ShellPkg/Library/UefiShellLevel2CommandsLib/Vol.c | 1 -
> 12 files changed, 53 deletions(-)
>
> diff --git a/ShellPkg/Library/UefiShellLevel2CommandsLib/Cp.c
> b/ShellPkg/Library/UefiShellLevel2CommandsLib/Cp.c
> index eb1f3b6..b8f6d31 100644
> --- a/ShellPkg/Library/UefiShellLevel2CommandsLib/Cp.c
> +++ b/ShellPkg/Library/UefiShellLevel2CommandsLib/Cp.c
> @@ -35,7 +35,6 @@
> @retval SHELL_OUT_OF_RESOURCES a memory allocation failed
> **/
> SHELL_STATUS
> -EFIAPI
> ValidateAndCopyFiles(
> IN CONST EFI_SHELL_FILE_INFO *FileList,
> IN CONST CHAR16 *DestDir,
> @@ -58,7 +57,6 @@ ValidateAndCopyFiles(
> @retval SHELL_SUCCESS The source file was copied to the destination
> **/
> SHELL_STATUS
> -EFIAPI
> CopySingleFile(
> IN CONST CHAR16 *Source,
> IN CONST CHAR16 *Dest,
> @@ -291,7 +289,6 @@ CopySingleFile(
> @retval SHELL_OUT_OF_RESOURCES a memory allocation failed
> **/
> SHELL_STATUS
> -EFIAPI
> ValidateAndCopyFiles(
> IN CONST EFI_SHELL_FILE_INFO *FileList,
> IN CONST CHAR16 *DestDir,
> @@ -576,7 +573,6 @@ ValidateAndCopyFiles(
> @retval SHELL_SUCCESS The operation was successful.
> **/
> SHELL_STATUS
> -EFIAPI
> ProcessValidateAndCopyFiles(
> IN EFI_SHELL_FILE_INFO *FileList,
> IN CONST CHAR16 *DestDir,
> diff --git a/ShellPkg/Library/UefiShellLevel2CommandsLib/Load.c
> b/ShellPkg/Library/UefiShellLevel2CommandsLib/Load.c
> index ff7c818..322d510 100644
> --- a/ShellPkg/Library/UefiShellLevel2CommandsLib/Load.c
> +++ b/ShellPkg/Library/UefiShellLevel2CommandsLib/Load.c
> @@ -28,7 +28,6 @@
>
> **/
> EFI_STATUS
> -EFIAPI
> ConnectAllEfi (
> VOID
> )
> @@ -74,7 +73,6 @@ ConnectAllEfi (
> @retval EFI_OUT_OF_RESOURCES there was insufficient memory **/
> EFI_STATUS -EFIAPI LoadDriver(
> IN CONST CHAR16 *FileName,
> IN CONST BOOLEAN Connect
> diff --git a/ShellPkg/Library/UefiShellLevel2CommandsLib/Ls.c
> b/ShellPkg/Library/UefiShellLevel2CommandsLib/Ls.c
> index 9b4c452..52ae18f 100644
> --- a/ShellPkg/Library/UefiShellLevel2CommandsLib/Ls.c
> +++ b/ShellPkg/Library/UefiShellLevel2CommandsLib/Ls.c
> @@ -22,7 +22,6 @@
> @param[in] TheList a list of files from the volume.
> **/
> EFI_STATUS
> -EFIAPI
> PrintSfoVolumeInfoTableEntry(
> IN CONST EFI_SHELL_FILE_INFO *TheList
> )
> @@ -152,7 +151,6 @@ PrintSfoVolumeInfoTableEntry(
>
> **/
> VOID
> -EFIAPI
> PrintFileInformation(
> IN CONST BOOLEAN Sfo,
> IN CONST EFI_SHELL_FILE_INFO *TheNode, @@ -263,7 +261,6 @@
> PrintFileInformation(
> @param[in] Path String with starting path.
> **/
> VOID
> -EFIAPI
> PrintNonSfoHeader(
> IN CONST CHAR16 *Path
> )
> @@ -300,7 +297,6 @@ PrintNonSfoHeader(
> @param[in] Dirs The number of directories.
> **/
> VOID
> -EFIAPI
> PrintNonSfoFooter(
> IN UINT64 Files,
> IN UINT64 Size,
> @@ -339,7 +335,6 @@ PrintNonSfoFooter(
> @retval SHELL_SUCCESS the printing was sucessful.
> **/
> SHELL_STATUS
> -EFIAPI
> PrintLsOutput(
> IN CONST BOOLEAN Rec,
> IN CONST UINT64 Attribs,
> diff --git a/ShellPkg/Library/UefiShellLevel2CommandsLib/Map.c
> b/ShellPkg/Library/UefiShellLevel2CommandsLib/Map.c
> index 035aff1..20eb528 100644
> --- a/ShellPkg/Library/UefiShellLevel2CommandsLib/Map.c
> +++ b/ShellPkg/Library/UefiShellLevel2CommandsLib/Map.c
> @@ -34,7 +34,6 @@
> @retval FALSE String has at least one other character.
> **/
> BOOLEAN
> -EFIAPI
> IsNumberLetterOnly(
> IN CONST CHAR16 *String,
> IN CONST UINTN Len
> @@ -65,7 +64,6 @@ IsNumberLetterOnly(
> items (";" normally).
> **/
> BOOLEAN
> -EFIAPI
> SearchList(
> IN CONST CHAR16 *List,
> IN CONST CHAR16 *MetaTarget,
> @@ -133,7 +131,6 @@ SearchList(
> @retval STR_MAP_MEDIA_FLOPPY The media is a floppy drive.
> **/
> CHAR16*
> -EFIAPI
> GetDeviceMediaType (
> IN EFI_DEVICE_PATH_PROTOCOL *DevicePath
> )
> @@ -179,7 +176,6 @@ GetDeviceMediaType (
> @retval FALSE The handle does not have removable storage.
> **/
> BOOLEAN
> -EFIAPI
> IsRemoveableDevice (
> IN EFI_DEVICE_PATH_PROTOCOL *DevicePath
> )
> @@ -216,7 +212,6 @@ IsRemoveableDevice (
> @retval FALSE The map should not be displayed.
> **/
> BOOLEAN
> -EFIAPI
> MappingListHasType(
> IN CONST CHAR16 *MapList,
> IN CONST CHAR16 *Specific,
> @@ -287,7 +282,6 @@ MappingListHasType(
> @retval EFI_SUCCESS The mapping was displayed.
> **/
> EFI_STATUS
> -EFIAPI
> PerformSingleMappingDisplay(
> IN CONST BOOLEAN Verbose,
> IN CONST BOOLEAN Consist,
> @@ -461,7 +455,6 @@ PerformSingleMappingDisplay(
> @retval EFI_NOT_FOUND Name was not a map on Handle.
> **/
> EFI_STATUS
> -EFIAPI
> PerformSingleMappingDelete(
> IN CONST CHAR16 *Specific,
> IN CONST EFI_HANDLE Handle
> @@ -512,7 +505,6 @@ CONST CHAR16 AnyF[] = L"F*";
>
> **/
> SHELL_STATUS
> -EFIAPI
> PerformMappingDisplay(
> IN CONST BOOLEAN Verbose,
> IN CONST BOOLEAN Consist,
> @@ -690,7 +682,6 @@ PerformMappingDisplay(
> @sa PerformMappingDisplay
> **/
> SHELL_STATUS
> -EFIAPI
> PerformMappingDisplay2(
> IN CONST BOOLEAN Verbose,
> IN CONST BOOLEAN Consist,
> @@ -743,7 +734,6 @@ PerformMappingDisplay2(
> @retval EFI_NOT_FOUND Specific could not be found.
> **/
> EFI_STATUS
> -EFIAPI
> PerformMappingDelete(
> IN CONST CHAR16 *Specific
> )
> @@ -874,7 +864,6 @@ PerformMappingDelete(
>
> **/
> SHELL_STATUS
> -EFIAPI
> AddMappingFromMapping(
> IN CONST CHAR16 *Map,
> IN CONST CHAR16 *SName
> @@ -931,7 +920,6 @@ AddMappingFromMapping(
>
> **/
> SHELL_STATUS
> -EFIAPI
> AddMappingFromHandle(
> IN CONST EFI_HANDLE Handle,
> IN CONST CHAR16 *SName
> diff --git a/ShellPkg/Library/UefiShellLevel2CommandsLib/Mv.c
> b/ShellPkg/Library/UefiShellLevel2CommandsLib/Mv.c
> index f93772c..efaaeb2 100644
> --- a/ShellPkg/Library/UefiShellLevel2CommandsLib/Mv.c
> +++ b/ShellPkg/Library/UefiShellLevel2CommandsLib/Mv.c
> @@ -26,7 +26,6 @@
> @retval FALSE The move is within a file system.
> **/
> BOOLEAN
> -EFIAPI
> IsBetweenFileSystem(
> IN CONST CHAR16 *FullName,
> IN CONST CHAR16 *Cwd,
> @@ -79,7 +78,6 @@ IsBetweenFileSystem(
> @retval FALSE The move is not
> **/
> BOOLEAN
> -EFIAPI
> IsValidMove(
> IN CONST CHAR16 *SourcePath,
> IN CONST CHAR16 *Cwd,
> @@ -161,7 +159,6 @@ IsValidMove(
> @retval SHELL_SUCCESS The operation was sucessful.
> **/
> SHELL_STATUS
> -EFIAPI
> GetDestinationLocation(
> IN CONST CHAR16 *DestParameter,
> IN OUT CHAR16 **DestPathPointer,
> @@ -286,7 +283,6 @@ GetDestinationLocation(
> @retval SHELL_SUCCESS The source file was moved to the destination.
> **/
> EFI_STATUS
> -EFIAPI
> MoveBetweenFileSystems(
> IN EFI_SHELL_FILE_INFO *Node,
> IN CONST CHAR16 *DestPath,
> @@ -334,7 +330,6 @@ MoveBetweenFileSystems(
> @retval SHELL_OUT_OF_RESOURCES a memory allocation failed
> **/
> EFI_STATUS
> -EFIAPI
> CreateFullDestPath(
> IN CONST CHAR16 **DestPath,
> OUT CHAR16 **FullDestPath,
> @@ -373,7 +368,6 @@ CreateFullDestPath(
> @retval SHELL_OUT_OF_RESOURCES A memory allocation failed.
> **/
> EFI_STATUS
> -EFIAPI
> MoveWithinFileSystems(
> IN EFI_SHELL_FILE_INFO *Node,
> IN CHAR16 *DestPath,
> @@ -454,7 +448,6 @@ MoveWithinFileSystems(
> @retval SHELL_OUT_OF_RESOURCES a memory allocation failed
> **/
> SHELL_STATUS
> -EFIAPI
> ValidateAndMoveFiles(
> IN EFI_SHELL_FILE_INFO *FileList,
> OUT VOID **Resp,
> diff --git a/ShellPkg/Library/UefiShellLevel2CommandsLib/Parse.c
> b/ShellPkg/Library/UefiShellLevel2CommandsLib/Parse.c
> index 12fe877..4b1973a 100644
> --- a/ShellPkg/Library/UefiShellLevel2CommandsLib/Parse.c
> +++ b/ShellPkg/Library/UefiShellLevel2CommandsLib/Parse.c
> @@ -78,7 +78,6 @@ IsStdInDataAvailable (
> Size was updated to the minimum space required.
> **/
> EFI_STATUS
> -EFIAPI
> ShellFileHandleReadStdInLine(
> IN SHELL_FILE_HANDLE Handle,
> IN OUT CHAR16 *Buffer,
> @@ -160,7 +159,6 @@ ShellFileHandleReadStdInLine(
> @sa ShellFileHandleReadLine
> **/
> CHAR16*
> -EFIAPI
> ParseReturnStdInLine (
> IN SHELL_FILE_HANDLE Handle
> )
> @@ -249,7 +247,6 @@ HandleStringWithEscapeCharForParse (
> @retval SHELL_SUCCESS The operation was successful.
> **/
> SHELL_STATUS
> -EFIAPI
> PerformParsing(
> IN CONST CHAR16 *FileName,
> IN CONST CHAR16 *TableName,
> diff --git a/ShellPkg/Library/UefiShellLevel2CommandsLib/Rm.c
> b/ShellPkg/Library/UefiShellLevel2CommandsLib/Rm.c
> index 0b23fba..618610d 100644
> --- a/ShellPkg/Library/UefiShellLevel2CommandsLib/Rm.c
> +++ b/ShellPkg/Library/UefiShellLevel2CommandsLib/Rm.c
> @@ -29,7 +29,6 @@ STATIC CONST SHELL_PARAM_ITEM ParamList[] = {
> @retval FALSE The directory has at least 1 file or directory in it.
> **/
> BOOLEAN
> -EFIAPI
> IsDirectoryEmpty (
> IN EFI_HANDLE FileHandle
> )
> @@ -66,7 +65,6 @@ IsDirectoryEmpty (
> @retval SHELL_DEVICE_ERROR A device error occured reading this Node.
> **/
> SHELL_STATUS
> -EFIAPI
> CascadeDelete(
> IN EFI_SHELL_FILE_INFO *Node,
> IN CONST BOOLEAN Quiet
> @@ -195,7 +193,6 @@ CascadeDelete(
> @param[in] Package RESERVED. Not used.
> **/
> BOOLEAN
> -EFIAPI
> IsValidDeleteTarget(
> IN CONST EFI_SHELL_FILE_INFO *List,
> IN CONST EFI_SHELL_FILE_INFO *Node, diff --git
> a/ShellPkg/Library/UefiShellLevel2CommandsLib/Set.c
> b/ShellPkg/Library/UefiShellLevel2CommandsLib/Set.c
> index d5e6a08..7ca1942 100644
> --- a/ShellPkg/Library/UefiShellLevel2CommandsLib/Set.c
> +++ b/ShellPkg/Library/UefiShellLevel2CommandsLib/Set.c
> @@ -24,7 +24,6 @@
> @return any return code from GetNextVariableName except
> EFI_NOT_FOUND **/ SHELL_STATUS -EFIAPI PrintAllShellEnvVars(
> VOID
> )
> diff --git a/ShellPkg/Library/UefiShellLevel2CommandsLib/TimeDate.c
> b/ShellPkg/Library/UefiShellLevel2CommandsLib/TimeDate.c
> index 533519d..3ebc72a 100644
> --- a/ShellPkg/Library/UefiShellLevel2CommandsLib/TimeDate.c
> +++ b/ShellPkg/Library/UefiShellLevel2CommandsLib/TimeDate.c
> @@ -28,7 +28,6 @@
> @retval FALSE String is invalid.
> **/
> BOOLEAN
> -EFIAPI
> InternalIsTimeLikeString (
> IN CONST CHAR16 *String,
> IN CONST CHAR16 Char,
> @@ -87,7 +86,6 @@ InternalIsTimeLikeString (
> @retval SHELL_SUCCESS The operation was successful.
> **/
> SHELL_STATUS
> -EFIAPI
> CheckAndSetDate (
> IN CONST CHAR16 *DateString
> )
> @@ -301,7 +299,6 @@ STATIC CONST SHELL_PARAM_ITEM TimeParamList3[] = {
> @retval SHELL_SUCCESS The operation was successful.
> **/
> SHELL_STATUS
> -EFIAPI
> CheckAndSetTime (
> IN CONST CHAR16 *TimeString,
> IN CONST INT16 Tz,
> @@ -701,7 +698,6 @@ STATIC CONST SHELL_PARAM_ITEM TimeZoneParamList3[]
> = {
> @retval SHELL_SUCCESS The operation was successful.
> **/
> SHELL_STATUS
> -EFIAPI
> CheckAndSetTimeZone (
> IN CONST CHAR16 *TimeZoneString
> )
> diff --git
> a/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Commands
> Lib.c
> b/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Commands
> Lib.c
> index 0dafb19..1491ee9 100644
> ---
> a/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Commands
> Lib.c
> +++
> b/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Command
> +++ sLib.c
> @@ -169,7 +169,6 @@ ShellLevel2CommandsLibDestructor (
> @retval other An allocated pointer to a fuly qualified path.
> **/
> CHAR16*
> -EFIAPI
> GetFullyQualifiedPath(
> IN CONST CHAR16* Path
> )
> @@ -216,7 +215,6 @@ GetFullyQualifiedPath(
> @retval EFI_SUCCESS The operation was successful.
> **/
> EFI_STATUS
> -EFIAPI
> VerifyIntermediateDirectories (
> IN CONST CHAR16 *Path
> )
> @@ -270,7 +268,6 @@ VerifyIntermediateDirectories (
> @return Char as an upper case character.
> **/
> CHAR16
> -EFIAPI
> InternalCharToUpper (
> IN CONST CHAR16 Char
> );
> @@ -286,7 +283,6 @@ InternalCharToUpper (
> @return The location in Source where there is a difference.
> **/
> CONST CHAR16*
> -EFIAPI
> StrniCmp(
> IN CONST CHAR16 *Source,
> IN CONST CHAR16 *Target,
> @@ -322,7 +318,6 @@ StrniCmp(
> @retval EFI_SUCCESS The operation was successful.
> **/
> EFI_STATUS
> -EFIAPI
> ShellLevel2StripQuotes (
> IN CONST CHAR16 *OriginalString,
> OUT CHAR16 **CleanString
> diff --git
> a/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Commands
> Lib.h
> b/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Commands
> Lib.h
> index 634515e..19e46a1 100644
> ---
> a/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Commands
> Lib.h
> +++
> b/ShellPkg/Library/UefiShellLevel2CommandsLib/UefiShellLevel2Command
> +++ sLib.h
> @@ -263,7 +263,6 @@ ShellCommandRunMv (
> @retval other pointer to a fuly qualified path.
> **/
> CHAR16*
> -EFIAPI
> GetFullyQualifiedPath(
> IN CONST CHAR16* Path
> );
> @@ -276,7 +275,6 @@ GetFullyQualifiedPath(
> @retval EFI_SUCCESS The operation was successful.
> **/
> EFI_STATUS
> -EFIAPI
> VerifyIntermediateDirectories (
> IN CONST CHAR16 *Path
> );
> @@ -292,7 +290,6 @@ VerifyIntermediateDirectories (
> @return non-zero if the strings are different.
> **/
> CONST CHAR16*
> -EFIAPI
> StrniCmp(
> IN CONST CHAR16 *Source,
> IN CONST CHAR16 *Target,
> @@ -310,7 +307,6 @@ StrniCmp(
> @retval EFI_SUCCESS The operation was successful.
> **/
> EFI_STATUS
> -EFIAPI
> ShellLevel2StripQuotes (
> IN CONST CHAR16 *OriginalString,
> OUT CHAR16 **CleanString
> @@ -343,7 +339,6 @@ ShellCommandRunVol (
> @retval SHELL_SUCCESS The source file was copied to the destination
> **/
> SHELL_STATUS
> -EFIAPI
> CopySingleFile(
> IN CONST CHAR16 *Source,
> IN CONST CHAR16 *Dest,
> @@ -364,7 +359,6 @@ CopySingleFile(
> @retval SHELL_DEVICE_ERROR A device error occured reading this Node.
> **/
> SHELL_STATUS
> -EFIAPI
> CascadeDelete(
> IN EFI_SHELL_FILE_INFO *Node,
> IN CONST BOOLEAN Quiet
> diff --git a/ShellPkg/Library/UefiShellLevel2CommandsLib/Vol.c
> b/ShellPkg/Library/UefiShellLevel2CommandsLib/Vol.c
> index a6f0296..f911c7e 100644
> --- a/ShellPkg/Library/UefiShellLevel2CommandsLib/Vol.c
> +++ b/ShellPkg/Library/UefiShellLevel2CommandsLib/Vol.c
> @@ -27,7 +27,6 @@
> @retval SHELL_SUCCESS The operation was sucessful.
> **/
> SHELL_STATUS
> -EFIAPI
> HandleVol(
> IN CONST CHAR16 *Path,
> IN CONST BOOLEAN Delete,
> --
> 2.9.0.windows.1
>
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
next prev parent reply other threads:[~2017-08-09 23:37 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-30 8:17 [PATCH 00/11] Remove unnecessary EFIAPI Ruiyu Ni
2016-09-30 8:17 ` [PATCH 01/11] ShellPkg/Shell: " Ruiyu Ni
2016-09-30 8:17 ` [PATCH 02/11] ShellPkg/UefiShellDebug1CommandsLib: " Ruiyu Ni
2016-09-30 8:17 ` [PATCH 03/11] ShellPkg/UefiShellDriver1CommandsLib: " Ruiyu Ni
2016-09-30 8:17 ` [PATCH 04/11] ShellPkg/UefiShellLevel1CommandsLib: " Ruiyu Ni
2016-09-30 8:17 ` [PATCH 05/11] ShellPkg/UefiShellLevel2CommandsLib: " Ruiyu Ni
2017-08-03 19:06 ` Palmer, Thomas
2017-08-07 5:56 ` Ni, Ruiyu
2017-08-07 17:01 ` Palmer, Thomas
2017-08-09 23:39 ` Palmer, Thomas [this message]
2017-08-10 0:41 ` Ni, Ruiyu
2017-08-10 0:59 ` Palmer, Thomas
2016-09-30 8:17 ` [PATCH 06/11] ShellPkg/UefiShellLevel3CommandsLib: " Ruiyu Ni
2016-09-30 8:17 ` [PATCH 07/11] ShellPkg/UefiShellParsingLib: " Ruiyu Ni
2016-09-30 8:17 ` [PATCH 08/11] ShellPkg/UefiShellBcfgCommandLib: " Ruiyu Ni
2016-09-30 8:17 ` [PATCH 09/11] ShellPkg/UefiShellCommandLib: " Ruiyu Ni
2016-09-30 8:17 ` [PATCH 10/11] ShellPkg/UefiShellLib: " Ruiyu Ni
2016-09-30 8:17 ` [PATCH 11/11] ShellPkg/UefiShellNetwork1CommandsLib: " Ruiyu Ni
2016-09-30 15:13 ` [PATCH 00/11] " Carsey, Jaben
2016-09-30 16:02 ` Ni, Ruiyu
2016-10-13 1:31 ` Ni, Ruiyu
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=CS1PR84MB0151B8CC55E71E765CF4936CED8B0@CS1PR84MB0151.NAMPRD84.PROD.OUTLOOK.COM \
--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