Opened 16 years ago

Closed 8 years ago

#2359 closed defect (wontfix)

Implement Widestring conversion for SDE

Reported by: dionw Owned by: hobu
Priority: normal Milestone:
Component: OGR_SF Version: svn-trunk
Severity: normal Keywords: sde widestring
Cc: warmerdam

Description

Seems related to changeset:14369 in which this enumeration value was removed.

Building (RHEL3, SDE 9.2) gives:

ogrsdelayer.cpp: In member function `int OGRSDELayer::Initialize(const char*, const char*, const char*)':
ogrsdelayer.cpp:224: `OFTWideString' undeclared (first use this function)
ogrsdelayer.cpp:224: (Each undeclared identifier is reported only once for each function it appears in.)
ogrsdelayer.cpp: In member function `OGRGeometry* OGRSDELayer::TranslateSDEGeometry(_SE_ShapeRecord*)':
ogrsdelayer.cpp:1676: warning: int format, long int arg (arg 4)
ogrsdelayer.cpp: In member function `virtual OGRErr OGRSDELayer::DeleteFeature(long int)':
ogrsdelayer.cpp:2413: warning: too few arguments for format
make: *** [../o/ogrsdelayer.o] Error 1

Change History (7)

comment:1 by Even Rouault, 16 years ago

Version: unspecifiedsvn-trunk

This is also the case for fme and idb drivers.

$ grep -r OFTWideString ogr ogr/ogrsf_frmts/fme/.svn/text-base/ogrfmelayercached.cpp.svn-base: else if( poFieldDef->GetType() == OFTWideString ) ogr/ogrsf_frmts/fme/.svn/text-base/ogrfmelayercached.cpp.svn-base: else if( poFieldDef->GetType() == OFTWideStringList ) ogr/ogrsf_frmts/fme/.svn/text-base/ogrfmelayercached.cpp.svn-base: eType = OFTWideString; ogr/ogrsf_frmts/fme/.svn/text-base/ogrfmelayercached.cpp.svn-base: eType = OFTWideStringList; ogr/ogrsf_frmts/fme/ogrfmelayercached.cpp: else if( poFieldDef->GetType() == OFTWideString ) ogr/ogrsf_frmts/fme/ogrfmelayercached.cpp: else if( poFieldDef->GetType() == OFTWideStringList ) ogr/ogrsf_frmts/fme/ogrfmelayercached.cpp: eType = OFTWideString; ogr/ogrsf_frmts/fme/ogrfmelayercached.cpp: eType = OFTWideStringList; ogr/ogrsf_frmts/idb/.svn/text-base/ogridbtablelayer.cpp.svn-base: case OFTWideStringList: ogr/ogrsf_frmts/idb/.svn/text-base/ogridbtablelayer.cpp.svn-base: case OFTWideString: ogr/ogrsf_frmts/idb/.svn/text-base/ogridbtablelayer.cpp.svn-base: case OFTWideStringList: ogr/ogrsf_frmts/idb/.svn/text-base/ogridbtablelayer.cpp.svn-base: case OFTWideString: ogr/ogrsf_frmts/idb/.svn/text-base/ogridbtablelayer.cpp.svn-base: case OFTWideStringList: ogr/ogrsf_frmts/idb/.svn/text-base/ogridbtablelayer.cpp.svn-base: case OFTWideString: ogr/ogrsf_frmts/idb/ogridbtablelayer.cpp: case OFTWideStringList: ogr/ogrsf_frmts/idb/ogridbtablelayer.cpp: case OFTWideString: ogr/ogrsf_frmts/idb/ogridbtablelayer.cpp: case OFTWideStringList: ogr/ogrsf_frmts/idb/ogridbtablelayer.cpp: case OFTWideString: ogr/ogrsf_frmts/idb/ogridbtablelayer.cpp: case OFTWideStringList: ogr/ogrsf_frmts/idb/ogridbtablelayer.cpp: case OFTWideString: ogr/ogrsf_frmts/sde/.svn/text-base/ogrsdelayer.cpp.svn-base: eOGRType = OFTWideString; ogr/ogrsf_frmts/sde/ogrsdelayer.cpp: eOGRType = OFTWideString;

comment:2 by Even Rouault, 16 years ago

As OFTWideString and OFTWideStringList have been removed from ogr_api.h, I think we'll have to explicitly mention it in GDAL 1.6.0 NEWS, so that people won't be surprised in case they recompile their own drivers.

comment:3 by warmerdam, 16 years ago

I have reintroduced the declaration for these symbols to reduce backward compatability hassles (r14410). If I'm going to bitch about other peoples backward compatability problems, I ought to take them seriously too!

comment:4 by warmerdam, 16 years ago

Cc: warmerdam added
Keywords: sde widestring added
Milestone: 1.6.0
Owner: changed from warmerdam to hobu

I have also removed references from the fme and idb drivers in trunk (r14412). I have not altered the SDE code which looks like it needs "real" support for translating wide string fields into UTF-8 per the RFC. Turning ticket over to Howard for this.

(PS the previous change was actually r14411)

comment:5 by hobu, 15 years ago

Milestone: 1.6.11.7.0
Summary: OGR SDE from trunk won't build, misses OFTWideString definitionImplement Widestring conversion for SDE

comment:6 by Even Rouault, 9 years ago

Milestone: 1.8.1

Removing obsolete milestone

comment:7 by Jukka Rahkonen, 8 years ago

Resolution: wontfix
Status: newclosed

I do not believe anybody is going to implement anything for SDE any more.

Note: See TracTickets for help on using tickets.