Re: [edk2] [patch] MdeModulePkg/PciBusDxe: Fix VS2012 build failure

2018-02-07 Thread Bi, Dandan
Hi Laszlo, Thanks for your comments. I will update the commit message. Regards, Dandan -Original Message- From: Laszlo Ersek [mailto:ler...@redhat.com] Sent: Wednesday, February 7, 2018 4:17 PM To: Zeng, Star ; Bi, Dandan ;

Re: [edk2] [patch] MdeModulePkg/PciBusDxe: Fix VS2012 build failure

2018-02-06 Thread Ni, Ruiyu
On 2/7/2018 9:30 AM, Dandan Bi wrote: Initialize local variable to suppress warning C4703: potentially uninitialized local pointer variable. Cc: Star Zeng Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Dandan Bi

Re: [edk2] [patch] MdeModulePkg/PciBusDxe: Fix VS2012 build failure

2018-02-06 Thread Bi, Dandan
Thank you star. I will add comment before commit the patch. Regards, Dandan -Original Message- From: Zeng, Star Sent: Wednesday, February 7, 2018 9:45 AM To: Bi, Dandan ; edk2-devel@lists.01.org Cc: Ni, Ruiyu ; Laszlo Ersek (ler...@redhat.com)

Re: [edk2] [patch] MdeModulePkg/PciBusDxe: Fix VS2012 build failure

2018-02-06 Thread Zeng, Star
Reviewed-by: Star Zeng Cc Laszlo. You can add similar comment like 1ea53108f6c1010a00a828d1d59ea28934025415 as recommended by https://bugzilla.tianocore.org/show_bug.cgi?id=607. Thanks, Star -Original Message- From: Bi, Dandan Sent: Wednesday, February 7, 2018

[edk2] [patch] MdeModulePkg/PciBusDxe: Fix VS2012 build failure

2018-02-06 Thread Dandan Bi
Initialize local variable to suppress warning C4703: potentially uninitialized local pointer variable. Cc: Star Zeng Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Dandan Bi ---