Opened 21 years ago

Last modified 20 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@… 20 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 by warmerdam, 21 years ago 

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!

by neteler@…, 20 years ago

Attachment: N2.DT1.gz​ added

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

comment:2 by warmerdam, 20 years ago 

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 by warmerdam, 20 years ago 

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.