Opened 9 years ago

Closed 9 years ago

#6053 closed defect (fixed)

GNMRule extends CPLString

Reported by: Even Rouault Owned by: bishop
Priority: normal Milestone:
Component: default Version: unspecified
Severity: normal Keywords: gnm
Cc:

Description

GNMRule class extends CPLString, which itself extends std::string. We have a known issue #4099 with CPLString extending std::string on Windows, which lead to std::string symbols being redefined. So we'll likely have the same with GNMRule. Perhaps it should be reconsidered not to make GNMRule extends CPLString, but just have a CPLString member variable and appropriate conversion operators ?

Change History (2)

comment:1 by Bishop, 9 years ago

Ok, it's not hard to change GNMRule such way. I'll do so.

comment:2 by Bishop, 9 years ago

Resolution: fixed
Status: newclosed

Fixed in r30550

Note: See TracTickets for help on using tickets.