The following document contains the results of FindBugs Report
FindBugs Version is 1.2.0
Threshold is Low
Effort is Default
| Classes | Bugs | Errors | Missing Classes |
|---|---|---|---|
| 684 | 77 | 0 | 0 |
| Bug | Category | Details | Line |
|---|---|---|---|
| org.jcurl.core.api.EnumBase$HashCodeComp implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to fact in org.jcurl.core.api.Measure.hashCode() | STYLE | DLS_DEAD_LOCAL_STORE | 87 |
| Bug | Category | Details | Line |
|---|---|---|---|
| org.jcurl.core.api.RockDouble.clone() does not call super.clone() | BAD_PRACTICE | CN_IDIOM_NO_SUPER_CALL | 53 |
| Dead store to prime in org.jcurl.core.api.RockDouble.hashCode() | STYLE | DLS_DEAD_LOCAL_STORE | 92 |
| Test for floating point equality in org.jcurl.core.api.RockDouble.setA(double) | STYLE | FE_FLOATING_POINT_EQUALITY | 111 |
| Test for floating point equality in org.jcurl.core.api.RockDouble.setLocation(double, double, double) | STYLE | FE_FLOATING_POINT_EQUALITY | 119 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to fact in org.jcurl.core.api.RockSet.hashCode() | STYLE | DLS_DEAD_LOCAL_STORE | 222 |
| Class org.jcurl.core.api.RockSet defines non-transient non-serializable instance field dark | BAD_PRACTICE | SE_BAD_FIELD | Not available |
| Class org.jcurl.core.api.RockSet defines non-transient non-serializable instance field light | BAD_PRACTICE | SE_BAD_FIELD | Not available |
| The field org.jcurl.core.api.RockSet.r2i is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Test for floating point equality in org.jcurl.core.api.RockSetUtils.getShotRocks(RockSet) | STYLE | FE_FLOATING_POINT_EQUALITY | 146 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Exception is caught when Exception is not thrown in org.jcurl.core.api.TransferObject.toString() | STYLE | REC_CATCH_EXCEPTION | 96 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Redundant nullcheck of ?, which is known to be non-null in org.jcurl.core.api.WeakHashSet.add(Object) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 133 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Exception is caught when Exception is not thrown in org.jcurl.core.helpers.BatikButler.<static initializer>() | STYLE | REC_CATCH_EXCEPTION | 50 |
| Bug | Category | Details | Line |
|---|---|---|---|
| org.jcurl.core.helpers.FilterIterator.next() can't throw NoSuchElement exception | BAD_PRACTICE | IT_NO_SUCH_ELEMENT | 65-67 |
| Bug | Category | Details | Line |
|---|---|---|---|
| org.jcurl.core.helpers.Service.providerClasses(Class, ClassLoader) may fail to close stream | BAD_PRACTICE | OS_OPEN_STREAM | 60 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Non-virtual method call in org.jcurl.core.helpers.Version.create(String, String) passes null for unconditionally dereferenced parameter of Version(int[], Date) | CORRECTNESS | NP_NULL_PARAM_DEREF_NONVIRTUAL | 56 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Load of known null value in org.jcurl.core.helpers.XmlSimpleWriter.startPrefixMapping(String, String) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 404 |
| Bug | Category | Details | Line |
|---|---|---|---|
| The field org.jcurl.core.impl.CollissionSpin.mu is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Test for floating point equality in org.jcurl.core.impl.CollissionSpinLoss.singleLoss(Point2D, Point2D, double[]) | STYLE | FE_FLOATING_POINT_EQUALITY | 187 |
| The method name org.jcurl.core.impl.CollissionSpinLoss.Loss() doesn't start with an lower case letter | BAD_PRACTICE | NM_METHOD_NAMING_CONVENTION | 140 |
| The field org.jcurl.core.impl.CollissionSpinLoss.U is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available |
| The field org.jcurl.core.impl.CollissionSpinLoss.mu is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to fact in org.jcurl.core.impl.CollissionStore$Tupel.hashCode() | STYLE | DLS_DEAD_LOCAL_STORE | 76 |
| Bug | Category | Details | Line |
|---|---|---|---|
| org.jcurl.core.impl.CollissionStore$TupelComp implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to g_2 in org.jcurl.core.impl.CoulombCurler.computeHackSpeed(double, Point2D) | STYLE | DLS_DEAD_LOCAL_STORE | 83 |
| The field org.jcurl.core.impl.CoulombCurler.mu is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The field org.jcurl.core.impl.CurlerDenny.b is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Test for floating point equality in org.jcurl.core.impl.CurveManager.setCurrentTime(double) | STYLE | FE_FLOATING_POINT_EQUALITY | 359 |
| Private method org.jcurl.core.impl.CurveManager.recomputeCurves(int) is never called | PERFORMANCE | UPM_UNCALLED_PRIVATE_METHOD | 321-329 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to prime in org.jcurl.core.impl.CurveStoreImpl.hashCode() | STYLE | DLS_DEAD_LOCAL_STORE | 122 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Test for floating point equality in org.jcurl.core.impl.StoredTrajectorySet.setCurrentTime(double) | STYLE | FE_FLOATING_POINT_EQUALITY | 108 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.jcurl.core.io.JCurlSerializer.isGzipped(String) invokes toString() method on a String | PERFORMANCE | DM_STRING_TOSTRING | 75 |
| Exception is caught when Exception is not thrown in org.jcurl.core.io.JCurlSerializer.read(InputStream) | STYLE | REC_CATCH_EXCEPTION | 97 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Exception is caught when Exception is not thrown in org.jcurl.core.jnlp.FileDialogWebstart.FileDialogWebstart() | STYLE | REC_CATCH_EXCEPTION | 144 |
| Exception is caught when Exception is not thrown in org.jcurl.core.jnlp.FileDialogWebstart.openFileDialog(String, String[], Component) | STYLE | REC_CATCH_EXCEPTION | 157 |
| Exception is caught when Exception is not thrown in org.jcurl.core.jnlp.FileDialogWebstart.saveAsFileDialog(String, String[], FileDialogService$Contents, Component) | STYLE | REC_CATCH_EXCEPTION | 171 |
| Exception is caught when Exception is not thrown in org.jcurl.core.jnlp.FileDialogWebstart.saveFileDialog(String, String[], InputStream, String, Component) | STYLE | REC_CATCH_EXCEPTION | 185 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Invocation of toString on an array in org.jcurl.core.jnlp.FileDialogWebstart$1.invoke(Object, Method, Object[]) | CORRECTNESS | DMI_INVOKING_TOSTRING_ON_ARRAY | 81 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to prime in org.jcurl.core.ui.BroomPromptModel$HandleMemento.hashCode() | STYLE | DLS_DEAD_LOCAL_STORE | 74 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to prime in org.jcurl.core.ui.BroomPromptModel$IndexMemento.hashCode() | STYLE | DLS_DEAD_LOCAL_STORE | 123 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to prime in org.jcurl.core.ui.BroomPromptModel$SplitMemento.hashCode() | STYLE | DLS_DEAD_LOCAL_STORE | 172 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to prime in org.jcurl.core.ui.DefaultBroomPromptModel.hashCode() | STYLE | DLS_DEAD_LOCAL_STORE | 104 |
| Bug | Category | Details | Line |
|---|---|---|---|
| org.jcurl.core.ui.FileNameExtensionFilter.getExtensions() may expose internal representation by returning FileNameExtensionFilter.extensions | MALICIOUS_CODE | EI_EXPOSE_REP | 87 |
| org.jcurl.core.ui.FileNameExtensionFilter.FileNameExtensionFilter(String, String[]) may expose internal representation by storing an externally mutable object into FileNameExtensionFilter.extensions | MALICIOUS_CODE | EI_EXPOSE_REP2 | 43 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to PRIME in org.jcurl.core.ui.FixpointZoomer.hashCode() | STYLE | DLS_DEAD_LOCAL_STORE | 234 |
| Bug | Category | Details | Line |
|---|---|---|---|
| org.jcurl.core.ui.IceShapes.ROCK_LABELS should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 87 |
| Private method org.jcurl.core.ui.IceShapes.interpolateHSB(Color, Color, double) is never called | PERFORMANCE | UPM_UNCALLED_PRIVATE_METHOD | 193-198 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Nullcheck of f at line 115 of value previously dereferenced in org.jcurl.core.ui.JCurlShaper.toShape(R1RNFunction, double, double) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 109 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to prime in org.jcurl.core.ui.Memento.hashCode() | STYLE | DLS_DEAD_LOCAL_STORE | 100 |
| The field org.jcurl.core.ui.Memento.context is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to prime in org.jcurl.core.ui.P2DMemento.hashCode() | STYLE | DLS_DEAD_LOCAL_STORE | 64 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Unchecked/unconfirmed cast from java.lang.Runnable to org.jcurl.core.ui.TaskExecutor$Task in org.jcurl.core.ui.TaskExecutor.execute(Runnable) | STYLE | BC_UNCONFIRMED_CAST | 210 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to midy in org.jcurl.math.BisectionSolver.findRoot(R1R1Function, double, double, double, double) | STYLE | DLS_DEAD_LOCAL_STORE | 60 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to highKey in org.jcurl.math.CurveCombined.findFunctionIndex(double) | STYLE | DLS_DEAD_LOCAL_STORE | 344 |
| Dead store to lowKey in org.jcurl.math.CurveCombined.findFunctionIndex(double) | STYLE | DLS_DEAD_LOCAL_STORE | 340 |
| Test for floating point equality in org.jcurl.math.CurveCombined.binarySearch(List, int, int, double) | STYLE | FE_FLOATING_POINT_EQUALITY | 227 |
| Test for floating point equality in org.jcurl.math.CurveCombined.binarySearch(double[], int, int, double) | STYLE | FE_FLOATING_POINT_EQUALITY | 127 |
| Test for floating point equality in org.jcurl.math.CurveCombined.dropTail(double) | STYLE | FE_FLOATING_POINT_EQUALITY | 311 |
| Computation of average could overflow in org.jcurl.math.CurveCombined.binarySearch(double[], int, int, double) | STYLE | IM_AVERAGE_COMPUTATION_COULD_OVERFLOW | 132 |
| Computation of average could overflow in org.jcurl.math.CurveCombined.binarySearch(double[], int, int, double) | STYLE | IM_AVERAGE_COMPUTATION_COULD_OVERFLOW | 139 |
| Computation of average could overflow in org.jcurl.math.CurveCombined.binarySearch(double[], int, int, double) | STYLE | IM_AVERAGE_COMPUTATION_COULD_OVERFLOW | 142 |
| Class org.jcurl.math.CurveCombined implements same interface as superclass | STYLE | RI_REDUNDANT_INTERFACES | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.jcurl.math.CurveCombined$Part.Part(double, R1RNFunction) invokes inefficient Double(double) constructor; use Double.valueOf(double) instead | PERFORMANCE | DM_FP_NUMBER_CTOR | 56 |
| org.jcurl.math.CurveCombined$Part defines compareTo(Object) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 49 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Private method org.jcurl.math.Interpolators.getExponentialInstance() is never called | PERFORMANCE | UPM_UNCALLED_PRIVATE_METHOD | 65 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Method calls static Math class method on a constant value | PERFORMANCE | UM_UNNECESSARY_MATH | 46 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to det in org.jcurl.math.MathVec.gauss(double[][], double[], double[]) | STYLE | DLS_DEAD_LOCAL_STORE | 132 |
| Bug | Category | Details | Line |
|---|---|---|---|
| org.jcurl.math.Polynome.Polynome(double[]) may expose internal representation by storing an externally mutable object into Polynome.params | MALICIOUS_CODE | EI_EXPOSE_REP2 | 147 |
| Bug | Category | Details | Line |
|---|---|---|---|
| org.jcurl.math.PolynomeCurve.PolynomeCurve(double[][]) may expose internal representation by storing an externally mutable object into PolynomeCurve.params | MALICIOUS_CODE | EI_EXPOSE_REP2 | 104 |
| Load of known null value in org.jcurl.math.PolynomeCurve.transform(AffineTransform, PolynomeCurve) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 88 |
| Bug | Category | Details | Line |
|---|---|---|---|
| The field org.jcurl.math.R1RNFunctionImpl.dim is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to a in org.jcurl.math.ShaperUtils.curveTo(R1RNFunction, double, double, GeneralPath, float) | STYLE | DLS_DEAD_LOCAL_STORE | 135 |
| Dead store to b in org.jcurl.math.ShaperUtils.curveTo(R1RNFunction, double, double, GeneralPath, float) | STYLE | DLS_DEAD_LOCAL_STORE | 136 |
| Dead store to eps in org.jcurl.math.ShaperUtils.curveTo(R1RNFunction, double, double, GeneralPath, float) | STYLE | DLS_DEAD_LOCAL_STORE | 105 |
| Dead store to eps in org.jcurl.math.ShaperUtils.quadTo(R1RNFunction, double, double, GeneralPath, float) | STYLE | DLS_DEAD_LOCAL_STORE | 436 |
| Private method org.jcurl.math.ShaperUtils.interpolate(float, float, float, Interpolator) is never called | PERFORMANCE | UPM_UNCALLED_PRIVATE_METHOD | 209-210 |