public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
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



  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