* [PATCH 0/3] Remove ImageDecoderLib and BmpImageDecoderLib
@ 2016-09-29 2:41 Ruiyu Ni
2016-09-29 2:48 ` Gao, Liming
0 siblings, 1 reply; 7+ messages in thread
From: Ruiyu Ni @ 2016-09-29 2:41 UTC (permalink / raw)
To: edk2-devel
The two libraries are created to support old BootLogoLib to
decode the images. Due to the new BootLogoLib starts using
HiiImageEx interfaces, the two libraries are not needed.
Ruiyu Ni (3):
MdeModulePkg/ImageDecoderLib: Retire it due to new BootLogoLib
MdeModulePkg/BmpImageDecoderLib: Retire it due to new BootLogoLib
MdeModulePkg: Remove ImageDecoderLib and BmpImageDecoderLib
.../BmpImageDecoderLib/BmpImageDecoderLib.c | 272 ---------------------
.../BmpImageDecoderLib/BmpImageDecoderLib.inf | 43 ----
.../BmpImageDecoderLib/BmpImageDecoderLib.uni | 26 --
.../Library/ImageDecoderLib/ImageDecoderLib.c | 121 ---------
.../Library/ImageDecoderLib/ImageDecoderLib.inf | 43 ----
.../Library/ImageDecoderLib/ImageDecoderLib.uni | 26 --
MdeModulePkg/MdeModulePkg.dec | 4 -
MdeModulePkg/MdeModulePkg.dsc | 3 -
8 files changed, 538 deletions(-)
delete mode 100644 MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.c
delete mode 100644 MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.inf
delete mode 100644 MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.uni
delete mode 100644 MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.c
delete mode 100644 MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.inf
delete mode 100644 MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.uni
--
2.9.0.windows.1
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 0/3] Remove ImageDecoderLib and BmpImageDecoderLib
2016-09-29 2:41 Ruiyu Ni
@ 2016-09-29 2:48 ` Gao, Liming
2016-09-29 2:52 ` Ni, Ruiyu
0 siblings, 1 reply; 7+ messages in thread
From: Gao, Liming @ 2016-09-29 2:48 UTC (permalink / raw)
To: Ni, Ruiyu, edk2-devel@lists.01.org
Do you remove Include/Library/ImageDecoderLib.h?
> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> Ruiyu Ni
> Sent: Thursday, September 29, 2016 10:42 AM
> To: edk2-devel@lists.01.org
> Subject: [edk2] [PATCH 0/3] Remove ImageDecoderLib and
> BmpImageDecoderLib
>
> The two libraries are created to support old BootLogoLib to
> decode the images. Due to the new BootLogoLib starts using
> HiiImageEx interfaces, the two libraries are not needed.
>
> Ruiyu Ni (3):
> MdeModulePkg/ImageDecoderLib: Retire it due to new BootLogoLib
> MdeModulePkg/BmpImageDecoderLib: Retire it due to new BootLogoLib
> MdeModulePkg: Remove ImageDecoderLib and BmpImageDecoderLib
>
> .../BmpImageDecoderLib/BmpImageDecoderLib.c | 272 ---------------------
> .../BmpImageDecoderLib/BmpImageDecoderLib.inf | 43 ----
> .../BmpImageDecoderLib/BmpImageDecoderLib.uni | 26 --
> .../Library/ImageDecoderLib/ImageDecoderLib.c | 121 ---------
> .../Library/ImageDecoderLib/ImageDecoderLib.inf | 43 ----
> .../Library/ImageDecoderLib/ImageDecoderLib.uni | 26 --
> MdeModulePkg/MdeModulePkg.dec | 4 -
> MdeModulePkg/MdeModulePkg.dsc | 3 -
> 8 files changed, 538 deletions(-)
> delete mode 100644
> MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.c
> delete mode 100644
> MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.inf
> delete mode 100644
> MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.uni
> delete mode 100644
> MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.c
> delete mode 100644
> MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.inf
> delete mode 100644
> MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.uni
>
> --
> 2.9.0.windows.1
>
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 0/3] Remove ImageDecoderLib and BmpImageDecoderLib
2016-09-29 2:48 ` Gao, Liming
@ 2016-09-29 2:52 ` Ni, Ruiyu
0 siblings, 0 replies; 7+ messages in thread
From: Ni, Ruiyu @ 2016-09-29 2:52 UTC (permalink / raw)
To: Gao, Liming, edk2-devel@lists.01.org
Yes! Thanks for the reminding. V2 will be posted.
Regards,
Ray
From: Gao, Liming
Sent: Thursday, September 29, 2016 10:49 AM
To: Ni, Ruiyu <ruiyu.ni@intel.com>; edk2-devel@lists.01.org
Subject: RE: [edk2] [PATCH 0/3] Remove ImageDecoderLib and BmpImageDecoderLib
Do you remove Include/Library/ImageDecoderLib.h?
> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> Ruiyu Ni
> Sent: Thursday, September 29, 2016 10:42 AM
> To: edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org>
> Subject: [edk2] [PATCH 0/3] Remove ImageDecoderLib and
> BmpImageDecoderLib
>
> The two libraries are created to support old BootLogoLib to
> decode the images. Due to the new BootLogoLib starts using
> HiiImageEx interfaces, the two libraries are not needed.
>
> Ruiyu Ni (3):
> MdeModulePkg/ImageDecoderLib: Retire it due to new BootLogoLib
> MdeModulePkg/BmpImageDecoderLib: Retire it due to new BootLogoLib
> MdeModulePkg: Remove ImageDecoderLib and BmpImageDecoderLib
>
> .../BmpImageDecoderLib/BmpImageDecoderLib.c | 272 ---------------------
> .../BmpImageDecoderLib/BmpImageDecoderLib.inf | 43 ----
> .../BmpImageDecoderLib/BmpImageDecoderLib.uni | 26 --
> .../Library/ImageDecoderLib/ImageDecoderLib.c | 121 ---------
> .../Library/ImageDecoderLib/ImageDecoderLib.inf | 43 ----
> .../Library/ImageDecoderLib/ImageDecoderLib.uni | 26 --
> MdeModulePkg/MdeModulePkg.dec | 4 -
> MdeModulePkg/MdeModulePkg.dsc | 3 -
> 8 files changed, 538 deletions(-)
> delete mode 100644
> MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.c
> delete mode 100644
> MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.inf
> delete mode 100644
> MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.uni
> delete mode 100644
> MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.c
> delete mode 100644
> MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.inf
> delete mode 100644
> MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.uni
>
> --
> 2.9.0.windows.1
>
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org>
> https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 0/3] Remove ImageDecoderLib and BmpImageDecoderLib
@ 2016-09-29 3:28 Ruiyu Ni
0 siblings, 0 replies; 7+ messages in thread
From: Ruiyu Ni @ 2016-09-29 3:28 UTC (permalink / raw)
To: edk2-devel
The two libraries are created to support old BootLogoLib to
decode the images. Due to the new BootLogoLib starts using
HiiImageEx interfaces, the two libraries are not needed.
Ruiyu Ni (3):
MdeModulePkg/ImageDecoderLib: Retire it due to new BootLogoLib
MdeModulePkg/BmpImageDecoderLib: Retire it due to new BootLogoLib
MdeModulePkg: Remove ImageDecoderLib and BmpImageDecoderLib
.../BmpImageDecoderLib/BmpImageDecoderLib.c | 272 ---------------------
.../BmpImageDecoderLib/BmpImageDecoderLib.inf | 43 ----
.../BmpImageDecoderLib/BmpImageDecoderLib.uni | 26 --
.../Library/ImageDecoderLib/ImageDecoderLib.c | 121 ---------
.../Library/ImageDecoderLib/ImageDecoderLib.inf | 43 ----
.../Library/ImageDecoderLib/ImageDecoderLib.uni | 26 --
MdeModulePkg/MdeModulePkg.dec | 4 -
MdeModulePkg/MdeModulePkg.dsc | 3 -
8 files changed, 538 deletions(-)
delete mode 100644 MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.c
delete mode 100644 MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.inf
delete mode 100644 MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.uni
delete mode 100644 MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.c
delete mode 100644 MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.inf
delete mode 100644 MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.uni
--
2.9.0.windows.1
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 0/3] Remove ImageDecoderLib and BmpImageDecoderLib
@ 2016-09-29 3:29 Ruiyu Ni
2016-09-29 3:50 ` Gao, Liming
0 siblings, 1 reply; 7+ messages in thread
From: Ruiyu Ni @ 2016-09-29 3:29 UTC (permalink / raw)
To: edk2-devel
The two libraries are created to support old BootLogoLib to
decode the images. Due to the new BootLogoLib starts using
HiiImageEx interfaces, the two libraries are not needed.
Ruiyu Ni (3):
MdeModulePkg/ImageDecoderLib: Retire it due to new BootLogoLib
MdeModulePkg/BmpImageDecoderLib: Retire it due to new BootLogoLib
MdeModulePkg: Remove ImageDecoderLib and BmpImageDecoderLib
.../BmpImageDecoderLib/BmpImageDecoderLib.c | 272 ---------------------
.../BmpImageDecoderLib/BmpImageDecoderLib.inf | 43 ----
.../BmpImageDecoderLib/BmpImageDecoderLib.uni | 26 --
.../Library/ImageDecoderLib/ImageDecoderLib.c | 121 ---------
.../Library/ImageDecoderLib/ImageDecoderLib.inf | 43 ----
.../Library/ImageDecoderLib/ImageDecoderLib.uni | 26 --
MdeModulePkg/MdeModulePkg.dec | 4 -
MdeModulePkg/MdeModulePkg.dsc | 3 -
8 files changed, 538 deletions(-)
delete mode 100644 MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.c
delete mode 100644 MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.inf
delete mode 100644 MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.uni
delete mode 100644 MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.c
delete mode 100644 MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.inf
delete mode 100644 MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.uni
--
2.9.0.windows.1
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 0/3] Remove ImageDecoderLib and BmpImageDecoderLib
2016-09-29 3:29 Ruiyu Ni
@ 2016-09-29 3:50 ` Gao, Liming
2016-09-29 6:17 ` Ni, Ruiyu
0 siblings, 1 reply; 7+ messages in thread
From: Gao, Liming @ 2016-09-29 3:50 UTC (permalink / raw)
To: Ni, Ruiyu, edk2-devel@lists.01.org
Your patch has updated. But, this covert letter is not.
Reviewed-by: Liming Gao <liming.gao@intel.com>
> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> Ruiyu Ni
> Sent: Thursday, September 29, 2016 11:29 AM
> To: edk2-devel@lists.01.org
> Subject: [edk2] [PATCH 0/3] Remove ImageDecoderLib and
> BmpImageDecoderLib
>
> The two libraries are created to support old BootLogoLib to
> decode the images. Due to the new BootLogoLib starts using
> HiiImageEx interfaces, the two libraries are not needed.
>
> Ruiyu Ni (3):
> MdeModulePkg/ImageDecoderLib: Retire it due to new BootLogoLib
> MdeModulePkg/BmpImageDecoderLib: Retire it due to new BootLogoLib
> MdeModulePkg: Remove ImageDecoderLib and BmpImageDecoderLib
>
> .../BmpImageDecoderLib/BmpImageDecoderLib.c | 272 ---------------------
> .../BmpImageDecoderLib/BmpImageDecoderLib.inf | 43 ----
> .../BmpImageDecoderLib/BmpImageDecoderLib.uni | 26 --
> .../Library/ImageDecoderLib/ImageDecoderLib.c | 121 ---------
> .../Library/ImageDecoderLib/ImageDecoderLib.inf | 43 ----
> .../Library/ImageDecoderLib/ImageDecoderLib.uni | 26 --
> MdeModulePkg/MdeModulePkg.dec | 4 -
> MdeModulePkg/MdeModulePkg.dsc | 3 -
> 8 files changed, 538 deletions(-)
> delete mode 100644
> MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.c
> delete mode 100644
> MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.inf
> delete mode 100644
> MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.uni
> delete mode 100644
> MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.c
> delete mode 100644
> MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.inf
> delete mode 100644
> MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.uni
>
> --
> 2.9.0.windows.1
>
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 0/3] Remove ImageDecoderLib and BmpImageDecoderLib
2016-09-29 3:50 ` Gao, Liming
@ 2016-09-29 6:17 ` Ni, Ruiyu
0 siblings, 0 replies; 7+ messages in thread
From: Ni, Ruiyu @ 2016-09-29 6:17 UTC (permalink / raw)
To: Gao, Liming, edk2-devel@lists.01.org
I re-used the old cover letter:)
Regards,
Ray
From: Gao, Liming
Sent: Thursday, September 29, 2016 11:51 AM
To: Ni, Ruiyu <ruiyu.ni@intel.com>; edk2-devel@lists.01.org
Subject: RE: [edk2] [PATCH 0/3] Remove ImageDecoderLib and BmpImageDecoderLib
Your patch has updated. But, this covert letter is not.
Reviewed-by: Liming Gao <liming.gao@intel.com<mailto:liming.gao@intel.com>>
> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> Ruiyu Ni
> Sent: Thursday, September 29, 2016 11:29 AM
> To: edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org>
> Subject: [edk2] [PATCH 0/3] Remove ImageDecoderLib and
> BmpImageDecoderLib
>
> The two libraries are created to support old BootLogoLib to
> decode the images. Due to the new BootLogoLib starts using
> HiiImageEx interfaces, the two libraries are not needed.
>
> Ruiyu Ni (3):
> MdeModulePkg/ImageDecoderLib: Retire it due to new BootLogoLib
> MdeModulePkg/BmpImageDecoderLib: Retire it due to new BootLogoLib
> MdeModulePkg: Remove ImageDecoderLib and BmpImageDecoderLib
>
> .../BmpImageDecoderLib/BmpImageDecoderLib.c | 272 ---------------------
> .../BmpImageDecoderLib/BmpImageDecoderLib.inf | 43 ----
> .../BmpImageDecoderLib/BmpImageDecoderLib.uni | 26 --
> .../Library/ImageDecoderLib/ImageDecoderLib.c | 121 ---------
> .../Library/ImageDecoderLib/ImageDecoderLib.inf | 43 ----
> .../Library/ImageDecoderLib/ImageDecoderLib.uni | 26 --
> MdeModulePkg/MdeModulePkg.dec | 4 -
> MdeModulePkg/MdeModulePkg.dsc | 3 -
> 8 files changed, 538 deletions(-)
> delete mode 100644
> MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.c
> delete mode 100644
> MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.inf
> delete mode 100644
> MdeModulePkg/Library/BmpImageDecoderLib/BmpImageDecoderLib.uni
> delete mode 100644
> MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.c
> delete mode 100644
> MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.inf
> delete mode 100644
> MdeModulePkg/Library/ImageDecoderLib/ImageDecoderLib.uni
>
> --
> 2.9.0.windows.1
>
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org>
> https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-09-29 6:17 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-29 3:28 [PATCH 0/3] Remove ImageDecoderLib and BmpImageDecoderLib Ruiyu Ni
-- strict thread matches above, loose matches on Subject: below --
2016-09-29 3:29 Ruiyu Ni
2016-09-29 3:50 ` Gao, Liming
2016-09-29 6:17 ` Ni, Ruiyu
2016-09-29 2:41 Ruiyu Ni
2016-09-29 2:48 ` Gao, Liming
2016-09-29 2:52 ` Ni, Ruiyu
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox