Fixed checkstyle errors detected by latest checkstyle version.

pom.xml

@@ -23,7 +23,7 @@
     <rugged.maven-assembly-plugin.version>2.5.3</rugged.maven-assembly-plugin.version>
     <rugged.maven-bundle-plugin.version>2.5.3</rugged.maven-bundle-plugin.version>
     <rugged.maven-changes-plugin.version>2.11</rugged.maven-changes-plugin.version>
-    <rugged.maven-checkstyle-plugin.version>2.13</rugged.maven-checkstyle-plugin.version>
+    <rugged.maven-checkstyle-plugin.version>2.17</rugged.maven-checkstyle-plugin.version>
     <rugged.maven-clean-plugin.version>2.6.1</rugged.maven-clean-plugin.version>
     <rugged.maven-compiler-plugin.version>3.2</rugged.maven-compiler-plugin.version>
     <rugged.maven-javadoc-plugin.version>2.10.1</rugged.maven-javadoc-plugin.version>

src/main/java/org/orekit/rugged/errors/Dump.java

@@ -74,7 +74,7 @@ class Dump {
     /** Simple constructor.
      * @param writer writer to the dump file
      */
-    public Dump(final PrintWriter writer) {
+    Dump(final PrintWriter writer) {
         this.writer          = writer;
         this.tiles           = new ArrayList<DumpedTileData>();
         this.sensors         = new ArrayList<DumpedSensorData>();
@@ -408,7 +408,7 @@ class Dump {
          * @param name of the tile
          * @param tile tile associated with this dump
          */
-        public DumpedTileData(final String name, final Tile tile) {
+        DumpedTileData(final String name, final Tile tile) {
             this.name       = name;
             this.tile       = tile;
             this.elevations = new OpenIntToDoubleHashMap();
@@ -469,7 +469,7 @@ class Dump {
          * @param dumpName name of the sensor dump (not the name of the sensor itself, for confidentiality reasons)
          * @param sensor dumped sensor
          */
-        public DumpedSensorData(final String dumpName, final LineSensor sensor) {
+        DumpedSensorData(final String dumpName, final LineSensor sensor) {
             this.dumpName = dumpName;
             this.sensor   = sensor;
             this.losMap   = new HashMap<Integer, List<Pair<AbsoluteDate, Vector3D>>>();

src/main/java/org/orekit/rugged/errors/DumpReplayer.java

@@ -1044,9 +1044,9 @@ public class DumpReplayer {
          * @param longitudeStep step in longitude (size of one raster element)
          * @param longitudeColumns number of longitude columns
          */
-        public ParsedTile(final String name,
-                          final double minLatitude, final double latitudeStep, final int latitudeRows,
-                          final double minLongitude, final double longitudeStep, final int longitudeColumns) {
+        ParsedTile(final String name,
+                   final double minLatitude, final double latitudeStep, final int latitudeRows,
+                   final double minLongitude, final double longitudeStep, final int longitudeColumns) {
             this.name             = name;
             this.minLatitude      = minLatitude;
             this.latitudeStep     = latitudeStep;
@@ -1121,7 +1121,7 @@ public class DumpReplayer {
         /** simple constructor.
          * @param name name of the sensor
          */
-        public ParsedSensor(final String name) {
+        ParsedSensor(final String name) {
             this.name     = name;
             this.losMap   = new HashMap<Integer, List<Pair<AbsoluteDate, Vector3D>>>();
             this.datation = new ArrayList<Pair<Double, AbsoluteDate>>();
@@ -1379,9 +1379,9 @@ public class DumpReplayer {
          * @param normal mean plane normal
          * @param cachedResults cached results
          */
-        public ParsedMeanPlane(final int minLine, final int maxLine,
-                               final int maxEval, final double accuracy, final Vector3D normal,
-                               final CrossingResult[] cachedResults) {
+        ParsedMeanPlane(final int minLine, final int maxLine,
+                        final int maxEval, final double accuracy, final Vector3D normal,
+                        final CrossingResult[] cachedResults) {
             this.minLine       = minLine;
             this.maxLine       = maxLine;
             this.maxEval       = maxEval;

src/main/java/org/orekit/rugged/errors/RuggedMessages.java

@@ -88,7 +88,7 @@ public enum RuggedMessages implements Localizable {
      * @param sourceFormat source English format to use when no
      * localized version is available
      */
-    private RuggedMessages(final String sourceFormat) {
+    RuggedMessages(final String sourceFormat) {
         this.sourceFormat = sourceFormat;
     }
 

src/main/java/org/orekit/rugged/intersection/duvenhage/DuvenhageAlgorithm.java

@@ -645,8 +645,8 @@ public class DuvenhageAlgorithm implements IntersectionAlgorithm {
          * it is on a top/bottom limit
          * @exception OrekitException if geodetic coordinates cannot be computed
          */
-        public LimitPoint(final ExtendedEllipsoid ellipsoid, final double referenceLongitude,
-                          final Vector3D cartesian, final boolean side)
+        LimitPoint(final ExtendedEllipsoid ellipsoid, final double referenceLongitude,
+                   final Vector3D cartesian, final boolean side)
             throws OrekitException {
             this(ellipsoid.transform(cartesian, ellipsoid.getBodyFrame(), null, referenceLongitude), side);
         }
@@ -656,7 +656,7 @@ public class DuvenhageAlgorithm implements IntersectionAlgorithm {
          * @param side if true, the point is on a side limit, otherwise
          * it is on a top/bottom limit
          */
-        public LimitPoint(final NormalizedGeodeticPoint point, final boolean side) {
+        LimitPoint(final NormalizedGeodeticPoint point, final boolean side) {
             this.point = point;
             this.side  = side;
         }

src/main/java/org/orekit/rugged/los/LOSBuilder.java

@@ -109,7 +109,7 @@ public class LOSBuilder {
         /** Simple constructor.
          * @param transform underlying time-independent transform
          */
-        public TransformAdapter(final TimeIndependentLOSTransform transform) {
+        TransformAdapter(final TimeIndependentLOSTransform transform) {
             this.transform = transform;
         }
 
@@ -164,7 +164,7 @@ public class LOSBuilder {
          * @param raw raw directions
          * @param transforms transforms to apply
          */
-        public TransformsSequenceLOS(final List<Vector3D> raw, final List<LOSTransform> transforms) {
+        TransformsSequenceLOS(final List<Vector3D> raw, final List<LOSTransform> transforms) {
 
             // copy the lists, to ensure immutability of the built object,
             // in case addTransform is called again after build
@@ -277,7 +277,7 @@ public class LOSBuilder {
          * @param raw raw directions
          * @param transforms transforms to apply (must be time-independent!)
          */
-        public FixedLOS(final List<Vector3D> raw, final List<LOSTransform> transforms) {
+        FixedLOS(final List<Vector3D> raw, final List<LOSTransform> transforms) {
             super(raw, transforms);
             transformed = new Vector3D[raw.size()];
         }