<HTML dir=ltr><HEAD>
<META http-equiv=Content-Type content="text/html; charset=unicode">
<META content="MSHTML 6.00.2900.5626" name=GENERATOR></HEAD>
<BODY>
<DIV><FONT face=Arial color=#000000 size=2>I was looking at the lwgeom_geos_c.c intersects function - starting at line 2039 and was thinking isnt' this</FONT></DIV>
<DIV><FONT face=Arial size=2>a bit redundant. By the time we get into this function we have already done a bounding box check. It probably isn't adding much overhead, but thought I would flag it as potentially redundant.</FONT></DIV>
<DIV><FONT face=Arial size=2></FONT> </DIV>
<DIV> /*<BR> * short-circuit 1: if geom2 bounding box does not overlap<BR> * geom1 bounding box we can prematurely return FALSE.<BR> * Do the test IFF BOUNDING BOX AVAILABLE.<BR> */<BR> if ( getbox2d_p(SERIALIZED_FORM(geom1), &box1) &&<BR> getbox2d_p(SERIALIZED_FORM(geom2), &box2) )<BR> {<BR> if ( box2.xmax < box1.xmin ) PG_RETURN_BOOL(FALSE);<BR> if ( box2.xmin > box1.xmax ) PG_RETURN_BOOL(FALSE);<BR> if ( box2.ymax < box1.ymin ) PG_RETURN_BOOL(FALSE);<BR> if ( box2.ymin > box2.ymax ) PG_RETURN_BOOL(FALSE);<BR> }</DIV>
<DIV> </DIV>
<DIV> </DIV>
<DIV>thanks,</DIV>
<DIV>Regina</DIV>
<DIV> </DIV></BODY></HTML>
<HTML><BODY><P><hr size=1></P>
<P><STRONG>
The substance of this message, including any attachments, may be confidential, legally privileged and/or exempt from disclosure pursuant to Massachusetts law. It is intended solely for the addressee. If you received this in error, please contact the sender and delete the material from any computer.
</STRONG></P></BODY></HTML>
<P><hr size=1></P>
<P><STRONG><font size="2" color="339900"> Help make the earth a greener place. If at all possible resist printing this email and join us in saving paper. </p> <p> </font></STRONG></P>