From 5a5025e485b299c76ebed79fa63ba88a5b219a12 Mon Sep 17 00:00:00 2001 From: Gary Lin Date: Thu, 27 Oct 2016 18:16:15 +0800 Subject: OvmfPkg: Make more use of ARRAY_SIZE() Convert the remaining pieces to make the code shorter and more readable. Cc: Justen Jordan Cc: Laszlo Ersek Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Gary Lin Reviewed-by: Laszlo Ersek [lersek@redhat.com: tweak subject line] Signed-off-by: Laszlo Ersek --- OvmfPkg/Csm/CsmSupportLib/LegacyRegion.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'OvmfPkg/Csm') diff --git a/OvmfPkg/Csm/CsmSupportLib/LegacyRegion.c b/OvmfPkg/Csm/CsmSupportLib/LegacyRegion.c index b929db04d9..8d5d2e58a9 100644 --- a/OvmfPkg/Csm/CsmSupportLib/LegacyRegion.c +++ b/OvmfPkg/Csm/CsmSupportLib/LegacyRegion.c @@ -130,18 +130,18 @@ LegacyRegionManipulationInternal ( // Loop to find the start PAM. // StartIndex = 0; - for (Index = 0; Index < (sizeof(mSectionArray) / sizeof (mSectionArray[0])); Index++) { + for (Index = 0; Index < ARRAY_SIZE (mSectionArray); Index++) { if ((Start >= mSectionArray[Index].Start) && (Start < (mSectionArray[Index].Start + mSectionArray[Index].Length))) { StartIndex = Index; break; } } - ASSERT (Index < (sizeof(mSectionArray) / sizeof (mSectionArray[0]))); + ASSERT (Index < ARRAY_SIZE (mSectionArray)); // // Program PAM until end PAM is encountered // - for (Index = StartIndex; Index < (sizeof(mSectionArray) / sizeof (mSectionArray[0])); Index++) { + for (Index = StartIndex; Index < ARRAY_SIZE (mSectionArray); Index++) { if (ReadEnable != NULL) { if (*ReadEnable) { PciOr8 ( @@ -177,7 +177,7 @@ LegacyRegionManipulationInternal ( break; } } - ASSERT (Index < (sizeof(mSectionArray) / sizeof (mSectionArray[0]))); + ASSERT (Index < ARRAY_SIZE (mSectionArray)); return EFI_SUCCESS; } -- cgit v1.2.3