Index: trunk/scripts/TagInfoExtract.java
===================================================================
--- trunk/scripts/TagInfoExtract.java	(revision 18050)
+++ trunk/scripts/TagInfoExtract.java	(revision 18051)
@@ -293,5 +293,6 @@
         }
 
-        private void fillTagsMap(Map<Tag, TagInfoTag> optionalTags, KeyedItem item, String value, String presetName, Set<TagInfoTag.Type> types, String descriptionPrefix, String iconUrl) {
+        private void fillTagsMap(Map<Tag, TagInfoTag> optionalTags, KeyedItem item, String value,
+                String presetName, Set<TagInfoTag.Type> types, String descriptionPrefix, String iconUrl) {
             optionalTags.compute(new Tag(item.key, value), (osmTag, tagInfoTag) -> {
                 if (tagInfoTag == null) {
Index: trunk/src/org/openstreetmap/josm/actions/RestorePropertyAction.java
===================================================================
--- trunk/src/org/openstreetmap/josm/actions/RestorePropertyAction.java	(revision 18050)
+++ trunk/src/org/openstreetmap/josm/actions/RestorePropertyAction.java	(revision 18051)
@@ -56,5 +56,5 @@
 
         Map<String, String> changes = TableHelper.selectedIndices(selectionModel).boxed()
-                .collect(HashMap::new, (m,i)->m.put(keyFn.apply(i), valueFn.apply(i)), HashMap::putAll);
+                .collect(HashMap::new, (m, i) -> m.put(keyFn.apply(i), valueFn.apply(i)), HashMap::putAll);
         ChangePropertyCommand command = new ChangePropertyCommand(Collections.singleton(primitive), changes);
         UndoRedoHandler.getInstance().add(command);
Index: trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ImagesLoader.java
===================================================================
--- trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ImagesLoader.java	(revision 18050)
+++ trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ImagesLoader.java	(revision 18051)
@@ -42,8 +42,8 @@
     /**
      * Constructs a new {@code ImagesLoader}.
-     * @param selection
-     * @param gpxLayer
+     * @param selection image files to load
+     * @param gpxLayer associated GPX layer
      */
-    public ImagesLoader(Collection<File> selection, GpxLayer gpxLayer) {
+    ImagesLoader(Collection<File> selection, GpxLayer gpxLayer) {
         super(tr("Extracting GPS locations from EXIF"));
         this.selection = selection;
Index: trunk/src/org/openstreetmap/josm/gui/layer/geoimage/SynchronizeTimeFromPhotoDialog.java
===================================================================
--- trunk/src/org/openstreetmap/josm/gui/layer/geoimage/SynchronizeTimeFromPhotoDialog.java	(revision 18050)
+++ trunk/src/org/openstreetmap/josm/gui/layer/geoimage/SynchronizeTimeFromPhotoDialog.java	(revision 18051)
@@ -59,5 +59,5 @@
      * @param images list of image entries
      */
-    public SynchronizeTimeFromPhotoDialog(Component parent, List<ImageEntry> images) {
+    SynchronizeTimeFromPhotoDialog(Component parent, List<ImageEntry> images) {
         super(parent, tr("Synchronize time from a photo of the GPS receiver"), tr("OK"), tr("Cancel"));
         setButtonIcons("ok", "cancel");
Index: trunk/src/org/openstreetmap/josm/gui/layer/gpx/GpxDataHelper.java
===================================================================
--- trunk/src/org/openstreetmap/josm/gui/layer/gpx/GpxDataHelper.java	(revision 18050)
+++ trunk/src/org/openstreetmap/josm/gui/layer/gpx/GpxDataHelper.java	(revision 18051)
@@ -33,4 +33,8 @@
  */
 public final class GpxDataHelper {
+
+    private GpxDataHelper() {
+        // Hide public constructor
+    }
 
     /**
Index: trunk/src/org/openstreetmap/josm/io/GpxReader.java
===================================================================
--- trunk/src/org/openstreetmap/josm/io/GpxReader.java	(revision 18050)
+++ trunk/src/org/openstreetmap/josm/io/GpxReader.java	(revision 18051)
@@ -133,5 +133,5 @@
         @Override
         public void startElement(String namespaceURI, String localName, String qName, Attributes atts) throws SAXException {
-            elements.push(new String[] { namespaceURI, localName, qName });
+            elements.push(new String[] {namespaceURI, localName, qName});
             switch(currentState) {
             case INIT:
Index: trunk/test/unit/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetValidationTest.java
===================================================================
--- trunk/test/unit/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetValidationTest.java	(revision 18050)
+++ trunk/test/unit/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetValidationTest.java	(revision 18051)
@@ -51,11 +51,13 @@
         TaggingPresetValidation.validate(primitive, label);
 
+        // CHECKSTYLE.OFF: LineLength
         assertTrue(label.isVisible());
         assertEquals("<html><ul>" +
-                "<li>Opening hours syntax (Hours without minutes)</li>" +
-                "<li>unusual value of width: meters is default; only positive values; point is decimal separator; if units, put space then unit</li>" +
-                "<li>unusual value of incline, use x% or x° or up or down instead</li>" +
-                "<li>suspicious tag combination (width on suspicious object)</li>" +
-                "<li>suspicious tag combination (incline on suspicious object)</li></ul>", label.getToolTipText());
+            "<li>Opening hours syntax (Hours without minutes)</li>" +
+            "<li>unusual value of width: meters is default; only positive values; point is decimal separator; if units, put space then unit</li>" +
+            "<li>unusual value of incline, use x% or x° or up or down instead</li>" +
+            "<li>suspicious tag combination (width on suspicious object)</li>" +
+            "<li>suspicious tag combination (incline on suspicious object)</li></ul>", label.getToolTipText());
+        // CHECKSTYLE.ON: LineLength
     }
 
Index: trunk/test/unit/org/openstreetmap/josm/testutils/annotations/AnnotationUtils.java
===================================================================
--- trunk/test/unit/org/openstreetmap/josm/testutils/annotations/AnnotationUtils.java	(revision 18050)
+++ trunk/test/unit/org/openstreetmap/josm/testutils/annotations/AnnotationUtils.java	(revision 18051)
@@ -62,4 +62,5 @@
                 } catch (UnsupportedOperationException e) {
                     // Probably an unmodifiable collection
+                    System.err.println("Unable to clear " + field);
                 }
             } else if (!isFinal) {
Index: trunk/test/unit/org/openstreetmap/josm/tools/JosmDecimalFormatSymbolsProviderTest.java
===================================================================
--- trunk/test/unit/org/openstreetmap/josm/tools/JosmDecimalFormatSymbolsProviderTest.java	(revision 18050)
+++ trunk/test/unit/org/openstreetmap/josm/tools/JosmDecimalFormatSymbolsProviderTest.java	(revision 18051)
@@ -35,7 +35,9 @@
     static void beforeAll() throws IOException {
         if (Utils.getJavaVersion() >= 9) {
-            assertEquals("SPI,JRE,CLDR", System.getProperty("java.locale.providers"), "This test must be launched with -Djava.locale.providers=SPI,JRE,CLDR");
+            assertEquals("SPI,JRE,CLDR", System.getProperty("java.locale.providers"),
+                    "This test must be launched with -Djava.locale.providers=SPI,JRE,CLDR");
             try (InputStream in = I18n.class.getResourceAsStream("/META-INF/services/java.text.spi.DecimalFormatSymbolsProvider")) {
-                assertEquals("org.openstreetmap.josm.tools.JosmDecimalFormatSymbolsProvider", new String(Utils.readBytesFromStream(in), StandardCharsets.UTF_8).trim());
+                assertEquals("org.openstreetmap.josm.tools.JosmDecimalFormatSymbolsProvider",
+                        new String(Utils.readBytesFromStream(in), StandardCharsets.UTF_8).trim());
             }
         }
Index: trunk/tools/checkstyle/META-INF/MANIFEST.MF
===================================================================
--- trunk/tools/checkstyle/META-INF/MANIFEST.MF	(revision 18050)
+++ trunk/tools/checkstyle/META-INF/MANIFEST.MF	(revision 18051)
@@ -3,5 +3,5 @@
 Bundle-Name: JOSM Checkstyle Eclipse Plugin
 Bundle-SymbolicName: org.openstreetmap.josm.checkstyle-eclipse-plugin;singleton:=true
-Bundle-Version: 8.35.0.qualifier
+Bundle-Version: 8.43.0.qualifier
 Require-Bundle: net.sf.eclipsecs.checkstyle,
  net.sf.eclipsecs.core,
Index: trunk/tools/ivy.xml
===================================================================
--- trunk/tools/ivy.xml	(revision 18050)
+++ trunk/tools/ivy.xml	(revision 18051)
@@ -17,5 +17,5 @@
         <dependency org="net.java.dev.javacc" name="javacc" rev="7.0.10" conf="javacc->default"/>
         <!-- checkstyle->default -->
-        <dependency org="com.puppycrawl.tools" name="checkstyle" rev="8.36" conf="checkstyle->default"/>
+        <dependency org="com.puppycrawl.tools" name="checkstyle" rev="8.43" conf="checkstyle->default"/>
         <!-- proguard->default -->
         <dependency org="com.guardsquare" name="proguard-ant" rev="7.1.0-beta3" conf="proguard->default"/>