[edk2] [patch 0/7]MdeModulePkg:UiApp remodeling task

2015-11-11 Thread Dandan Bi
Our current UiApp code is huge and hard to understand and maintain. So we split it to several independent libraries:device manager,boot manager, boot maintenance manager.Make the code clear and easy to maintain. Dandan Bi (7): MdeModulePkg:Create Boot Manager Library MdeModulePkg:Create Device

Re: [edk2] [patch 0/7]MdeModulePkg:UiApp remodeling task

2015-11-18 Thread Gao, Liming
Dandan: I have some comments. 1) gEfiSetupEnterGuid is not required. Library constructor() can directly do the initialization. 2) gEfiIfrFrontPageGuid can be defined in Package.dec file, and can be referred in C and VFR file. 3) The separate library is NULL class Library instance. They need t

Re: [edk2] [patch 0/7]MdeModulePkg:UiApp remodeling task

2015-11-18 Thread Bi, Dandan
Thanks for your comments. I will update the code and send patches later. Thanks, Dandan -Original Message- From: Gao, Liming Sent: Wednesday, November 18, 2015 5:11 PM To: Bi, Dandan; Dong, Eric; edk2-devel@lists.01.org Subject: RE: [patch 0/7]MdeModulePkg:UiApp remodeling task Dandan: