Opened 19 years ago

Closed 16 years ago

#682 closed enhancement (fixed)

.tab support for .jpg.

Reported by: warmerdam Owned by: warmerdam
Priority: high Milestone:
Component: GDAL_Raster Version: unspecified
Severity: minor Keywords:
Cc: dron

Description

Frank Warmerdam wrote :

> Sebastien Grignard wrote:
>
>> Hello all.
>> I've added support for reading georeferences from MapInfo .tab files in the
jpeg driver.
>> The code has been taken from the GeoTiff driver so it should support the same
file parameters.
>> Included is the diff against the current cvs.
>
>
>
> Sebastien,
>
> I tried out this code, but it doesn't handle the GCPs properly so I backed it
> out.  If you want to submit this as a bug report along I might work on fixing
> it up.  The full resolution would involve including the GCPs within the
> JPGDataset object itself, ensuring they get initialized and cleaned up
> properly, and implementing the methods to fetch them.
>
> Of course your implementation will work fine for "north up" images with only
> a modest memory leak.
>
> I have also been kicking around the idea of a more comprehensive approach
> to mapinfo .tab support where there was a .tab driver that reads the tab file
> and then uses GDAL to read the subdataset instead of having to update each of
> the format drivers that could have an associated .tab file. I'm not sure which
> would be better.
>
> Best regards,


I have updated the jpeg dataset code following your advice and using the geotiff
dataset as a reference.
Here is the resulting diff, I hope it's correct as I personnally don't work with
GCPs.

Sincerely,

-- 
Sebastien Grignard
R&D Developer
Archivideo
40, rue des Veyettes - 35000 Rennes FRANCE
Phone: +033 2 99 86 30 20

Attachments (1)

gdal-jpgdataset.diff (2.6 KB ) - added by warmerdam 19 years ago.
patch to jpgdataset.cpp to supply .tab support.

Download all attachments as: .zip

Change History (2)

by warmerdam, 19 years ago

Attachment: gdal-jpgdataset.diff added

patch to jpgdataset.cpp to supply .tab support.

comment:1 by dron, 16 years ago

Cc: dron added
Resolution: fixed
Status: assignedclosed

This one should be fixed now in trunk with r14169.

Best regards, Andrey

Note: See TracTickets for help on using tickets.