Jaben, If we want to promote FreeArgList to library API. The structure ARG_LIST will also be in the library header file. Let's wait and see if there is more such usage and abstract it then.
I will post a v2 for those that need update. Regards, Ray >-----Original Message----- >From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Carsey, >Jaben >Sent: Friday, July 15, 2016 5:06 AM >To: Ni, Ruiyu <ruiyu...@intel.com>; edk2-devel@lists.01.org >Cc: Carsey, Jaben <jaben.car...@intel.com> >Subject: Re: [edk2] [PATCH 00/27] ShellPkg: Handle memory allocation failure > >Please check copyright. I notice that at minimum EfiCompress.c and LoadPciRom >need an update. > >I think #8 has an error. Replied to that email separately. > >Is it worth moving "IfConfig6FreeArgList" to a shared library? Looked like >there were 2 copies... > >Otherwise for the series >Reviewed-By: Jaben Carsey <jaben.car...@intel.com> > >> -----Original Message----- >> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of >> Ruiyu Ni >> Sent: Thursday, July 14, 2016 2:30 AM >> To: edk2-devel@lists.01.org >> Subject: [edk2] [PATCH 00/27] ShellPkg: Handle memory allocation failure >> Importance: High >> >> The patch serials remove almost all assertions on memory allocation >> result, replaces with the error handling code. >> >> https://github.com/niruiyu/edk2/commits/shell_assert >> >> Ruiyu Ni (27): >> ShellPkg/Shell.c: Handle memory allocation failure >> ShellPkg/IsVolatileEnv: Handle memory allocation failure >> ShellPkg/ShellAddEnvVarToList: Handle memory allocation failure >> ShellPkg/ShellProtocol.c: Handle memory allocation failure >> ShellPkg/Dp: Handle memory allocation failure >> ShellPkg/UefiHandleParsingLib.c: Handle memory allocation failure >> ShellPkg/UefiShellBcfgCommandLib: Handle memory allocation failure >> ShellPkg/ConsistMapping.c: Handle memory allocation failure >> ShellPkg/UefiShellCommandLib.c: Handle memory allocation failure >> ShellPkg/DMem: Handle memory allocation failure >> ShellPkg/EfiCompress: Handle memory allocation failure >> ShellPkg/EfiDecompress: Handle memory allocation failure >> ShellPkg/LoadPciRom: Handle memory allocation failure >> ShellPkg/Edit: Handle memory allocation failure >> ShellPkg/DevTree: Handle memory allocation failure >> ShellPkg/DrvCfg: Handle memory allocation failure >> ShellPkg/DrvDiag: Handle memory allocation failure >> ShellPkg/For: Handle memory allocation failure >> ShellPkg/If: Handle memory allocation failure >> ShellPkg/Cd: Handle memory allocation failure >> ShellPkg/Cp: Handle memory allocation failure >> ShellPkg/Mv: Handle memory allocation failure >> ShellPkg/UefiShellLib: Handle memory allocation failure >> ShellPkg/IfConfig: Handle memory allocation failure >> ShellPkg/UefiShellLib.c: Handle memory allocation failure >> ShellPkg/Ifconfig6: Handle memory allocation failure >> ShellPkg/Ping6: Handle memory allocation failure >> >> ShellPkg/Application/Shell/FileHandleWrappers.c | 19 +- >> ShellPkg/Application/Shell/Shell.c | 27 +- >> ShellPkg/Application/Shell/ShellEnvVar.c | 73 +-- >> ShellPkg/Application/Shell/ShellEnvVar.h | 22 +- >> .../Application/Shell/ShellParametersProtocol.c | 5 +- >> ShellPkg/Application/Shell/ShellProtocol.c | 142 +++--- >> ShellPkg/Application/Shell/ShellProtocol.h | 17 +- >> ShellPkg/Library/UefiDpLib/Dp.c | 8 +- >> .../UefiHandleParsingLib/UefiHandleParsingLib.c | 55 ++- >> .../UefiShellBcfgCommandLib.c | 47 +- >> .../Library/UefiShellCommandLib/ConsistMapping.c | 497 +++++++++++++- >> ------- >> .../UefiShellCommandLib/UefiShellCommandLib.c | 34 +- >> ShellPkg/Library/UefiShellDebug1CommandsLib/Dmem.c | 6 +- >> .../UefiShellDebug1CommandsLib/Edit/FileBuffer.c | 41 +- >> .../UefiShellDebug1CommandsLib/EfiCompress.c | 28 +- >> .../UefiShellDebug1CommandsLib/EfiDecompress.c | 52 ++- >> .../UefiShellDebug1CommandsLib/LoadPciRom.c | 6 +- >> .../Library/UefiShellDriver1CommandsLib/DevTree.c | 13 +- >> .../Library/UefiShellDriver1CommandsLib/DrvCfg.c | 17 +- >> .../Library/UefiShellDriver1CommandsLib/DrvDiag.c | 17 +- >> ShellPkg/Library/UefiShellLevel1CommandsLib/For.c | 12 +- >> ShellPkg/Library/UefiShellLevel1CommandsLib/If.c | 95 ++-- >> ShellPkg/Library/UefiShellLevel2CommandsLib/Cd.c | 34 +- >> ShellPkg/Library/UefiShellLevel2CommandsLib/Cp.c | 19 +- >> ShellPkg/Library/UefiShellLevel2CommandsLib/Mv.c | 14 +- >> ShellPkg/Library/UefiShellLib/UefiShellLib.c | 23 +- >> .../UefiShellNetwork1CommandsLib/Ifconfig.c | 77 +++- >> .../UefiShellNetwork1CommandsLib.uni | 1 + >> .../UefiShellNetwork2CommandsLib/Ifconfig6.c | 75 +++- >> .../Library/UefiShellNetwork2CommandsLib/Ping6.c | 6 +- >> .../UefiShellNetwork2CommandsLib.uni | 2 + >> 31 files changed, 954 insertions(+), 530 deletions(-) >> >> -- >> 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 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel