You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by lu...@apache.org on 2015/11/02 15:34:28 UTC

[3/4] [math] Fixed checkstyle warnings.

Fixed checkstyle warnings.

These warnings correspond to redundant modifiers (public, static, ...).
They are identified by recent versions of checkstyle, in particular the
one shipped with Eclipse. They are not detected by our
maven-checkstyle-plugin yet because it is not the latest one and still
depends on an older version of checkstyle.


Project: http://git-wip-us.apache.org/repos/asf/commons-math/repo
Commit: http://git-wip-us.apache.org/repos/asf/commons-math/commit/2114da38
Tree: http://git-wip-us.apache.org/repos/asf/commons-math/tree/2114da38
Diff: http://git-wip-us.apache.org/repos/asf/commons-math/diff/2114da38

Branch: refs/heads/MATH_3_X
Commit: 2114da384abce47f4b12784711156a7d74d24bf1
Parents: a07663c
Author: Luc Maisonobe <lu...@apache.org>
Authored: Mon Nov 2 15:23:30 2015 +0100
Committer: Luc Maisonobe <lu...@apache.org>
Committed: Mon Nov 2 15:23:30 2015 +0100

----------------------------------------------------------------------
 .../differentiation/DerivativeStructure.java       |  2 +-
 .../BicubicInterpolatingFunction.java              |  2 +-
 .../BicubicSplineInterpolatingFunction.java        |  5 ++---
 .../interpolation/InterpolatingMicrosphere.java    |  5 ++---
 .../TricubicInterpolatingFunction.java             |  2 +-
 .../TricubicSplineInterpolatingFunction.java       |  2 +-
 .../analysis/polynomials/PolynomialsUtils.java     |  2 +-
 .../math3/exception/util/LocalizedFormats.java     |  2 +-
 .../apache/commons/math3/fitting/CurveFitter.java  |  2 +-
 .../fitting/leastsquares/GaussNewtonOptimizer.java |  2 +-
 .../geometry/euclidean/oned/IntervalsSet.java      |  2 +-
 .../euclidean/threed/OutlineExtractor.java         |  2 +-
 .../geometry/euclidean/threed/PolyhedronsSet.java  |  6 +++---
 .../euclidean/threed/SphericalCoordinates.java     |  2 +-
 .../math3/geometry/euclidean/twod/Line.java        |  4 ++--
 .../math3/geometry/euclidean/twod/NestedLoops.java |  2 +-
 .../math3/geometry/euclidean/twod/PolygonsSet.java | 14 +++++++-------
 .../geometry/partitioning/BoundaryProjector.java   |  2 +-
 .../geometry/partitioning/BoundarySizeVisitor.java |  2 +-
 .../geometry/partitioning/Characterization.java    |  2 +-
 .../math3/geometry/partitioning/InsideFinder.java  |  2 +-
 .../math3/geometry/partitioning/Region.java        |  2 +-
 .../math3/geometry/partitioning/RegionFactory.java |  4 ++--
 .../math3/geometry/partitioning/SubHyperplane.java |  2 +-
 .../geometry/partitioning/utilities/AVLTree.java   |  2 +-
 .../math3/geometry/spherical/oned/ArcsSet.java     |  2 +-
 .../math3/geometry/spherical/twod/Circle.java      |  2 +-
 .../geometry/spherical/twod/EdgesBuilder.java      |  2 +-
 .../spherical/twod/PropertiesComputer.java         |  4 ++--
 .../math3/linear/BiDiagonalTransformer.java        |  2 +-
 .../math3/linear/HessenbergTransformer.java        |  2 +-
 .../apache/commons/math3/linear/MatrixUtils.java   |  4 ++--
 .../commons/math3/linear/SchurTransformer.java     |  2 +-
 .../org/apache/commons/math3/linear/SymmLQ.java    |  2 +-
 .../math3/linear/TriDiagonalTransformer.java       |  2 +-
 .../ml/clustering/KMeansPlusPlusClusterer.java     |  2 +-
 .../commons/math3/ml/neuralnet/MapUtils.java       |  3 +--
 .../commons/math3/ode/ExpandableStatefulODE.java   |  2 +-
 .../apache/commons/math3/ode/JacobianMatrices.java |  4 ++--
 .../commons/math3/ode/MultistepIntegrator.java     |  4 ++--
 .../commons/math3/ode/ParameterConfiguration.java  |  2 +-
 .../math3/ode/ParameterJacobianWrapper.java        |  6 +++---
 .../commons/math3/ode/ParameterizedWrapper.java    |  2 +-
 .../commons/math3/ode/events/EventHandler.java     |  2 +-
 .../commons/math3/ode/events/EventState.java       |  2 +-
 .../math3/ode/nonstiff/AdamsMoultonIntegrator.java |  2 +-
 .../ClassicalRungeKuttaStepInterpolator.java       |  4 ++--
 .../nonstiff/DormandPrince54StepInterpolator.java  |  4 ++--
 .../nonstiff/DormandPrince853StepInterpolator.java |  4 ++--
 .../math3/ode/nonstiff/EulerStepInterpolator.java  |  4 ++--
 .../math3/ode/nonstiff/GillStepInterpolator.java   |  4 ++--
 .../GraggBulirschStoerStepInterpolator.java        | 17 ++++++++---------
 .../ode/nonstiff/HighamHall54StepInterpolator.java |  4 ++--
 .../math3/ode/nonstiff/LutherStepInterpolator.java |  4 ++--
 .../ode/nonstiff/MidpointStepInterpolator.java     |  4 ++--
 .../ode/nonstiff/RungeKuttaStepInterpolator.java   |  2 +-
 .../nonstiff/ThreeEighthesStepInterpolator.java    |  4 ++--
 .../math3/ode/sampling/StepNormalizerBounds.java   |  2 +-
 .../apache/commons/math3/optim/PointValuePair.java |  2 +-
 .../commons/math3/optim/PointVectorValuePair.java  |  2 +-
 .../commons/math3/optim/linear/Relationship.java   |  2 +-
 .../scalar/MultivariateFunctionMappingAdapter.java |  6 +++---
 .../NonLinearConjugateGradientOptimizer.java       |  2 +-
 .../nonlinear/scalar/noderiv/CMAESOptimizer.java   |  4 ++--
 .../commons/math3/optimization/PointValuePair.java |  2 +-
 .../math3/optimization/PointVectorValuePair.java   |  2 +-
 .../math3/optimization/direct/CMAESOptimizer.java  |  2 +-
 .../direct/MultivariateFunctionMappingAdapter.java |  8 ++++----
 .../math3/optimization/fitting/CurveFitter.java    |  4 ++--
 .../NonLinearConjugateGradientOptimizer.java       |  2 +-
 .../math3/optimization/linear/Relationship.java    |  2 +-
 .../math3/random/EmpiricalDistribution.java        |  4 ++--
 .../stat/clustering/KMeansPlusPlusClusterer.java   |  2 +-
 .../correlation/StorelessBivariateCovariance.java  |  4 ++--
 .../descriptive/AggregateSummaryStatistics.java    |  2 +-
 .../math3/stat/descriptive/moment/FirstMoment.java |  4 ++--
 .../stat/descriptive/moment/FourthMoment.java      |  4 ++--
 .../math3/stat/descriptive/moment/ThirdMoment.java |  4 ++--
 .../stat/descriptive/rank/PSquarePercentile.java   |  2 +-
 .../math3/stat/descriptive/rank/Percentile.java    |  4 ++--
 .../commons/math3/stat/ranking/NaturalRanking.java |  2 +-
 .../math3/transform/FastFourierTransformer.java    |  3 +--
 .../apache/commons/math3/util/Combinations.java    |  9 ++++-----
 .../org/apache/commons/math3/util/FastMath.java    |  6 +++---
 .../org/apache/commons/math3/util/MathArrays.java  |  6 +++---
 .../commons/math3/util/ResizableDoubleArray.java   |  2 +-
 86 files changed, 140 insertions(+), 146 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/analysis/differentiation/DerivativeStructure.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/analysis/differentiation/DerivativeStructure.java b/src/main/java/org/apache/commons/math3/analysis/differentiation/DerivativeStructure.java
index 7c45d18..c625931 100644
--- a/src/main/java/org/apache/commons/math3/analysis/differentiation/DerivativeStructure.java
+++ b/src/main/java/org/apache/commons/math3/analysis/differentiation/DerivativeStructure.java
@@ -1177,7 +1177,7 @@ public class DerivativeStructure implements RealFieldElement<DerivativeStructure
          * @param order derivation order
          * @param data partial derivatives
          */
