Opened 7 years ago

Closed 7 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 Changed 7 years ago by Even Rouault

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

comment:2 Changed 7 years ago by Kurt Schwehr

Owner: changed from warmerdam to Kurt Schwehr
Status: newassigned

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

comment:3 Changed 7 years ago by Kurt Schwehr

Resolution: fixed
Status: assignedclosed

r27699 on trunk and r27700 on 1.11

Note: See TracTickets for help on using tickets.