Remove code wrapped by DISABLE_NEW_DEPRECATED_INTERFACES

[edk2-devel] [PATCH 0/2] Remove code wrapped by DISABLE_NEW_DEPRECATED_INTERFACES

Posted by Zhang, Shenglei 3 years, 8 months ago

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2777
Code wrapped by DISABLE_NEW_DEPRECATED_INTERFACES is deprecated.
So remove it from MdePkg and MdeModulePkg.

Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Hao A Wu <hao.a.wu@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Shenglei Zhang (2):
  MdePkg: Remove code wrapped by DISABLE_NEW_DEPRECATED_INTERFACES
  MdeModulePkg: Remove code wrapped by DISABLE_NEW_DEPRECATED_INTERFACES

 .../DxePrintLibPrint2Protocol/PrintLib.c      | 145 ----
 MdeModulePkg/Universal/PrintDxe/Print.c       |  37 +-
 MdePkg/Library/BaseLib/String.c               | 626 ------------------
 MdePkg/Library/BasePcdLibNull/PcdLib.c        | 361 ----------
 MdePkg/Library/BasePrintLib/PrintLib.c        | 118 ----
 MdePkg/Library/DxePcdLib/DxePcdLib.c          | 399 -----------
 MdePkg/Library/PeiPcdLib/PeiPcdLib.c          | 397 -----------
 MdePkg/Library/UefiLib/UefiLib.c              |  92 ---
 MdeModulePkg/MdeModulePkg.dsc                 |   1 -
 MdePkg/Include/Library/BaseLib.h              | 409 ------------
 MdePkg/Include/Library/PcdLib.h               | 520 ---------------
 MdePkg/Include/Library/PrintLib.h             | 110 ---
 MdePkg/Include/Library/UefiLib.h              |  53 --
 MdePkg/MdePkg.dsc                             |   1 -
 14 files changed, 6 insertions(+), 3263 deletions(-)

-- 
2.18.0.windows.1


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#63717): https://edk2.groups.io/g/devel/message/63717
Mute This Topic: https://groups.io/mt/76002480/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH 0/2] Remove code wrapped by DISABLE_NEW_DEPRECATED_INTERFACES

Posted by Liming Gao 3 years, 8 months ago

Shenglei:
  Thanks for your update. The change is good. Reviewed-by: Liming Gao <liming.gao@intel.com>

  Per Leif change, this patch will be merged after stable tag 202008 is created. 

Thanks
Liming
-----Original Message-----
From: Zhang, Shenglei <shenglei.zhang@intel.com> 
Sent: 2020年8月5日 14:08
To: devel@edk2.groups.io
Cc: Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A <hao.a.wu@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming <liming.gao@intel.com>
Subject: [PATCH 0/2] Remove code wrapped by DISABLE_NEW_DEPRECATED_INTERFACES

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2777
Code wrapped by DISABLE_NEW_DEPRECATED_INTERFACES is deprecated.
So remove it from MdePkg and MdeModulePkg.

Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Hao A Wu <hao.a.wu@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Shenglei Zhang (2):
  MdePkg: Remove code wrapped by DISABLE_NEW_DEPRECATED_INTERFACES
  MdeModulePkg: Remove code wrapped by DISABLE_NEW_DEPRECATED_INTERFACES

 .../DxePrintLibPrint2Protocol/PrintLib.c      | 145 ----
 MdeModulePkg/Universal/PrintDxe/Print.c       |  37 +-
 MdePkg/Library/BaseLib/String.c               | 626 ------------------
 MdePkg/Library/BasePcdLibNull/PcdLib.c        | 361 ----------
 MdePkg/Library/BasePrintLib/PrintLib.c        | 118 ----
 MdePkg/Library/DxePcdLib/DxePcdLib.c          | 399 -----------
 MdePkg/Library/PeiPcdLib/PeiPcdLib.c          | 397 -----------
 MdePkg/Library/UefiLib/UefiLib.c              |  92 ---
 MdeModulePkg/MdeModulePkg.dsc                 |   1 -
 MdePkg/Include/Library/BaseLib.h              | 409 ------------
 MdePkg/Include/Library/PcdLib.h               | 520 ---------------
 MdePkg/Include/Library/PrintLib.h             | 110 ---
 MdePkg/Include/Library/UefiLib.h              |  53 --
 MdePkg/MdePkg.dsc                             |   1 -
 14 files changed, 6 insertions(+), 3263 deletions(-)

-- 
2.18.0.windows.1


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#64269): https://edk2.groups.io/g/devel/message/64269
Mute This Topic: https://groups.io/mt/76002480/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH 0/2] Remove code wrapped by DISABLE_NEW_DEPRECATED_INTERFACES

Posted by gaoliming 3 years, 7 months ago

Now, stable tag 202008 is created. I plan to merge this patch set before end of this month.

Thanks
Liming

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#65066): https://edk2.groups.io/g/devel/message/65066
Mute This Topic: https://groups.io/mt/76002480/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

回复: [edk2-devel] [PATCH 0/2] Remove code wrapped by DISABLE_NEW_DEPRECATED_INTERFACES

Posted by gaoliming 3 years, 6 months ago

Hi, all

   I create PR https://github.com/tianocore/edk2/pull/1002 to verify this patch set. It can pass CI. I will merge it tomorrow. 

 

With this change, the deprecated interfaces will not exist. If you meet with the platform build failure, please update the platform code to consume new interface. 

 

Thanks

Liming

发件人: bounce+27952+65066+4905953+8761045@groups.io <bounce+27952+65066+4905953+8761045@groups.io> 代表 gaoliming
发送时间: 2020年9月7日 9:07
收件人: Gao; Liming <liming.gao@intel.com>; devel@edk2.groups.io
主题: Re: [edk2-devel] [PATCH 0/2] Remove code wrapped by DISABLE_NEW_DEPRECATED_INTERFACES

 

Now, stable tag 202008 is created. I plan to merge this patch set before end of this month. 

Thanks
Liming 





-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#66120): https://edk2.groups.io/g/devel/message/66120
Mute This Topic: https://groups.io/mt/77454947/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-