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: Mon, 7 Aug 2017 17:01:39 +0000 [thread overview]
Message-ID: <CS1PR84MB0151A8CD5A7C1777EAE77109EDB50@CS1PR84MB0151.NAMPRD84.PROD.OUTLOOK.COM> (raw)
In-Reply-To: <734D49CCEBEEF84792F5B80ED585239D5B9CE1E7@SHSMSX104.ccr.corp.intel.com>
Ruiyu,
I personally do not have a preference whether it has EFIAPI or not. But the declaration must be consistent with the implementation. When this function is not declared correctly, then the shell "cp" command does not work on GCC DEBUG builds. EFIAPI does carry weight in GCC and even CLANG/LLVM builds so consistency is important.
Does this have to remain an internal function? We could avoid this issue entirely if it were in a public header. It seems quite unusual for the EDK2 source to refer to a function in this manner.
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-07 16:59 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 [this message]
2017-08-09 23:39 ` Palmer, Thomas
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=CS1PR84MB0151A8CD5A7C1777EAE77109EDB50@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