Reviewed-by: Liming Gao <gaolim...@byosoft.com.cn> > -----邮件原件----- > 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Chao Li > 发送时间: 2022年10月20日 17:25 > 收件人: devel@edk2.groups.io > 抄送: Bob Feng <bob.c.f...@intel.com>; Liming Gao > <gaolim...@byosoft.com.cn>; Yuwei Chen <yuwei.c...@intel.com> > 主题: [edk2-devel] [PATCH v2] BaseTools: Fixed the multiple pairs brackets > issue in GenFv > > If operation Werro is turned on when compiling BaseTools, the > multi-brackets warning will be reported. This issue is comes from on of > the LoongArch enabled patche. Removed extra pairs brackets to fix it. > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4111 > > Cc: Bob Feng <bob.c.f...@intel.com> > Cc: Liming Gao <gaolim...@byosoft.com.cn> > Cc: Yuwei Chen <yuwei.c...@intel.com> > Signed-off-by: Chao Li <lic...@loongson.cn> > --- > BaseTools/Source/C/GenFv/GenFvInternalLib.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/BaseTools/Source/C/GenFv/GenFvInternalLib.c > b/BaseTools/Source/C/GenFv/GenFvInternalLib.c > index 5c3d54f5f7..b5b9425003 100644 > --- a/BaseTools/Source/C/GenFv/GenFvInternalLib.c > +++ b/BaseTools/Source/C/GenFv/GenFvInternalLib.c > @@ -3559,7 +3559,7 @@ Returns: > } > > > > // Machine type is LOONGARCH64, set a flag so LoongArch64 reset > vector processed. > > - if ((MachineType == EFI_IMAGE_MACHINE_LOONGARCH64)) { > > + if (MachineType == EFI_IMAGE_MACHINE_LOONGARCH64) { > > VerboseMsg("Located LoongArch64 SEC core in child FV"); > > mLoongArch = TRUE; > > } > > @@ -3721,7 +3721,7 @@ Returns: > mRiscV = TRUE; > > } > > > > - if ( (ImageContext.Machine == EFI_IMAGE_MACHINE_LOONGARCH64) ) > { > > + if (ImageContext.Machine == EFI_IMAGE_MACHINE_LOONGARCH64) { > > mLoongArch = TRUE; > > } > > > > @@ -4002,7 +4002,7 @@ Returns: > mArm = TRUE; > > } > > > > - if ( (ImageContext.Machine == EFI_IMAGE_MACHINE_LOONGARCH64) ) > { > > + if (ImageContext.Machine == EFI_IMAGE_MACHINE_LOONGARCH64) { > > mLoongArch = TRUE; > > } > > > > -- > 2.27.0 > > > > -=-=-=-=-=-= > Groups.io Links: You receive all messages sent to this group. > View/Reply Online (#95422): https://edk2.groups.io/g/devel/message/95422 > Mute This Topic: https://groups.io/mt/94450039/4905953 > Group Owner: devel+ow...@edk2.groups.io > Unsubscribe: https://edk2.groups.io/g/devel/unsub > [gaolim...@byosoft.com.cn] > -=-=-=-=-=-= >
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#95424): https://edk2.groups.io/g/devel/message/95424 Mute This Topic: https://groups.io/mt/94450077/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
