Ticket #20341: 20341-core-V4.diff

src/org/openstreetmap/josm/data/gpx/GpxImageEntry.java

@@ -4 +4 @@
 import static org.openstreetmap.josm.tools.I18n.tr;
 
 import java.io.File;
+import java.io.IOException;
 import java.util.Date;
 import java.util.List;
+import java.util.Locale;
 import java.util.Objects;
 import java.util.function.Consumer;
 
@@ -18 +20 @@
 import org.openstreetmap.josm.tools.Logging;
 
 import com.drew.imaging.jpeg.JpegMetadataReader;
+import com.drew.imaging.jpeg.JpegProcessingException;
 import com.drew.imaging.png.PngMetadataReader;
+import com.drew.imaging.png.PngProcessingException;
 import com.drew.imaging.tiff.TiffMetadataReader;
+import com.drew.imaging.tiff.TiffProcessingException;
 import com.drew.metadata.Directory;
 import com.drew.metadata.Metadata;
 import com.drew.metadata.MetadataException;
@@ -587 +592 @@
 
         try {
             // try to parse metadata according to extension
-            String ext = fn.substring(fn.lastIndexOf(".") + 1).toLowerCase();
+            String ext = fn.substring(fn.lastIndexOf('.') + 1).toLowerCase(Locale.US);
             switch (ext) {
             case "jpg":
             case "jpeg":
@@ -603 +608 @@
             default:
                 throw new NoMetadataReaderWarning(ext);
             }
-        } catch (Exception topException) {
+        } catch (JpegProcessingException | TiffProcessingException | PngProcessingException | IOException
+                | NoMetadataReaderWarning topException) {
             //try other formats (e.g. JPEG file with .png extension)
             try {
                 metadata = JpegMetadataReader.readMetadata(file);
-            } catch (Exception ex1) {
+            } catch (JpegProcessingException | IOException ex1) {
                 try {
                     metadata = TiffMetadataReader.readMetadata(file);
-                } catch (Exception ex2) {
+                } catch (TiffProcessingException | IOException ex2) {
                     try {
                         metadata = PngMetadataReader.readMetadata(file);
-                    } catch (Exception ex3) {
+                    } catch (PngProcessingException | IOException ex3) {
 
                         Logging.warn(topException);
                         Logging.info(tr("Can''t parse metadata for file \"{0}\". Using last modified date as timestamp.", fn));

src/org/openstreetmap/josm/gui/io/importexport/ImageImporter.java

@@ -7 +7 @@
 import java.io.IOException;
 import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Collections;
 import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
+import java.util.stream.Collectors;
 
 import javax.imageio.ImageIO;
 
@@ -29 +31 @@
     /**
      * The supported image file types on the current system
      */
-    public static final String[] SUPPORTED_FILE_TYPES = ImageIO.getReaderFileSuffixes();
+    public static final List<String> SUPPORTED_FILE_TYPES = Collections
+            .unmodifiableList(Arrays.stream(ImageIO.getReaderFileSuffixes()).sorted().collect(Collectors.toList()));
 
     /**
      * The default file filter

src/org/openstreetmap/josm/gui/layer/geoimage/ImageDisplay.java

@@ -242 +242 @@
         }
 
         private boolean updateImageEntry(Image img) {
+            if (img == null) {
+                synchronized (ImageDisplay.this) {
+                    errorLoading = true;
+                    ImageDisplay.this.repaint();
+                    return false;
+                }
+            }
             if (!(entry.getWidth() > 0 && entry.getHeight() > 0)) {
                 synchronized (entry) {
                     int width = img.getWidth(this);