Opened 16 years ago

Last modified 15 years ago

#289 closed defect (fixed)

OGRPoint::get*() members should be const

Reported by: mcoletti@… Owned by: warmerdam
Priority: high Milestone:
Component: OGR_SF Version: unspecified
Severity: minor Keywords:
Cc:

Description

OGRPoint::getX(), OGRPoint::getY(), and OGRPoint::getZ() should be const members
since they do not change object state.

Attachments (1)

N2.DT1.gz (486.4 KB) - added by neteler@… 15 years ago.
DTED1 with potentiall NULL values (the sea in the West part of the map)

Download all attachments as: .zip

Change History (4)

comment:1 Changed 16 years ago by warmerdam

Agreed, I will plan to update this shortly.

Note, this is the sort of const correctness change that causes havoc with
old apps linked against the library!

Changed 15 years ago by neteler@…

Attachment: N2.DT1.gz added

DTED1 with potentiall NULL values (the sea in the West part of the map)

comment:2 Changed 15 years ago by warmerdam

This should be done before the 1.2.0 release if I am going to get to it ever. 

Ignore the note about DTED.  It was misplaced.

comment:3 Changed 15 years ago by warmerdam

I have made a comprehensive pass through ogr_geometry.h making everything 
const that makes sense.  Things still seem to build, though I am sure I will
suffer for this eventually. :-)


Note: See TracTickets for help on using tickets.