Re: [edk2-devel] [PATCH v1 19/19] MdePkg: Initialise VA_LIST variables before use

2019-08-23 Thread Liming Gao
Is this warning reported on ARM arch? In Base.h, VA_START is defined as below for ARM arch. Do you mean VS2017 report the warning for below macro? If so, can you propose the change in VA_START macro to fix this warning, then doesn't need to update consumer source code. #define

Re: [edk2-devel] [PATCH v1 19/19] MdePkg: Initialise VA_LIST variables before use

2019-08-23 Thread Alexei Fedorov
Reviewed-by: Alexei Fedorov Alexei From: Sami Mujawar Sent: 23 August 2019 11:55 To: devel@edk2.groups.io Cc: Sami Mujawar ; Alexei Fedorov ; ard.biesheu...@linaro.org ; leif.lindh...@linaro.org ; Matteo Carlini ; michael.d.kin...@intel.com ;

[edk2-devel] [PATCH v1 19/19] MdePkg: Initialise VA_LIST variables before use

2019-08-23 Thread Sami Mujawar
The VS2017 compiler reports 'warning C6001: Using uninitialized memory 'Marker'.' for VA_LIST variables. To fix this issue declare a VA_LIST global variable and use this to initialise VA_LIST variables before use. Note: The VA_LIST cannot be assigned a NULL value because some compilers define