Ticket #18397: 18397.patch

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

@@ -112 +112 @@
             addLayers(loadLayers(r.getGpxData(), parsedProperly, fileName, tr("Markers from {0}", fileName)));
         } catch (SAXException e) {
             Logging.error(e);
-            throw new IOException(tr("Parsing data for layer ''{0}'' failed", fileName), e);
+            throw new IOException(e.getLocalizedMessage(), e);
         }
     }
 

src/org/openstreetmap/josm/io/GpxReader.java

@@ -644 +644 @@
         } catch (SAXException e) {
             if (tryToFinish) {
                 parser.tryToFinish();
-                if (parser.data.isEmpty())
-                    throw e;
-                String message = e.getMessage();
+                String message = e.getLocalizedMessage();
                 if (e instanceof SAXParseException) {
+                    boolean dot = message.lastIndexOf('.') == message.length() - 1;
+                    if (dot)
+                        message = message.substring(0, message.length() - 1);
                     SAXParseException spe = (SAXParseException) e;
                     message += ' ' + tr("(at line {0}, column {1})", spe.getLineNumber(), spe.getColumnNumber());
+                    if (dot)
+                        message += '.';
                 }
-                Logging.warn(message);
+                if (parser.data.creator != null && !parser.data.creator.trim().isEmpty()) {
+                    message += "\n" + tr("The file was created by \"{0}\".", parser.data.creator);
+                }
+                SAXException ex = new SAXException(message, e);
+                if (parser.data.isEmpty())
+                    throw ex;
+                Logging.warn(ex);
                 return false;
             } else
                 throw e;