Would gatekeeper help review this patch? The priorty of bitwise operator is lower than relational operaotr. So following code has potential problem.
In Check_TCON ( TCON *tc )
Is_True ( TCON_v1(*tc)|TCON_v2(*tc)|TCON_v3(*tc) == 0,
("High order word of %s TCON non zero %x",
Mtype_Name(TCON_ty(*tc)), TCON_v1(*tc)) );
The original meaning is checking if TCON_V1, TCON_V2, TCON_V3 all zero, now
the assertion will not happen when TCON_V2 is non-zeron.
in control.cxx
#define CGI_IS_INT_TYPE 0x0001
#define CGI_is_int_type(c) ((c)->flags & CGI_IS_INT_TYPE != 0)
When CGI_IS_INT_TYPE changes the marco value, the second marco will not
correct.
Regards
Shi Hui
operator_priority.patch
Description: Binary data
------------------------------------------------------------------------------ WhatsUp Gold - Download Free Network Management Software The most intuitive, comprehensive, and cost-effective network management toolset available today. Delivers lowest initial acquisition cost and overall TCO of any competing solution. http://p.sf.net/sfu/whatsupgold-sd
_______________________________________________ Open64-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/open64-devel
