Opened 13 years ago

Closed 13 years ago

#3118 closed enhancement (fixed)

OGREnvelope intersect

Reported by: bishop Owned by: warmerdam
Priority: normal Milestone: 1.7.0
Component: OGR_SF Version: svn-trunk
Severity: normal Keywords: OGREnvelope intersect
Cc:

Description

As the oposite to Merge method I added Intersect method

Attachments (2)

ogrenvelope_intersect.patch (1.0 KB ) - added by bishop 13 years ago.
OGREnvelope intersect
ogrenvelope_intersect1.patch (971 bytes ) - added by bishop 13 years ago.
OGREnvelope intersect fix metod

Download all attachments as: .zip

Change History (5)

by bishop, 13 years ago

Attachment: ogrenvelope_intersect.patch added

OGREnvelope intersect

comment:1 by Even Rouault, 13 years ago

your implementation of Intersect() can lead to an invalid envelop (a min > max) if the intersection is null. You should reset all minx,maxx,miny,maxy to 0 in that case.

I don't see the usefullness of void Intersect( double dfX, double dfY ) (I know there's an equivalent method for Merge(), but for intersect I'm more skeptical)

by bishop, 13 years ago

OGREnvelope intersect fix metod

comment:2 by bishop, 13 years ago

I made changes as was proposaled

comment:3 by Even Rouault, 13 years ago

Milestone: 1.7.0
Resolution: fixed
Status: newclosed

Commited in trunk in r17602

Note: See TracTickets for help on using tickets.