Opened 10 years ago

Closed 10 years ago

#5656 closed defect (fixed)

Inconsistent case for relativeToVRT

Reported by: Kurt Schwehr Owned by: Kurt Schwehr
Priority: normal Milestone:
Component: default Version: svn-trunk
Severity: normal Keywords:
Cc:

Description

For both 1.11 branch and trunk:

source:trunk/gdal/frmts/vrt/vrtdataset.cpp#L871​ and source:trunk/gdal/frmts/vrt/vrt_tutorial.dox#L645​

Are the only two places that have an "R" at the beginning. These should be changed to "relativeToVRT". e.g. 

bRelativeToVRT = 
       CSLFetchBoolean( papszOptions, "RelativeToVRT", FALSE );

should become 

bRelativeToVRT = 
       CSLFetchBoolean( papszOptions, "relativeToVRT", FALSE );

Correct?

Change History (3)

comment:1 by Even Rouault, 10 years ago

Please do not hesitate to directly push fix for such trivial changes. I'd note that the string comparison is case insensitive

comment:2 by Kurt Schwehr, 10 years ago

Owner: changed from warmerdam to Kurt Schwehr
Status: new → assigned

Will do. Just wanted to make sure there wasn't some sort of unintended consequence of the change.

comment:3 by Kurt Schwehr, 10 years ago

Resolution: → fixed
Status: assigned → closed

r27699 on trunk and r27700 on 1.11

Note: See TracTickets for help on using tickets.