Index: trunk/src/org/openstreetmap/josm/gui/history/OpenChangesetPopupMenu.java
===================================================================
--- trunk/src/org/openstreetmap/josm/gui/history/OpenChangesetPopupMenu.java	(revision 13042)
+++ trunk/src/org/openstreetmap/josm/gui/history/OpenChangesetPopupMenu.java	(revision 13043)
@@ -56,5 +56,6 @@
             new ChangesetViewerEntry(tr("Open {0}", "OSMCha (OSM Changeset Analyzer)"), "https://osmcha.mapbox.com/changesets/{0}"),
             new ChangesetViewerEntry(tr("Open {0}", "OSM History Viewer"), "http://osmhv.openstreetmap.de/changeset.jsp?id={0}"),
-            new ChangesetViewerEntry(tr("Open {0}", "WhoDidIt (OSM Changeset Analyzer)"), "http://simon04.dev.openstreetmap.org/whodidit/index.html?changeset={0}&show=1")
+            new ChangesetViewerEntry(tr("Open {0}", "WhoDidIt (OSM Changeset Analyzer)"),
+                    "http://simon04.dev.openstreetmap.org/whodidit/index.html?changeset={0}&show=1")
     );
 
@@ -63,6 +64,8 @@
      */
     public static class ChangesetViewerEntry {
+        /** Name to be displayed in popup menu */
         @StructEntry
         public String name;
+        /** Templated service url. <code>{0}</code> will be replaced by changeset id */
         @StructEntry
         public String url;
@@ -80,6 +83,5 @@
 
         Action toAction(final long changesetId) {
-            final String url = MessageFormat.format(this.url, Long.toString(changesetId));
-            return new OpenBrowserAction(name, url);
+            return new OpenBrowserAction(name, MessageFormat.format(this.url, Long.toString(changesetId)));
         }
     }
