Reviewed-by: Ray Ni <[email protected]> > -----Original Message----- > From: Guo, Gua <[email protected]> > Sent: Monday, June 27, 2022 1:03 PM > To: Ni, Ray <[email protected]>; [email protected] > Cc: Dong, Guo <[email protected]>; Lu, James <[email protected]> > Subject: RE: [PATCH] UefiPayloadPkg: Align Attribute value with UPL spec > > Yes, #2 is chosen in by the python interpreter. > > -----Original Message----- > From: Guo, Gua > Sent: Monday, June 27, 2022 12:07 PM > To: Ni, Ray <[email protected]>; [email protected] > Cc: Dong, Guo <[email protected]>; Lu, James <[email protected]> > Subject: RE: [PATCH] UefiPayloadPkg: Align Attribute value with UPL spec > > The python logic will like for below > > > upld_info_hdr.Attribute |= 1 if BuildTarget == "DEBUG" else 0 > =========> > > if BuildTarget == "DEBUG": > > upld_info_hdr.Attribute |= 1 > > else: > > upld_info_hdr.Attribute |= 0 > > Thanks, > Gua > > -----Original Message----- > From: Ni, Ray <[email protected]> > Sent: Monday, June 27, 2022 11:44 AM > To: Guo, Gua <[email protected]>; [email protected] > Cc: Dong, Guo <[email protected]>; Lu, James <[email protected]> > Subject: RE: [PATCH] UefiPayloadPkg: Align Attribute value with UPL spec > > > + upld_info_hdr.Attribute |= 1 if BuildTarget == "DEBUG" else 0 > > > I am not python expert. Above statement might be interpreted as: > 1. (upld_info.hdr.Attribute |= 1) if BuildTarget == "DEBUG" else 0 2. > upld_info_hdr.Attribute |= (1 if BuildTarget == > "DEBUG" else 0) > > Are we sure that the #2 is chosen in by the python interpreter?
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#90781): https://edk2.groups.io/g/devel/message/90781 Mute This Topic: https://groups.io/mt/92013929/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
