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: Thu, 10 Aug 2017 00:59:23 +0000 [thread overview]
Message-ID: <CS1PR84MB0151986D04CD736B68AA509DED880@CS1PR84MB0151.NAMPRD84.PROD.OUTLOOK.COM> (raw)
In-Reply-To: <734D49CCEBEEF84792F5B80ED585239D5B9E5CDF@SHSMSX104.ccr.corp.intel.com>
It is not urgent for me.
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: Wednesday, August 9, 2017 7:42 PM
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
I've submitted a tracker for this:
https://bugzilla.tianocore.org/show_bug.cgi?id=664
It's in my to-do list. do you need the fix urgent?
Regards,
Ray
>-----Original Message-----
>From: Palmer, Thomas [mailto:thomas.palmer@hpe.com]
>Sent: Thursday, August 10, 2017 7:40 AM
>To: Ni, Ruiyu <ruiyu.ni@intel.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
>
>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-10 0:57 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
2017-08-10 0:41 ` Ni, Ruiyu
2017-08-10 0:59 ` Palmer, Thomas [this message]
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=CS1PR84MB0151986D04CD736B68AA509DED880@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