[PATCH] Adding support for ISCE image format

[mazhe]

Hello,

I would like to submit a new driver to support the image format used by the Jet Propulsion Laboratory's ISCE (Interferometric synthetic aperture radar Scientific Computing Environment) project.

It is the successor to the ROI_PAC project, which support was added at the beginning of the year.

The driver read compatible images, storing metadata in the ISCE domain. Consider it an initial support to be improved, as georeferencing is still absent (I have to find proper documentation).

The patch also include a minimal testsuite.

Thank you very much for your time and feedback.

[mazhe]

Patch for the test suite.

[Even Rouault]

A few remarks :

• poDriver->pfnIdentify = ISCEDataset::Identify; should be removed, since the Identify() method isn't specific enough (would identify any foo.bar accompanied with a foo.bar.xml). And the content of the Identify() method moved into Open() to avoid confusion.
• if ( psNode == NULL || CPLGetXMLNode( psNode, "=imageFile" ) == NULL ) will leak psNode if is not NULL but there's no imageFile node
• CPLXMLNode *papszCur : wrong use of hungarian notation. Should be psCur instead
• papszISCE2GDALDatatypes, papszGDAL2ISCEDatatypes, papszSchemeNames should be static
• the SCHEME creation option should be advertized in the GDAL_DMD_CREATIONOPTIONLIST driver metadata item (with a 'select-string' type for the enumerated list). See frmts/raw/envidataset.cpp

[mazhe]

Thank you very much,

Regarding Identify(), I assumed that opening the XML and looking for "<imageFile" would be a no-go, do you agree?

I am applying the other remarks.

[Even Rouault]

Regarding Identify(), I assumed that opening the XML and looking for "<imageFile" would be a no-go, do you agree?

Well, Identify() is supposed to be super fast, so ideally we shouldn't have to open side-car files. I can see 3 possibilities :

• make Identify() open the XML file and check for "<imageFile" in the first bytes of the file
• keep things like they are, but no longer advertize Identify() at the driver level
• make the driver open the XML file as the main file.

[mazhe]

I had the same idea as no1, but I was afraid of someday finding an image with a large comment and/or whitespace padding at the beginning of the image.

I'll go with solution no2 until I can figure out something better.

[mazhe]

All reviews fix integrated, add support for byte order

[Even Rouault]

gdal_autotest_isce.diff doesn't contain gdrivers/data/isce.slc. Should be added as attachment rather than a diff I think

[mazhe]

ISCE data test file.

[mazhe]

Oops, sorry, here it is.

[Even Rouault]

trunk r29352 "Add ISCE raster driver (patch by Matthieu Volat, #5991)"

trunk r29353 "ISCE: fix Windows build"