-        public DataTransferObject(final int variables, final int order, final double[] data) {
+        DataTransferObject(final int variables, final int order, final double[] data) {
             this.variables = variables;
             this.order     = order;
             this.data      = data;

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/analysis/interpolation/BicubicInterpolatingFunction.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/analysis/interpolation/BicubicInterpolatingFunction.java b/src/main/java/org/apache/commons/math3/analysis/interpolation/BicubicInterpolatingFunction.java
index 7fe947f..454d8f8 100644
--- a/src/main/java/org/apache/commons/math3/analysis/interpolation/BicubicInterpolatingFunction.java
+++ b/src/main/java/org/apache/commons/math3/analysis/interpolation/BicubicInterpolatingFunction.java
@@ -273,7 +273,7 @@ class BicubicFunction implements BivariateFunction {
      *
      * @param coeff Spline coefficients.
      */
-    public BicubicFunction(double[] coeff) {
+    BicubicFunction(double[] coeff) {
         a = new double[N][N];
         for (int j = 0; j < N; j++) {
             final double[] aJ = a[j];

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/analysis/interpolation/BicubicSplineInterpolatingFunction.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/analysis/interpolation/BicubicSplineInterpolatingFunction.java b/src/main/java/org/apache/commons/math3/analysis/interpolation/BicubicSplineInterpolatingFunction.java
index 522c2a5..7056094 100644
--- a/src/main/java/org/apache/commons/math3/analysis/interpolation/BicubicSplineInterpolatingFunction.java
+++ b/src/main/java/org/apache/commons/math3/analysis/interpolation/BicubicSplineInterpolatingFunction.java
@@ -457,7 +457,7 @@ class BicubicSplineFunction implements BivariateFunction {
      *
      * @param coeff Spline coefficients.
      */
-    public BicubicSplineFunction(double[] coeff) {
+    BicubicSplineFunction(double[] coeff) {
         this(coeff, false);
     }
 
@@ -469,8 +469,7 @@ class BicubicSplineFunction implements BivariateFunction {
      * needed for calling any of the methods that compute the partial derivatives
      * this function.
      */
-    public BicubicSplineFunction(double[] coeff,
-                                 boolean initializeDerivatives) {
+    BicubicSplineFunction(double[] coeff, boolean initializeDerivatives) {
         a = new double[N][N];
         for (int i = 0; i < N; i++) {
             for (int j = 0; j < N; j++) {

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/analysis/interpolation/InterpolatingMicrosphere.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/analysis/interpolation/InterpolatingMicrosphere.java b/src/main/java/org/apache/commons/math3/analysis/interpolation/InterpolatingMicrosphere.java
index 7cc6971..dc600bd 100644
--- a/src/main/java/org/apache/commons/math3/analysis/interpolation/InterpolatingMicrosphere.java
+++ b/src/main/java/org/apache/commons/math3/analysis/interpolation/InterpolatingMicrosphere.java
@@ -334,7 +334,7 @@ public class InterpolatingMicrosphere {
          * @param n Normal vector characterizing a surface element
          * of the microsphere. No copy is made.
          */
-        public Facet(double[] n) {
+        Facet(double[] n) {
             normal = n;
         }
 
@@ -361,8 +361,7 @@ public class InterpolatingMicrosphere {
          * @param illumination Illumination.
          * @param sample Data value.
          */
-        public FacetData(double illumination,
-                         double sample) {
+        FacetData(double illumination, double sample) {
             this.illumination = illumination;
             this.sample = sample;
         }

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/analysis/interpolation/TricubicInterpolatingFunction.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/analysis/interpolation/TricubicInterpolatingFunction.java b/src/main/java/org/apache/commons/math3/analysis/interpolation/TricubicInterpolatingFunction.java
index 9344d89..11b66d6 100644
--- a/src/main/java/org/apache/commons/math3/analysis/interpolation/TricubicInterpolatingFunction.java
+++ b/src/main/java/org/apache/commons/math3/analysis/interpolation/TricubicInterpolatingFunction.java
@@ -452,7 +452,7 @@ class TricubicFunction
     /**
      * @param aV List of spline coefficients.
      */
-    public TricubicFunction(double[] aV) {
+    TricubicFunction(double[] aV) {
         for (int i = 0; i < N; i++) {
             for (int j = 0; j < N; j++) {
                 for (int k = 0; k < N; k++) {

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/analysis/interpolation/TricubicSplineInterpolatingFunction.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/analysis/interpolation/TricubicSplineInterpolatingFunction.java b/src/main/java/org/apache/commons/math3/analysis/interpolation/TricubicSplineInterpolatingFunction.java
index 4260606..755d771 100644
--- a/src/main/java/org/apache/commons/math3/analysis/interpolation/TricubicSplineInterpolatingFunction.java
+++ b/src/main/java/org/apache/commons/math3/analysis/interpolation/TricubicSplineInterpolatingFunction.java
@@ -426,7 +426,7 @@ class TricubicSplineFunction
     /**
      * @param aV List of spline coefficients.
      */
-    public TricubicSplineFunction(double[] aV) {
+    TricubicSplineFunction(double[] aV) {
         for (int i = 0; i < N; i++) {
             for (int j = 0; j < N; j++) {
                 for (int k = 0; k < N; k++) {

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/analysis/polynomials/PolynomialsUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/analysis/polynomials/PolynomialsUtils.java b/src/main/java/org/apache/commons/math3/analysis/polynomials/PolynomialsUtils.java
index e6eccef..3f7cfec 100644
--- a/src/main/java/org/apache/commons/math3/analysis/polynomials/PolynomialsUtils.java
+++ b/src/main/java/org/apache/commons/math3/analysis/polynomials/PolynomialsUtils.java
@@ -264,7 +264,7 @@ public class PolynomialsUtils {
          * @param v first exponent
          * @param w second exponent
          */
-        public JacobiKey(final int v, final int w) {
+        JacobiKey(final int v, final int w) {
             this.v = v;
             this.w = w;
         }

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/exception/util/LocalizedFormats.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/exception/util/LocalizedFormats.java b/src/main/java/org/apache/commons/math3/exception/util/LocalizedFormats.java
index 7fc662f..36dba58 100644
--- a/src/main/java/org/apache/commons/math3/exception/util/LocalizedFormats.java
+++ b/src/main/java/org/apache/commons/math3/exception/util/LocalizedFormats.java
@@ -380,7 +380,7 @@ public enum LocalizedFormats implements Localizable {
      * @param sourceFormat source English format to use when no
      * localized version is available
      */
-    private LocalizedFormats(final String sourceFormat) {
+    LocalizedFormats(final String sourceFormat) {
         this.sourceFormat = sourceFormat;
     }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/fitting/CurveFitter.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/fitting/CurveFitter.java b/src/main/java/org/apache/commons/math3/fitting/CurveFitter.java
index 0ddeacc..239d9e8 100644
--- a/src/main/java/org/apache/commons/math3/fitting/CurveFitter.java
+++ b/src/main/java/org/apache/commons/math3/fitting/CurveFitter.java
@@ -190,7 +190,7 @@ public class CurveFitter<T extends ParametricUnivariateFunction> {
         /**
          * @param f function to fit.
          */
-        public TheoreticalValuesFunction(final ParametricUnivariateFunction f) {
+        TheoreticalValuesFunction(final ParametricUnivariateFunction f) {
             this.f = f;
         }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/fitting/leastsquares/GaussNewtonOptimizer.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/fitting/leastsquares/GaussNewtonOptimizer.java b/src/main/java/org/apache/commons/math3/fitting/leastsquares/GaussNewtonOptimizer.java
index bc4503c..dc758c1 100644
--- a/src/main/java/org/apache/commons/math3/fitting/leastsquares/GaussNewtonOptimizer.java
+++ b/src/main/java/org/apache/commons/math3/fitting/leastsquares/GaussNewtonOptimizer.java
@@ -50,7 +50,7 @@ public class GaussNewtonOptimizer implements LeastSquaresOptimizer {
 
     /** The decomposition algorithm to use to solve the normal equations. */
     //TODO move to linear package and expand options?
-    public static enum Decomposition {
+    public enum Decomposition {
         /**
          * Solve by forming the normal equations (J<sup>T</sup>Jx=J<sup>T</sup>r) and
          * using the {@link LUDecomposition}.

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/euclidean/oned/IntervalsSet.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/euclidean/oned/IntervalsSet.java b/src/main/java/org/apache/commons/math3/geometry/euclidean/oned/IntervalsSet.java
index 383ea9f..5ce7edb 100644
--- a/src/main/java/org/apache/commons/math3/geometry/euclidean/oned/IntervalsSet.java
+++ b/src/main/java/org/apache/commons/math3/geometry/euclidean/oned/IntervalsSet.java
@@ -589,7 +589,7 @@ public class IntervalsSet extends AbstractRegion<Euclidean1D, Euclidean1D> imple
 
         /** Simple constructor.
          */
-        public SubIntervalsIterator() {
+        SubIntervalsIterator() {
 
             current = getFirstIntervalBoundary();
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/euclidean/threed/OutlineExtractor.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/euclidean/threed/OutlineExtractor.java b/src/main/java/org/apache/commons/math3/geometry/euclidean/threed/OutlineExtractor.java
index 328018b..0f8af88 100644
--- a/src/main/java/org/apache/commons/math3/geometry/euclidean/threed/OutlineExtractor.java
+++ b/src/main/java/org/apache/commons/math3/geometry/euclidean/threed/OutlineExtractor.java
@@ -130,7 +130,7 @@ public class OutlineExtractor {
         /** Simple constructor.
          * @param tolerance tolerance below which points are considered identical
          */
-        public BoundaryProjector(final double tolerance) {
+        BoundaryProjector(final double tolerance) {
             this.projected = new PolygonsSet(new BSPTree<Euclidean2D>(Boolean.FALSE), tolerance);
             this.tolerance = tolerance;
         }

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/euclidean/threed/PolyhedronsSet.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/euclidean/threed/PolyhedronsSet.java b/src/main/java/org/apache/commons/math3/geometry/euclidean/threed/PolyhedronsSet.java
index 7116c11..f190e22 100644
--- a/src/main/java/org/apache/commons/math3/geometry/euclidean/threed/PolyhedronsSet.java
+++ b/src/main/java/org/apache/commons/math3/geometry/euclidean/threed/PolyhedronsSet.java
@@ -443,7 +443,7 @@ public class PolyhedronsSet extends AbstractRegion<Euclidean3D, Euclidean2D> {
     private class FacetsContributionVisitor implements BSPTreeVisitor<Euclidean3D> {
 
         /** Simple constructor. */
-        public FacetsContributionVisitor() {
+        FacetsContributionVisitor() {
             setSize(0);
             setBarycenter((Point<Euclidean3D>) new Vector3D(0, 0, 0));
         }
@@ -626,7 +626,7 @@ public class PolyhedronsSet extends AbstractRegion<Euclidean3D, Euclidean2D> {
          * @param center center point of the rotation
          * @param rotation vectorial rotation
          */
-        public RotationTransform(final Vector3D center, final Rotation rotation) {
+        RotationTransform(final Vector3D center, final Rotation rotation) {
             this.center   = center;
             this.rotation = rotation;
         }
@@ -697,7 +697,7 @@ public class PolyhedronsSet extends AbstractRegion<Euclidean3D, Euclidean2D> {
         /** Build a translation transform.
          * @param translation translation vector
          */
-        public TranslationTransform(final Vector3D translation) {
+        TranslationTransform(final Vector3D translation) {
             this.translation = translation;
         }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/euclidean/threed/SphericalCoordinates.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/euclidean/threed/SphericalCoordinates.java b/src/main/java/org/apache/commons/math3/geometry/euclidean/threed/SphericalCoordinates.java
index 23d818e..016e0a0 100644
--- a/src/main/java/org/apache/commons/math3/geometry/euclidean/threed/SphericalCoordinates.java
+++ b/src/main/java/org/apache/commons/math3/geometry/euclidean/threed/SphericalCoordinates.java
@@ -377,7 +377,7 @@ public class SphericalCoordinates implements Serializable {
          * @param y ordinate
          * @param z height
          */
-        public DataTransferObject(final double x, final double y, final double z) {
+        DataTransferObject(final double x, final double y, final double z) {
             this.x = x;
             this.y = y;
             this.z = z;

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/euclidean/twod/Line.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/euclidean/twod/Line.java b/src/main/java/org/apache/commons/math3/geometry/euclidean/twod/Line.java
index b5587ad..e367278 100644
--- a/src/main/java/org/apache/commons/math3/geometry/euclidean/twod/Line.java
+++ b/src/main/java/org/apache/commons/math3/geometry/euclidean/twod/Line.java
@@ -514,8 +514,8 @@ public class Line implements Hyperplane<Euclidean2D>, Embedding<Euclidean2D, Euc
          * @exception MathIllegalArgumentException if the transform is non invertible
          * @since 3.6
          */
-        public LineTransform(final double cXX, final double cYX, final double cXY,
-                             final double cYY, final double cX1, final double cY1)
+        LineTransform(final double cXX, final double cYX, final double cXY,
+                      final double cYY, final double cX1, final double cY1)
             throws MathIllegalArgumentException {
 
             this.cXX = cXX;

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/euclidean/twod/NestedLoops.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/euclidean/twod/NestedLoops.java b/src/main/java/org/apache/commons/math3/geometry/euclidean/twod/NestedLoops.java
index a58caf1..929366a 100644
--- a/src/main/java/org/apache/commons/math3/geometry/euclidean/twod/NestedLoops.java
+++ b/src/main/java/org/apache/commons/math3/geometry/euclidean/twod/NestedLoops.java
@@ -69,7 +69,7 @@ class NestedLoops {
      * @param tolerance tolerance below which points are considered identical
      * @since 3.3
      */
-    public NestedLoops(final double tolerance) {
+    NestedLoops(final double tolerance) {
         this.surrounded = new ArrayList<NestedLoops>();
         this.tolerance  = tolerance;
     }

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/euclidean/twod/PolygonsSet.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/euclidean/twod/PolygonsSet.java b/src/main/java/org/apache/commons/math3/geometry/euclidean/twod/PolygonsSet.java
index 46268f5..61fae9f 100644
--- a/src/main/java/org/apache/commons/math3/geometry/euclidean/twod/PolygonsSet.java
+++ b/src/main/java/org/apache/commons/math3/geometry/euclidean/twod/PolygonsSet.java
@@ -426,7 +426,7 @@ public class PolygonsSet extends AbstractRegion<Euclidean2D, Euclidean1D> {
         /** Build a non-processed vertex not owned by any node yet.
          * @param location vertex location
          */
-        public Vertex(final Vector2D location) {
+        Vertex(final Vector2D location) {
             this.location = location;
             this.incoming = null;
             this.outgoing = null;
@@ -528,7 +528,7 @@ public class PolygonsSet extends AbstractRegion<Euclidean2D, Euclidean1D> {
          * @param end end vertex
          * @param line line supporting the edge
          */
-        public Edge(final Vertex start, final Vertex end, final Line line) {
+        Edge(final Vertex start, final Vertex end, final Line line) {
 
             this.start = start;
             this.end   = end;
@@ -971,10 +971,10 @@ public class PolygonsSet extends AbstractRegion<Euclidean2D, Euclidean1D> {
          * @param startNode node whose intersection with current node defines start point
          * @param endNode node whose intersection with current node defines end point
          */
-        public ConnectableSegment(final Vector2D start, final Vector2D end, final Line line,
-                                  final BSPTree<Euclidean2D> node,
-                                  final BSPTree<Euclidean2D> startNode,
-                                  final BSPTree<Euclidean2D> endNode) {
+        ConnectableSegment(final Vector2D start, final Vector2D end, final Line line,
+                           final BSPTree<Euclidean2D> node,
+                           final BSPTree<Euclidean2D> startNode,
+                           final BSPTree<Euclidean2D> endNode) {
             super(start, end, line);
             this.node      = node;
             this.startNode = startNode;
@@ -1061,7 +1061,7 @@ public class PolygonsSet extends AbstractRegion<Euclidean2D, Euclidean1D> {
         /** Simple constructor.
          * @param tolerance tolerance for close nodes connection
          */
-        public SegmentsBuilder(final double tolerance) {
+        SegmentsBuilder(final double tolerance) {
             this.tolerance = tolerance;
             this.segments  = new ArrayList<ConnectableSegment>();
         }

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/partitioning/BoundaryProjector.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/partitioning/BoundaryProjector.java b/src/main/java/org/apache/commons/math3/geometry/partitioning/BoundaryProjector.java
index 9f660bf..4fbf4b9 100644
--- a/src/main/java/org/apache/commons/math3/geometry/partitioning/BoundaryProjector.java
+++ b/src/main/java/org/apache/commons/math3/geometry/partitioning/BoundaryProjector.java
@@ -46,7 +46,7 @@ class BoundaryProjector<S extends Space, T extends Space> implements BSPTreeVisi
     /** Simple constructor.
      * @param original original point
      */
-    public BoundaryProjector(final Point<S> original) {
+    BoundaryProjector(final Point<S> original) {
         this.original  = original;
         this.projected = null;
         this.leaf      = null;

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/partitioning/BoundarySizeVisitor.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/partitioning/BoundarySizeVisitor.java b/src/main/java/org/apache/commons/math3/geometry/partitioning/BoundarySizeVisitor.java
index e29083a..054838b 100644
--- a/src/main/java/org/apache/commons/math3/geometry/partitioning/BoundarySizeVisitor.java
+++ b/src/main/java/org/apache/commons/math3/geometry/partitioning/BoundarySizeVisitor.java
@@ -29,7 +29,7 @@ class BoundarySizeVisitor<S extends Space> implements BSPTreeVisitor<S> {
 
     /** Simple constructor.
      */
-    public BoundarySizeVisitor() {
+    BoundarySizeVisitor() {
         boundarySize = 0;
     }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/partitioning/Characterization.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/partitioning/Characterization.java b/src/main/java/org/apache/commons/math3/geometry/partitioning/Characterization.java
index ca43be6..f8ec2f9 100644
--- a/src/main/java/org/apache/commons/math3/geometry/partitioning/Characterization.java
+++ b/src/main/java/org/apache/commons/math3/geometry/partitioning/Characterization.java
@@ -53,7 +53,7 @@ class Characterization<S extends Space> {
      * @param node current BSP tree node
      * @param sub sub-hyperplane to characterize
      */
-    public Characterization(final BSPTree<S> node, final SubHyperplane<S> sub) {
+    Characterization(final BSPTree<S> node, final SubHyperplane<S> sub) {
         outsideTouching  = null;
         insideTouching   = null;
         outsideSplitters = new NodesSet<S>();

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/partitioning/InsideFinder.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/partitioning/InsideFinder.java b/src/main/java/org/apache/commons/math3/geometry/partitioning/InsideFinder.java
index 82a6d9e..b1db90a 100644
--- a/src/main/java/org/apache/commons/math3/geometry/partitioning/InsideFinder.java
+++ b/src/main/java/org/apache/commons/math3/geometry/partitioning/InsideFinder.java
@@ -37,7 +37,7 @@ class InsideFinder<S extends Space> {
     /** Simple constructor.
      * @param region region on which to operate
      */
-    public InsideFinder(final Region<S> region) {
+    InsideFinder(final Region<S> region) {
         this.region = region;
         plusFound  = false;
         minusFound = false;

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/partitioning/Region.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/partitioning/Region.java b/src/main/java/org/apache/commons/math3/geometry/partitioning/Region.java
index f7bc3aa..9ff3946 100644
--- a/src/main/java/org/apache/commons/math3/geometry/partitioning/Region.java
+++ b/src/main/java/org/apache/commons/math3/geometry/partitioning/Region.java
@@ -53,7 +53,7 @@ import org.apache.commons.math3.geometry.Point;
 public interface Region<S extends Space> {
 
     /** Enumerate for the location of a point with respect to the region. */
-    public static enum Location {
+    enum Location {
         /** Code for points inside the partition. */
         INSIDE,
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/partitioning/RegionFactory.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/partitioning/RegionFactory.java b/src/main/java/org/apache/commons/math3/geometry/partitioning/RegionFactory.java
index 0f46848..86605be 100644
--- a/src/main/java/org/apache/commons/math3/geometry/partitioning/RegionFactory.java
+++ b/src/main/java/org/apache/commons/math3/geometry/partitioning/RegionFactory.java
@@ -295,7 +295,7 @@ public class RegionFactory<S extends Space> {
          * @param region1 region to subtract from
          * @param region2 region to subtract
          */
-        public DifferenceMerger(final Region<S> region1, final Region<S> region2) {
+        DifferenceMerger(final Region<S> region1, final Region<S> region2) {
             this.region1 = region1.copySelf();
             this.region2 = region2.copySelf();
         }
@@ -358,7 +358,7 @@ public class RegionFactory<S extends Space> {
         /** Simple constructor.
          * @param inside inside/outside indicator to use for ambiguous nodes
          */
-        public VanishingToLeaf(final boolean inside) {
+        VanishingToLeaf(final boolean inside) {
             this.inside = inside;
         }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/partitioning/SubHyperplane.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/partitioning/SubHyperplane.java b/src/main/java/org/apache/commons/math3/geometry/partitioning/SubHyperplane.java
index 9b6625b..2069f6f 100644
--- a/src/main/java/org/apache/commons/math3/geometry/partitioning/SubHyperplane.java
+++ b/src/main/java/org/apache/commons/math3/geometry/partitioning/SubHyperplane.java
@@ -94,7 +94,7 @@ public interface SubHyperplane<S extends Space> {
     /** Class holding the results of the {@link #split split} method.
      * @param <U> Type of the embedding space.
      */
-    public static class SplitSubHyperplane<U extends Space> {
+    class SplitSubHyperplane<U extends Space> {
 
         /** Part of the sub-hyperplane on the plus side of the splitting hyperplane. */
         private final SubHyperplane<U> plus;

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/partitioning/utilities/AVLTree.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/partitioning/utilities/AVLTree.java b/src/main/java/org/apache/commons/math3/geometry/partitioning/utilities/AVLTree.java
index 9412421..00c9d3e 100644
--- a/src/main/java/org/apache/commons/math3/geometry/partitioning/utilities/AVLTree.java
+++ b/src/main/java/org/apache/commons/math3/geometry/partitioning/utilities/AVLTree.java
@@ -201,7 +201,7 @@ public class AVLTree<T extends Comparable<T>> {
     }
 
     /** Enum for tree skew factor. */
-    private static enum Skew {
+    private enum Skew {
         /** Code for left high trees. */
         LEFT_HIGH,
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/spherical/oned/ArcsSet.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/spherical/oned/ArcsSet.java b/src/main/java/org/apache/commons/math3/geometry/spherical/oned/ArcsSet.java
index c0ca6c1..0a00aa7 100644
--- a/src/main/java/org/apache/commons/math3/geometry/spherical/oned/ArcsSet.java
+++ b/src/main/java/org/apache/commons/math3/geometry/spherical/oned/ArcsSet.java
@@ -598,7 +598,7 @@ public class ArcsSet extends AbstractRegion<Sphere1D, Sphere1D> implements Itera
 
         /** Simple constructor.
          */
-        public SubArcsIterator() {
+        SubArcsIterator() {
 
             firstStart = getFirstArcStart();
             current    = firstStart;

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/spherical/twod/Circle.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/spherical/twod/Circle.java b/src/main/java/org/apache/commons/math3/geometry/spherical/twod/Circle.java
index d6d47de..a34db6d 100644
--- a/src/main/java/org/apache/commons/math3/geometry/spherical/twod/Circle.java
+++ b/src/main/java/org/apache/commons/math3/geometry/spherical/twod/Circle.java
@@ -295,7 +295,7 @@ public class Circle implements Hyperplane<Sphere2D>, Embedding<Sphere2D, Sphere1
         /** Build a transform from a {@code Rotation}.
          * @param rotation rotation to use
          */
-        public CircleTransform(final Rotation rotation) {
+        CircleTransform(final Rotation rotation) {
             this.rotation = rotation;
         }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/spherical/twod/EdgesBuilder.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/spherical/twod/EdgesBuilder.java b/src/main/java/org/apache/commons/math3/geometry/spherical/twod/EdgesBuilder.java
index 4aa712c..844cfb1 100644
--- a/src/main/java/org/apache/commons/math3/geometry/spherical/twod/EdgesBuilder.java
+++ b/src/main/java/org/apache/commons/math3/geometry/spherical/twod/EdgesBuilder.java
@@ -52,7 +52,7 @@ class EdgesBuilder implements BSPTreeVisitor<Sphere2D> {
      * @param root tree root
      * @param tolerance below which points are consider to be identical
      */
-    public EdgesBuilder(final BSPTree<Sphere2D> root, final double tolerance) {
+    EdgesBuilder(final BSPTree<Sphere2D> root, final double tolerance) {
         this.root            = root;
         this.tolerance       = tolerance;
         this.edgeToNode      = new IdentityHashMap<Edge, BSPTree<Sphere2D>>();

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/geometry/spherical/twod/PropertiesComputer.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/geometry/spherical/twod/PropertiesComputer.java b/src/main/java/org/apache/commons/math3/geometry/spherical/twod/PropertiesComputer.java
index 9d15b11..593180f 100644
--- a/src/main/java/org/apache/commons/math3/geometry/spherical/twod/PropertiesComputer.java
+++ b/src/main/java/org/apache/commons/math3/geometry/spherical/twod/PropertiesComputer.java
@@ -44,9 +44,9 @@ class PropertiesComputer implements BSPTreeVisitor<Sphere2D> {
     private final List<Vector3D> convexCellsInsidePoints;
 
     /** Simple constructor.
- * @param tolerance below which points are consider to be identical
+     * @param tolerance below which points are consider to be identical
      */
-    public PropertiesComputer(final double tolerance) {
+    PropertiesComputer(final double tolerance) {
         this.tolerance              = tolerance;
         this.summedArea             = 0;
         this.summedBarycenter       = Vector3D.ZERO;

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/linear/BiDiagonalTransformer.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/linear/BiDiagonalTransformer.java b/src/main/java/org/apache/commons/math3/linear/BiDiagonalTransformer.java
index cfbcb68..4a450e1 100644
--- a/src/main/java/org/apache/commons/math3/linear/BiDiagonalTransformer.java
+++ b/src/main/java/org/apache/commons/math3/linear/BiDiagonalTransformer.java
@@ -58,7 +58,7 @@ class BiDiagonalTransformer {
      * Build the transformation to bi-diagonal shape of a matrix.
      * @param matrix the matrix to transform.
      */
-    public BiDiagonalTransformer(RealMatrix matrix) {
+    BiDiagonalTransformer(RealMatrix matrix) {
 
         final int m = matrix.getRowDimension();
         final int n = matrix.getColumnDimension();

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/linear/HessenbergTransformer.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/linear/HessenbergTransformer.java b/src/main/java/org/apache/commons/math3/linear/HessenbergTransformer.java
index 8fefc09..d1caa46 100644
--- a/src/main/java/org/apache/commons/math3/linear/HessenbergTransformer.java
+++ b/src/main/java/org/apache/commons/math3/linear/HessenbergTransformer.java
@@ -56,7 +56,7 @@ class HessenbergTransformer {
      * @param matrix matrix to transform
      * @throws NonSquareMatrixException if the matrix is not square
      */
-    public HessenbergTransformer(final RealMatrix matrix) {
+    HessenbergTransformer(final RealMatrix matrix) {
         if (!matrix.isSquare()) {
             throw new NonSquareMatrixException(matrix.getRowDimension(),
                     matrix.getColumnDimension());

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/linear/MatrixUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/linear/MatrixUtils.java b/src/main/java/org/apache/commons/math3/linear/MatrixUtils.java
index 23c11e0..08bcfc9 100644
--- a/src/main/java/org/apache/commons/math3/linear/MatrixUtils.java
+++ b/src/main/java/org/apache/commons/math3/linear/MatrixUtils.java
@@ -625,7 +625,7 @@ public class MatrixUtils {
         /** Converted array. */
         private double[][] data;
         /** Simple constructor. */
-        public FractionMatrixConverter() {
+        FractionMatrixConverter() {
             super(Fraction.ZERO);
         }
 
@@ -670,7 +670,7 @@ public class MatrixUtils {
         /** Converted array. */
         private double[][] data;
         /** Simple constructor. */
-        public BigFractionMatrixConverter() {
+        BigFractionMatrixConverter() {
             super(BigFraction.ZERO);
         }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/linear/SchurTransformer.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/linear/SchurTransformer.java b/src/main/java/org/apache/commons/math3/linear/SchurTransformer.java
index cd54ca4..6cdf876 100644
--- a/src/main/java/org/apache/commons/math3/linear/SchurTransformer.java
+++ b/src/main/java/org/apache/commons/math3/linear/SchurTransformer.java
@@ -65,7 +65,7 @@ class SchurTransformer {
      * @param matrix matrix to transform
      * @throws NonSquareMatrixException if the matrix is not square
      */
-    public SchurTransformer(final RealMatrix matrix) {
+    SchurTransformer(final RealMatrix matrix) {
         if (!matrix.isSquare()) {
             throw new NonSquareMatrixException(matrix.getRowDimension(),
                                                matrix.getColumnDimension());

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/linear/SymmLQ.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/linear/SymmLQ.java b/src/main/java/org/apache/commons/math3/linear/SymmLQ.java
index e529a6a..03a4bed 100644
--- a/src/main/java/org/apache/commons/math3/linear/SymmLQ.java
+++ b/src/main/java/org/apache/commons/math3/linear/SymmLQ.java
@@ -380,7 +380,7 @@ public class SymmLQ
          * @param check {@code true} if self-adjointedness of both matrix and
          * preconditioner should be checked
          */
-        public State(final RealLinearOperator a,
+        State(final RealLinearOperator a,
             final RealLinearOperator m,
             final RealVector b,
             final boolean goodb,

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/linear/TriDiagonalTransformer.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/linear/TriDiagonalTransformer.java b/src/main/java/org/apache/commons/math3/linear/TriDiagonalTransformer.java
index 9d2f986..c79f9de 100644
--- a/src/main/java/org/apache/commons/math3/linear/TriDiagonalTransformer.java
+++ b/src/main/java/org/apache/commons/math3/linear/TriDiagonalTransformer.java
@@ -58,7 +58,7 @@ class TriDiagonalTransformer {
      * @param matrix Symmetrical matrix to transform.
      * @throws NonSquareMatrixException if the matrix is not square.
      */
-    public TriDiagonalTransformer(RealMatrix matrix) {
+    TriDiagonalTransformer(RealMatrix matrix) {
         if (!matrix.isSquare()) {
             throw new NonSquareMatrixException(matrix.getRowDimension(),
                                                matrix.getColumnDimension());

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ml/clustering/KMeansPlusPlusClusterer.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ml/clustering/KMeansPlusPlusClusterer.java b/src/main/java/org/apache/commons/math3/ml/clustering/KMeansPlusPlusClusterer.java
index 2e68c2d..2e57fac 100644
--- a/src/main/java/org/apache/commons/math3/ml/clustering/KMeansPlusPlusClusterer.java
+++ b/src/main/java/org/apache/commons/math3/ml/clustering/KMeansPlusPlusClusterer.java
@@ -42,7 +42,7 @@ import org.apache.commons.math3.util.MathUtils;
 public class KMeansPlusPlusClusterer<T extends Clusterable> extends Clusterer<T> {
 
     /** Strategies to use for replacing an empty cluster. */
-    public static enum EmptyClusterStrategy {
+    public enum EmptyClusterStrategy {
 
         /** Split the cluster with largest distance variance. */
         LARGEST_VARIANCE,

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ml/neuralnet/MapUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ml/neuralnet/MapUtils.java b/src/main/java/org/apache/commons/math3/ml/neuralnet/MapUtils.java
index f5ecf02..83036c2 100644
--- a/src/main/java/org/apache/commons/math3/ml/neuralnet/MapUtils.java
+++ b/src/main/java/org/apache/commons/math3/ml/neuralnet/MapUtils.java
@@ -301,8 +301,7 @@ public class MapUtils {
          * @param neuron Neuron.
          * @param value Value.
          */
-        public PairNeuronDouble(Neuron neuron,
-                                double value) {
+        PairNeuronDouble(Neuron neuron, double value) {
             this.neuron = neuron;
             this.value = value;
         }

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/ExpandableStatefulODE.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/ExpandableStatefulODE.java b/src/main/java/org/apache/commons/math3/ode/ExpandableStatefulODE.java
index dc7d4a9..72d3dcb 100644
--- a/src/main/java/org/apache/commons/math3/ode/ExpandableStatefulODE.java
+++ b/src/main/java/org/apache/commons/math3/ode/ExpandableStatefulODE.java
@@ -317,7 +317,7 @@ public class ExpandableStatefulODE {
          * @param equation secondary differential equation
          * @param firstIndex index to use for the first element in the complete arrays
          */
-        public SecondaryComponent(final SecondaryEquations equation, final int firstIndex) {
+        SecondaryComponent(final SecondaryEquations equation, final int firstIndex) {
             final int n   = equation.getDimension();
             this.equation = equation;
             mapper        = new EquationsMapper(firstIndex, n);

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/JacobianMatrices.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/JacobianMatrices.java b/src/main/java/org/apache/commons/math3/ode/JacobianMatrices.java
index 0ce552d..3710a6b 100644
--- a/src/main/java/org/apache/commons/math3/ode/JacobianMatrices.java
+++ b/src/main/java/org/apache/commons/math3/ode/JacobianMatrices.java
@@ -431,8 +431,8 @@ public class JacobianMatrices {
          * @exception DimensionMismatchException if there is a dimension mismatch between
          * the steps array {@code hY} and the equation dimension
          */
-        public MainStateJacobianWrapper(final FirstOrderDifferentialEquations ode,
-                                        final double[] hY)
+        MainStateJacobianWrapper(final FirstOrderDifferentialEquations ode,
+                                 final double[] hY)
             throws DimensionMismatchException {
             this.ode = ode;
             this.hY = hY.clone();

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/MultistepIntegrator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/MultistepIntegrator.java b/src/main/java/org/apache/commons/math3/ode/MultistepIntegrator.java
index 0fad1a2..17baed7 100644
--- a/src/main/java/org/apache/commons/math3/ode/MultistepIntegrator.java
+++ b/src/main/java/org/apache/commons/math3/ode/MultistepIntegrator.java
@@ -355,7 +355,7 @@ public abstract class MultistepIntegrator extends AdaptiveStepsizeIntegrator {
          * @param nSteps number of steps of the multistep method (excluding the one being computed)
          * @param n problem dimension
          */
-        public NordsieckInitializer(final int nSteps, final int n) {
+        NordsieckInitializer(final int nSteps, final int n) {
             this.count = 0;
             this.t     = new double[nSteps];
             this.y     = new double[nSteps][n];
@@ -438,7 +438,7 @@ public abstract class MultistepIntegrator extends AdaptiveStepsizeIntegrator {
         private static final long serialVersionUID = -1914085471038046418L;
 
         /** Simple constructor. */
-        public InitializationCompletedMarkerException() {
+        InitializationCompletedMarkerException() {
             super((Throwable) null);
         }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/ParameterConfiguration.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/ParameterConfiguration.java b/src/main/java/org/apache/commons/math3/ode/ParameterConfiguration.java
index 7af2cd5..bca7e68 100644
--- a/src/main/java/org/apache/commons/math3/ode/ParameterConfiguration.java
+++ b/src/main/java/org/apache/commons/math3/ode/ParameterConfiguration.java
@@ -38,7 +38,7 @@ class ParameterConfiguration implements Serializable {
      * @param parameterName parameter name
      * @param hP parameter step
      */
-    public ParameterConfiguration(final String parameterName, final double hP) {
+    ParameterConfiguration(final String parameterName, final double hP) {
         this.parameterName = parameterName;
         this.hP = hP;
     }

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/ParameterJacobianWrapper.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/ParameterJacobianWrapper.java b/src/main/java/org/apache/commons/math3/ode/ParameterJacobianWrapper.java
index 40fcd97..1d3936f 100644
--- a/src/main/java/org/apache/commons/math3/ode/ParameterJacobianWrapper.java
+++ b/src/main/java/org/apache/commons/math3/ode/ParameterJacobianWrapper.java
@@ -46,9 +46,9 @@ class ParameterJacobianWrapper implements ParameterJacobianProvider {
      * @param paramsAndSteps parameters and steps to compute the Jacobians df/dp
      * @see JacobianMatrices#setParameterStep(String, double)
      */
-    public ParameterJacobianWrapper(final FirstOrderDifferentialEquations fode,
-                                    final ParameterizedODE pode,
-                                    final ParameterConfiguration[] paramsAndSteps) {
+    ParameterJacobianWrapper(final FirstOrderDifferentialEquations fode,
+                             final ParameterizedODE pode,
+                             final ParameterConfiguration[] paramsAndSteps) {
         this.fode = fode;
         this.pode = pode;
         this.hParam = new HashMap<String, Double>();

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/ParameterizedWrapper.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/ParameterizedWrapper.java b/src/main/java/org/apache/commons/math3/ode/ParameterizedWrapper.java
index 1a0bad6..74c8e99 100644
--- a/src/main/java/org/apache/commons/math3/ode/ParameterizedWrapper.java
+++ b/src/main/java/org/apache/commons/math3/ode/ParameterizedWrapper.java
@@ -35,7 +35,7 @@ class ParameterizedWrapper implements ParameterizedODE {
     /** Simple constructor.
      * @param ode original first order differential equations
      */
-    public ParameterizedWrapper(final FirstOrderDifferentialEquations ode) {
+    ParameterizedWrapper(final FirstOrderDifferentialEquations ode) {
         this.fode = ode;
     }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/events/EventHandler.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/events/EventHandler.java b/src/main/java/org/apache/commons/math3/ode/events/EventHandler.java
index 8fb39a4..58533bc 100644
--- a/src/main/java/org/apache/commons/math3/ode/events/EventHandler.java
+++ b/src/main/java/org/apache/commons/math3/ode/events/EventHandler.java
@@ -50,7 +50,7 @@ package org.apache.commons.math3.ode.events;
 public interface EventHandler  {
 
     /** Enumerate for actions to be performed when an event occurs. */
-    public enum Action {
+    enum Action {
 
         /** Stop indicator.
          * <p>This value should be used as the return value of the {@link

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/events/EventState.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/events/EventState.java b/src/main/java/org/apache/commons/math3/ode/events/EventState.java
index cf64f36..b838654 100644
--- a/src/main/java/org/apache/commons/math3/ode/events/EventState.java
+++ b/src/main/java/org/apache/commons/math3/ode/events/EventState.java
@@ -414,7 +414,7 @@ public class EventState {
         /** Simple constructor.
          * @param exception exception to wrap
          */
-        public LocalMaxCountExceededException(final MaxCountExceededException exception) {
+        LocalMaxCountExceededException(final MaxCountExceededException exception) {
             wrapped = exception;
         }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/nonstiff/AdamsMoultonIntegrator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/nonstiff/AdamsMoultonIntegrator.java b/src/main/java/org/apache/commons/math3/ode/nonstiff/AdamsMoultonIntegrator.java
index c019fab..c12c19b 100644
--- a/src/main/java/org/apache/commons/math3/ode/nonstiff/AdamsMoultonIntegrator.java
+++ b/src/main/java/org/apache/commons/math3/ode/nonstiff/AdamsMoultonIntegrator.java
@@ -367,7 +367,7 @@ public class AdamsMoultonIntegrator extends AdamsIntegrator {
          * @param scaled current scaled first derivative
          * @param state state to correct (will be overwritten after visit)
          */
-        public Corrector(final double[] previous, final double[] scaled, final double[] state) {
+        Corrector(final double[] previous, final double[] scaled, final double[] state) {
             this.previous = previous;
             this.scaled   = scaled;
             this.after    = state;

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/nonstiff/ClassicalRungeKuttaStepInterpolator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/nonstiff/ClassicalRungeKuttaStepInterpolator.java b/src/main/java/org/apache/commons/math3/ode/nonstiff/ClassicalRungeKuttaStepInterpolator.java
index ba9b11f..5825cc4 100644
--- a/src/main/java/org/apache/commons/math3/ode/nonstiff/ClassicalRungeKuttaStepInterpolator.java
+++ b/src/main/java/org/apache/commons/math3/ode/nonstiff/ClassicalRungeKuttaStepInterpolator.java
@@ -68,7 +68,7 @@ class ClassicalRungeKuttaStepInterpolator
      * interpolators by cloning an uninitialized model and latter initializing
      * the copy.
      */
-    public ClassicalRungeKuttaStepInterpolator() {
+    ClassicalRungeKuttaStepInterpolator() {
     }
 
     /** Copy constructor.
@@ -76,7 +76,7 @@ class ClassicalRungeKuttaStepInterpolator
      * copy: its arrays are separated from the original arrays of the
      * instance
      */
-    public ClassicalRungeKuttaStepInterpolator(final ClassicalRungeKuttaStepInterpolator interpolator) {
+    ClassicalRungeKuttaStepInterpolator(final ClassicalRungeKuttaStepInterpolator interpolator) {
         super(interpolator);
     }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/nonstiff/DormandPrince54StepInterpolator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/nonstiff/DormandPrince54StepInterpolator.java b/src/main/java/org/apache/commons/math3/ode/nonstiff/DormandPrince54StepInterpolator.java
index a013584..b62dc0c 100644
--- a/src/main/java/org/apache/commons/math3/ode/nonstiff/DormandPrince54StepInterpolator.java
+++ b/src/main/java/org/apache/commons/math3/ode/nonstiff/DormandPrince54StepInterpolator.java
@@ -97,7 +97,7 @@ class DormandPrince54StepInterpolator
    * prototyping design pattern to create the step interpolators by
    * cloning an uninitialized model and latter initializing the copy.
    */
-  public DormandPrince54StepInterpolator() {
+  DormandPrince54StepInterpolator() {
     super();
     v1 = null;
     v2 = null;
@@ -111,7 +111,7 @@ class DormandPrince54StepInterpolator
    * copy: its arrays are separated from the original arrays of the
    * instance
    */
-  public DormandPrince54StepInterpolator(final DormandPrince54StepInterpolator interpolator) {
+  DormandPrince54StepInterpolator(final DormandPrince54StepInterpolator interpolator) {
 
     super(interpolator);
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/nonstiff/DormandPrince853StepInterpolator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/nonstiff/DormandPrince853StepInterpolator.java b/src/main/java/org/apache/commons/math3/ode/nonstiff/DormandPrince853StepInterpolator.java
index 3ff3a85..443d106 100644
--- a/src/main/java/org/apache/commons/math3/ode/nonstiff/DormandPrince853StepInterpolator.java
+++ b/src/main/java/org/apache/commons/math3/ode/nonstiff/DormandPrince853StepInterpolator.java
@@ -227,7 +227,7 @@ class DormandPrince853StepInterpolator
    * prototyping design pattern to create the step interpolators by
    * cloning an uninitialized model and latter initializing the copy.
    */
-  public DormandPrince853StepInterpolator() {
+  DormandPrince853StepInterpolator() {
     super();
     yDotKLast = null;
     v         = null;
@@ -239,7 +239,7 @@ class DormandPrince853StepInterpolator
    * copy: its arrays are separated from the original arrays of the
    * instance
    */
-  public DormandPrince853StepInterpolator(final DormandPrince853StepInterpolator interpolator) {
+  DormandPrince853StepInterpolator(final DormandPrince853StepInterpolator interpolator) {
 
     super(interpolator);
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/nonstiff/EulerStepInterpolator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/nonstiff/EulerStepInterpolator.java b/src/main/java/org/apache/commons/math3/ode/nonstiff/EulerStepInterpolator.java
index 3c45780..7a5063d 100644
--- a/src/main/java/org/apache/commons/math3/ode/nonstiff/EulerStepInterpolator.java
+++ b/src/main/java/org/apache/commons/math3/ode/nonstiff/EulerStepInterpolator.java
@@ -59,7 +59,7 @@ class EulerStepInterpolator
    * to create the step interpolators by cloning an uninitialized model
    * and later initializing the copy.
    */
-  public EulerStepInterpolator() {
+  EulerStepInterpolator() {
   }
 
   /** Copy constructor.
@@ -67,7 +67,7 @@ class EulerStepInterpolator
    * copy: its arrays are separated from the original arrays of the
    * instance
    */
-  public EulerStepInterpolator(final EulerStepInterpolator interpolator) {
+  EulerStepInterpolator(final EulerStepInterpolator interpolator) {
     super(interpolator);
   }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/nonstiff/GillStepInterpolator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/nonstiff/GillStepInterpolator.java b/src/main/java/org/apache/commons/math3/ode/nonstiff/GillStepInterpolator.java
index 49081f5..85d8fff 100644
--- a/src/main/java/org/apache/commons/math3/ode/nonstiff/GillStepInterpolator.java
+++ b/src/main/java/org/apache/commons/math3/ode/nonstiff/GillStepInterpolator.java
@@ -75,7 +75,7 @@ class GillStepInterpolator
    * to create the step interpolators by cloning an uninitialized model
    * and later initializing the copy.
    */
-  public GillStepInterpolator() {
+  GillStepInterpolator() {
   }
 
   /** Copy constructor.
@@ -83,7 +83,7 @@ class GillStepInterpolator
    * copy: its arrays are separated from the original arrays of the
    * instance
    */
-  public GillStepInterpolator(final GillStepInterpolator interpolator) {
+  GillStepInterpolator(final GillStepInterpolator interpolator) {
     super(interpolator);
   }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/nonstiff/GraggBulirschStoerStepInterpolator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/nonstiff/GraggBulirschStoerStepInterpolator.java b/src/main/java/org/apache/commons/math3/ode/nonstiff/GraggBulirschStoerStepInterpolator.java
index c1d61b3..b70f896 100644
--- a/src/main/java/org/apache/commons/math3/ode/nonstiff/GraggBulirschStoerStepInterpolator.java
+++ b/src/main/java/org/apache/commons/math3/ode/nonstiff/GraggBulirschStoerStepInterpolator.java
@@ -107,7 +107,7 @@ class GraggBulirschStoerStepInterpolator
    * This constructor should not be used directly, it is only intended
    * for the serialization process.
    */
-  public GraggBulirschStoerStepInterpolator() {
+  GraggBulirschStoerStepInterpolator() {
     y0Dot    = null;
     y1       = null;
     y1Dot    = null;
@@ -129,12 +129,12 @@ class GraggBulirschStoerStepInterpolator
    * @param primaryMapper equations mapper for the primary equations set
    * @param secondaryMappers equations mappers for the secondary equations sets
    */
-  public GraggBulirschStoerStepInterpolator(final double[] y, final double[] y0Dot,
-                                            final double[] y1, final double[] y1Dot,
-                                            final double[][] yMidDots,
-                                            final boolean forward,
-                                            final EquationsMapper primaryMapper,
-                                            final EquationsMapper[] secondaryMappers) {
+  GraggBulirschStoerStepInterpolator(final double[] y, final double[] y0Dot,
+                                     final double[] y1, final double[] y1Dot,
+                                     final double[][] yMidDots,
+                                     final boolean forward,
+                                     final EquationsMapper primaryMapper,
+                                     final EquationsMapper[] secondaryMappers) {
 
     super(y, forward, primaryMapper, secondaryMappers);
     this.y0Dot    = y0Dot;
@@ -151,8 +151,7 @@ class GraggBulirschStoerStepInterpolator
    * copy: its arrays are separated from the original arrays of the
    * instance
    */
-  public GraggBulirschStoerStepInterpolator
-    (final GraggBulirschStoerStepInterpolator interpolator) {
+  GraggBulirschStoerStepInterpolator(final GraggBulirschStoerStepInterpolator interpolator) {
 
     super(interpolator);
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/nonstiff/HighamHall54StepInterpolator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/nonstiff/HighamHall54StepInterpolator.java b/src/main/java/org/apache/commons/math3/ode/nonstiff/HighamHall54StepInterpolator.java
index cecbafb..36acb42 100644
--- a/src/main/java/org/apache/commons/math3/ode/nonstiff/HighamHall54StepInterpolator.java
+++ b/src/main/java/org/apache/commons/math3/ode/nonstiff/HighamHall54StepInterpolator.java
@@ -45,7 +45,7 @@ class HighamHall54StepInterpolator
    * to create the step interpolators by cloning an uninitialized model
    * and later initializing the copy.
    */
-  public HighamHall54StepInterpolator() {
+  HighamHall54StepInterpolator() {
     super();
   }
 
@@ -54,7 +54,7 @@ class HighamHall54StepInterpolator
    * copy: its arrays are separated from the original arrays of the
    * instance
    */
-  public HighamHall54StepInterpolator(final HighamHall54StepInterpolator interpolator) {
+  HighamHall54StepInterpolator(final HighamHall54StepInterpolator interpolator) {
     super(interpolator);
   }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/nonstiff/LutherStepInterpolator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/nonstiff/LutherStepInterpolator.java b/src/main/java/org/apache/commons/math3/ode/nonstiff/LutherStepInterpolator.java
index 6f54b70..86f22af 100644
--- a/src/main/java/org/apache/commons/math3/ode/nonstiff/LutherStepInterpolator.java
+++ b/src/main/java/org/apache/commons/math3/ode/nonstiff/LutherStepInterpolator.java
@@ -51,7 +51,7 @@ class LutherStepInterpolator extends RungeKuttaStepInterpolator {
      * to create the step interpolators by cloning an uninitialized model
      * and later initializing the copy.
      */
-    public LutherStepInterpolator() {
+    LutherStepInterpolator() {
     }
 
     /** Copy constructor.
@@ -59,7 +59,7 @@ class LutherStepInterpolator extends RungeKuttaStepInterpolator {
      * copy: its arrays are separated from the original arrays of the
      * instance
      */
-    public LutherStepInterpolator(final LutherStepInterpolator interpolator) {
+    LutherStepInterpolator(final LutherStepInterpolator interpolator) {
         super(interpolator);
     }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/nonstiff/MidpointStepInterpolator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/nonstiff/MidpointStepInterpolator.java b/src/main/java/org/apache/commons/math3/ode/nonstiff/MidpointStepInterpolator.java
index 7346f7a..ac3cddb 100644
--- a/src/main/java/org/apache/commons/math3/ode/nonstiff/MidpointStepInterpolator.java
+++ b/src/main/java/org/apache/commons/math3/ode/nonstiff/MidpointStepInterpolator.java
@@ -61,7 +61,7 @@ class MidpointStepInterpolator
    * to create the step interpolators by cloning an uninitialized model
    * and later initializing the copy.
    */
-  public MidpointStepInterpolator() {
+  MidpointStepInterpolator() {
   }
 
   /** Copy constructor.
@@ -69,7 +69,7 @@ class MidpointStepInterpolator
    * copy: its arrays are separated from the original arrays of the
    * instance
    */
-  public MidpointStepInterpolator(final MidpointStepInterpolator interpolator) {
+  MidpointStepInterpolator(final MidpointStepInterpolator interpolator) {
     super(interpolator);
   }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/nonstiff/RungeKuttaStepInterpolator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/nonstiff/RungeKuttaStepInterpolator.java b/src/main/java/org/apache/commons/math3/ode/nonstiff/RungeKuttaStepInterpolator.java
index c15c590..1ae7cb9 100644
--- a/src/main/java/org/apache/commons/math3/ode/nonstiff/RungeKuttaStepInterpolator.java
+++ b/src/main/java/org/apache/commons/math3/ode/nonstiff/RungeKuttaStepInterpolator.java
@@ -79,7 +79,7 @@ abstract class RungeKuttaStepInterpolator
   * @param interpolator interpolator to copy from.
 
   */
-  public RungeKuttaStepInterpolator(final RungeKuttaStepInterpolator interpolator) {
+  RungeKuttaStepInterpolator(final RungeKuttaStepInterpolator interpolator) {
 
     super(interpolator);
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/nonstiff/ThreeEighthesStepInterpolator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/nonstiff/ThreeEighthesStepInterpolator.java b/src/main/java/org/apache/commons/math3/ode/nonstiff/ThreeEighthesStepInterpolator.java
index 0f263d1..55d2751 100644
--- a/src/main/java/org/apache/commons/math3/ode/nonstiff/ThreeEighthesStepInterpolator.java
+++ b/src/main/java/org/apache/commons/math3/ode/nonstiff/ThreeEighthesStepInterpolator.java
@@ -71,7 +71,7 @@ class ThreeEighthesStepInterpolator
    * to create the step interpolators by cloning an uninitialized model
    * and later initializing the copy.
    */
-  public ThreeEighthesStepInterpolator() {
+  ThreeEighthesStepInterpolator() {
   }
 
   /** Copy constructor.
@@ -79,7 +79,7 @@ class ThreeEighthesStepInterpolator
    * copy: its arrays are separated from the original arrays of the
    * instance
    */
-  public ThreeEighthesStepInterpolator(final ThreeEighthesStepInterpolator interpolator) {
+  ThreeEighthesStepInterpolator(final ThreeEighthesStepInterpolator interpolator) {
     super(interpolator);
   }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/ode/sampling/StepNormalizerBounds.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/ode/sampling/StepNormalizerBounds.java b/src/main/java/org/apache/commons/math3/ode/sampling/StepNormalizerBounds.java
index ff1ee2c..0ccc58a 100644
--- a/src/main/java/org/apache/commons/math3/ode/sampling/StepNormalizerBounds.java
+++ b/src/main/java/org/apache/commons/math3/ode/sampling/StepNormalizerBounds.java
@@ -56,7 +56,7 @@ public enum StepNormalizerBounds {
      * @param last Whether the last point should be passed to the
      * underlying fixed step size step handler.
      */
-    private StepNormalizerBounds(final boolean first, final boolean last) {
+    StepNormalizerBounds(final boolean first, final boolean last) {
         this.first = first;
         this.last = last;
     }

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/optim/PointValuePair.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/optim/PointValuePair.java b/src/main/java/org/apache/commons/math3/optim/PointValuePair.java
index 0f27a52..c466fc7 100644
--- a/src/main/java/org/apache/commons/math3/optim/PointValuePair.java
+++ b/src/main/java/org/apache/commons/math3/optim/PointValuePair.java
@@ -106,7 +106,7 @@ public class PointValuePair extends Pair<double[], Double> implements Serializab
          * @param point Point coordinates.
          * @param value Value of the objective function at the point.
          */
-        public DataTransferObject(final double[] point, final double value) {
+        DataTransferObject(final double[] point, final double value) {
             this.point = point.clone();
             this.value = value;
         }

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/optim/PointVectorValuePair.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/optim/PointVectorValuePair.java b/src/main/java/org/apache/commons/math3/optim/PointVectorValuePair.java
index 0dd70f4..478e350 100644
--- a/src/main/java/org/apache/commons/math3/optim/PointVectorValuePair.java
+++ b/src/main/java/org/apache/commons/math3/optim/PointVectorValuePair.java
@@ -131,7 +131,7 @@ public class PointVectorValuePair extends Pair<double[], double[]> implements Se
          * @param point Point coordinates.
          * @param value Value of the objective function at the point.
          */
-        public DataTransferObject(final double[] point, final double[] value) {
+        DataTransferObject(final double[] point, final double[] value) {
             this.point = point.clone();
             this.value = value.clone();
         }

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/optim/linear/Relationship.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/optim/linear/Relationship.java b/src/main/java/org/apache/commons/math3/optim/linear/Relationship.java
index 58612c7..5e28a58 100644
--- a/src/main/java/org/apache/commons/math3/optim/linear/Relationship.java
+++ b/src/main/java/org/apache/commons/math3/optim/linear/Relationship.java
@@ -37,7 +37,7 @@ public enum Relationship {
      *
      * @param stringValue Display string for the relationship.
      */
-    private Relationship(String stringValue) {
+    Relationship(String stringValue) {
         this.stringValue = stringValue;
     }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/optim/nonlinear/scalar/MultivariateFunctionMappingAdapter.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/optim/nonlinear/scalar/MultivariateFunctionMappingAdapter.java b/src/main/java/org/apache/commons/math3/optim/nonlinear/scalar/MultivariateFunctionMappingAdapter.java
index 80daace..3c5127c 100644
--- a/src/main/java/org/apache/commons/math3/optim/nonlinear/scalar/MultivariateFunctionMappingAdapter.java
+++ b/src/main/java/org/apache/commons/math3/optim/nonlinear/scalar/MultivariateFunctionMappingAdapter.java
@@ -222,7 +222,7 @@ public class MultivariateFunctionMappingAdapter
          *
          * @param lower lower bound
          */
-        public LowerBoundMapper(final double lower) {
+        LowerBoundMapper(final double lower) {
             this.lower = lower;
         }
 
@@ -247,7 +247,7 @@ public class MultivariateFunctionMappingAdapter
         /** Simple constructor.
          * @param upper upper bound
          */
-        public UpperBoundMapper(final double upper) {
+        UpperBoundMapper(final double upper) {
             this.upper = upper;
         }
 
@@ -276,7 +276,7 @@ public class MultivariateFunctionMappingAdapter
          * @param lower lower bound
          * @param upper upper bound
          */
-        public LowerUpperBoundMapper(final double lower, final double upper) {
+        LowerUpperBoundMapper(final double lower, final double upper) {
             boundingFunction   = new Sigmoid(lower, upper);
             unboundingFunction = new Logit(lower, upper);
         }

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/optim/nonlinear/scalar/gradient/NonLinearConjugateGradientOptimizer.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/optim/nonlinear/scalar/gradient/NonLinearConjugateGradientOptimizer.java b/src/main/java/org/apache/commons/math3/optim/nonlinear/scalar/gradient/NonLinearConjugateGradientOptimizer.java
index 3e6fa89..9074122 100644
--- a/src/main/java/org/apache/commons/math3/optim/nonlinear/scalar/gradient/NonLinearConjugateGradientOptimizer.java
+++ b/src/main/java/org/apache/commons/math3/optim/nonlinear/scalar/gradient/NonLinearConjugateGradientOptimizer.java
@@ -70,7 +70,7 @@ public class NonLinearConjugateGradientOptimizer
      *
      * @since 2.0
      */
-    public static enum Formula {
+    public enum Formula {
         /** Fletcher-Reeves formula. */
         FLETCHER_REEVES,
         /** Polak-Ribière formula. */

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/optim/nonlinear/scalar/noderiv/CMAESOptimizer.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/optim/nonlinear/scalar/noderiv/CMAESOptimizer.java b/src/main/java/org/apache/commons/math3/optim/nonlinear/scalar/noderiv/CMAESOptimizer.java
index 82cc748..13566be 100644
--- a/src/main/java/org/apache/commons/math3/optim/nonlinear/scalar/noderiv/CMAESOptimizer.java
+++ b/src/main/java/org/apache/commons/math3/optim/nonlinear/scalar/noderiv/CMAESOptimizer.java
@@ -911,7 +911,7 @@ public class CMAESOptimizer
          * @param value Function value.
          * @param penalty Out-of-bounds penalty.
         */
-        public ValuePenaltyPair(final double value, final double penalty) {
+        ValuePenaltyPair(final double value, final double penalty) {
             this.value   = value;
             this.penalty = penalty;
         }
@@ -931,7 +931,7 @@ public class CMAESOptimizer
 
         /** Simple constructor.
          */
-        public FitnessFunction() {
+        FitnessFunction() {
             isRepairMode = true;
         }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/optimization/PointValuePair.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/optimization/PointValuePair.java b/src/main/java/org/apache/commons/math3/optimization/PointValuePair.java
index 7db9261..a338e69 100644
--- a/src/main/java/org/apache/commons/math3/optimization/PointValuePair.java
+++ b/src/main/java/org/apache/commons/math3/optimization/PointValuePair.java
@@ -111,7 +111,7 @@ public class PointValuePair extends Pair<double[], Double> implements Serializab
          * @param point Point coordinates.
          * @param value Value of the objective function at the point.
          */
-        public DataTransferObject(final double[] point, final double value) {
+        DataTransferObject(final double[] point, final double value) {
             this.point = point.clone();
             this.value = value;
         }

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/optimization/PointVectorValuePair.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/optimization/PointVectorValuePair.java b/src/main/java/org/apache/commons/math3/optimization/PointVectorValuePair.java
index 4c1884d..4cacf97 100644
--- a/src/main/java/org/apache/commons/math3/optimization/PointVectorValuePair.java
+++ b/src/main/java/org/apache/commons/math3/optimization/PointVectorValuePair.java
@@ -136,7 +136,7 @@ public class PointVectorValuePair extends Pair<double[], double[]> implements Se
          * @param point Point coordinates.
          * @param value Value of the objective function at the point.
          */
-        public DataTransferObject(final double[] point, final double[] value) {
+        DataTransferObject(final double[] point, final double[] value) {
             this.point = point.clone();
             this.value = value.clone();
         }

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/optimization/direct/CMAESOptimizer.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/optimization/direct/CMAESOptimizer.java b/src/main/java/org/apache/commons/math3/optimization/direct/CMAESOptimizer.java
index 661b569..388a6f7 100644
--- a/src/main/java/org/apache/commons/math3/optimization/direct/CMAESOptimizer.java
+++ b/src/main/java/org/apache/commons/math3/optimization/direct/CMAESOptimizer.java
@@ -1013,7 +1013,7 @@ public class CMAESOptimizer
 
         /** Simple constructor.
          */
-        public FitnessFunction() {
+        FitnessFunction() {
             valueRange = 1;
             isRepairMode = true;
         }

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/optimization/direct/MultivariateFunctionMappingAdapter.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/optimization/direct/MultivariateFunctionMappingAdapter.java b/src/main/java/org/apache/commons/math3/optimization/direct/MultivariateFunctionMappingAdapter.java
index 334138b..32f2a2c 100644
--- a/src/main/java/org/apache/commons/math3/optimization/direct/MultivariateFunctionMappingAdapter.java
+++ b/src/main/java/org/apache/commons/math3/optimization/direct/MultivariateFunctionMappingAdapter.java
@@ -203,7 +203,7 @@ public class MultivariateFunctionMappingAdapter implements MultivariateFunction
 
         /** Simple constructor.
          */
-        public NoBoundsMapper() {
+        NoBoundsMapper() {
         }
 
         /** {@inheritDoc} */
@@ -227,7 +227,7 @@ public class MultivariateFunctionMappingAdapter implements MultivariateFunction
         /** Simple constructor.
          * @param lower lower bound
          */
-        public LowerBoundMapper(final double lower) {
+        LowerBoundMapper(final double lower) {
             this.lower = lower;
         }
 
@@ -252,7 +252,7 @@ public class MultivariateFunctionMappingAdapter implements MultivariateFunction
         /** Simple constructor.
          * @param upper upper bound
          */
-        public UpperBoundMapper(final double upper) {
+        UpperBoundMapper(final double upper) {
             this.upper = upper;
         }
 
@@ -281,7 +281,7 @@ public class MultivariateFunctionMappingAdapter implements MultivariateFunction
          * @param lower lower bound
          * @param upper upper bound
          */
-        public LowerUpperBoundMapper(final double lower, final double upper) {
+        LowerUpperBoundMapper(final double lower, final double upper) {
             boundingFunction   = new Sigmoid(lower, upper);
             unboundingFunction = new Logit(lower, upper);
         }

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/optimization/fitting/CurveFitter.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/optimization/fitting/CurveFitter.java b/src/main/java/org/apache/commons/math3/optimization/fitting/CurveFitter.java
index ad1e025..dce5bc7 100644
--- a/src/main/java/org/apache/commons/math3/optimization/fitting/CurveFitter.java
+++ b/src/main/java/org/apache/commons/math3/optimization/fitting/CurveFitter.java
@@ -207,7 +207,7 @@ public class CurveFitter<T extends ParametricUnivariateFunction> {
         /** Simple constructor.
          * @param f function to fit.
          */
-        public OldTheoreticalValuesFunction(final ParametricUnivariateFunction f) {
+        OldTheoreticalValuesFunction(final ParametricUnivariateFunction f) {
             this.f = f;
         }
 
@@ -249,7 +249,7 @@ public class CurveFitter<T extends ParametricUnivariateFunction> {
         /** Simple constructor.
          * @param f function to fit.
          */
-        public TheoreticalValuesFunction(final ParametricUnivariateFunction f) {
+        TheoreticalValuesFunction(final ParametricUnivariateFunction f) {
             this.f = f;
         }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/optimization/general/NonLinearConjugateGradientOptimizer.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/optimization/general/NonLinearConjugateGradientOptimizer.java b/src/main/java/org/apache/commons/math3/optimization/general/NonLinearConjugateGradientOptimizer.java
index c076d11..ee16472 100644
--- a/src/main/java/org/apache/commons/math3/optimization/general/NonLinearConjugateGradientOptimizer.java
+++ b/src/main/java/org/apache/commons/math3/optimization/general/NonLinearConjugateGradientOptimizer.java
@@ -284,7 +284,7 @@ public class NonLinearConjugateGradientOptimizer
         /** Simple constructor.
          * @param searchDirection search direction
          */
-        public LineSearchFunction(final double[] searchDirection) {
+        LineSearchFunction(final double[] searchDirection) {
             this.searchDirection = searchDirection;
         }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/optimization/linear/Relationship.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/optimization/linear/Relationship.java b/src/main/java/org/apache/commons/math3/optimization/linear/Relationship.java
index 0da755f..3b1adc0 100644
--- a/src/main/java/org/apache/commons/math3/optimization/linear/Relationship.java
+++ b/src/main/java/org/apache/commons/math3/optimization/linear/Relationship.java
@@ -40,7 +40,7 @@ public enum Relationship {
     /** Simple constructor.
      * @param stringValue display string for the relationship
      */
-    private Relationship(String stringValue) {
+    Relationship(String stringValue) {
         this.stringValue = stringValue;
     }
 

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/random/EmpiricalDistribution.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/random/EmpiricalDistribution.java b/src/main/java/org/apache/commons/math3/random/EmpiricalDistribution.java
index 74b1bdd..06b0a9b 100644
--- a/src/main/java/org/apache/commons/math3/random/EmpiricalDistribution.java
+++ b/src/main/java/org/apache/commons/math3/random/EmpiricalDistribution.java
@@ -349,7 +349,7 @@ public class EmpiricalDistribution extends AbstractRealDistribution {
          *
          * @param in BufferedReader input stream
          */
-        public StreamDataAdapter(BufferedReader in){
+        StreamDataAdapter(BufferedReader in){
             super();
             inputStream = in;
         }
@@ -398,7 +398,7 @@ public class EmpiricalDistribution extends AbstractRealDistribution {
          * @param in double[] array holding the data
          * @throws NullArgumentException if in is null
          */
-        public ArrayDataAdapter(double[] in) throws NullArgumentException {
+        ArrayDataAdapter(double[] in) throws NullArgumentException {
             super();
             MathUtils.checkNotNull(in);
             inputArray = in;

http://git-wip-us.apache.org/repos/asf/commons-math/blob/2114da38/src/main/java/org/apache/commons/math3/stat/clustering/KMeansPlusPlusClusterer.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/math3/stat/clustering/KMeansPlusPlusClusterer.java b/src/main/java/org/apache/commons/math3/stat/clustering/KMeansPlusPlusClusterer.java
index ace584e..07cec09 100644
--- a/src/main/java/org/apache/commons/math3/stat/clustering/KMeansPlusPlusClusterer.java
+++ b/src/main/java/org/apache/commons/math3/stat/clustering/KMeansPlusPlusClusterer.java
@@ -42,7 +42,7 @@ import org.apache.commons.math3.util.MathUtils;
 public class KMeansPlusPlusClusterer<T extends Clusterable<T>> {
 
     /** Strategies to use for replacing an empty cluster. */
-    public static enum EmptyClusterStrategy {
+    public enum EmptyClusterStrategy {
 
         /** Split the cluster with largest distance variance. */
         LARGEST_VARIANCE,