There should be no reason for the build tools to care about the native
word size of a particular target, so relying on a definition of MAX_UINTN
is definitely wrong, and most likely inaccurate on 32-bit build hosts.
So refactor the code in CommonLib and DevicePath so we no longer rely
on this definition.
Changes since v1:
- miss type change in #1 causing a build failure on MSVC
- add acks from Jaben
Cc: Laszlo Ersek <ler...@redhat.com>
Cc: Yonghong Zhu <yonghong....@intel.com>
Cc: Liming Gao <liming....@intel.com>
Cc: Bob Feng <bob.c.f...@intel.com>
Cc: Jaben Carsey <jaben.car...@intel.com>
Ard Biesheuvel (6):
BaseTools/CommonLib: avoid using 'native' word size in IP address
handling
BaseTools/CommonLib: use explicit 64-bit type in Strtoi()
BaseTools/DevicePath: use explicit 64-bit number parsing routines
BaseTools/DevicePath: use MAX_UINT16 as default device path max size
BaseTools/CommonLib: get rid of 'native' type string parsing routines
BaseTools/CommonLib: drop definition of MAX_UINTN
BaseTools/Source/C/Common/CommonLib.h | 25 ---
BaseTools/Source/C/Common/CommonLib.c | 206 ++----------------
.../Source/C/DevicePath/DevicePathFromText.c | 4 +-
.../Source/C/DevicePath/DevicePathUtilities.c | 4 +-
4 files changed, 25 insertions(+), 214 deletions(-)
--
2.19.1
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
