[PATCH] Geotagged image viewer should rotate images according to EXIF orientation tag

[anonymous]

Geotagged image viewer should rotate images according to EXIF orientation tag.

[Cobra]

+1

very annoying. should be rather simple to implement, I hope.

[m.zdila@…]

Adding my vote. Thanks in advance!

[bastiK]

Thanks for the bump, I forgot about this one...

[m.zdila@…]

patch

[m.zdila@…]

Please check and apply my patch. I tested it with all 4 rotations. Image flipping is not implemented but AFAIK it is very uncommon. Thanks in advance.

[bastiK]

I'll apply it, when merge window is open, about next We.

One minor thing: There should be a field exifOrientation in ImageEntry class and it should be initialized in GeoImageLayer.extractExif and not in ImageDisplay.run.

[m.zdila@…]

I am examining exifOrientation in ImageDisplay.run because entry with EXIF is not available at org.openstreetmap.josm.gui.layer.geoimage.CorrelateGpxWithImages line 382. I can try to fix it if you give me a hint.

[bastiK]

Ok, now I see the problem. Maybe add method setOrientation to ImageDisplay, then put your new exif code in ExifReader to be called from CorrelateGpxWithImages line 382. Then in GeoImageLayer.extractExif just put the line e.setOrientation(dir.getInt(ExifDirectory.TAG_ORIENTATION));.

[bastiK]

In [4241/josm]:

applied #5605 - Geotagged image viewer should rotate images according to EXIF orientation tag (patch by m.zdila, some modifications)