The following document contains the results of FindBugs
FindBugs Version is 3.0.1
Threshold is
Effort is min
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The field org.orekit.attitudes.TabulatedLofOffset.table is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The field org.orekit.attitudes.TabulatedProvider.table is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The field org.orekit.bodies.JPLCelestialBody.rawPVProvider is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The field org.orekit.data.FundamentalNutationArguments.gmstFunction is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| The field org.orekit.data.FundamentalNutationArguments.gmstRateFunction is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.orekit.estimation.measurements.PV.getCovarianceMatrix() may expose internal representation by returning PV.covarianceMatrix | MALICIOUS_CODE | EI_EXPOSE_REP | 303 | Medium |
| new org.orekit.estimation.measurements.PV(AbsoluteDate, Vector3D, Vector3D, double[][], double, int) may expose internal representation by storing an externally mutable object into PV.covarianceMatrix | MALICIOUS_CODE | EI_EXPOSE_REP2 | 280 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.orekit.estimation.measurements.modifiers.DynamicOutlierFilter.getSigma() may expose internal representation by returning DynamicOutlierFilter.sigma | MALICIOUS_CODE | EI_EXPOSE_REP | 54 | Medium |
| org.orekit.estimation.measurements.modifiers.DynamicOutlierFilter.setSigma(double[]) may expose internal representation by storing an externally mutable object into DynamicOutlierFilter.sigma | MALICIOUS_CODE | EI_EXPOSE_REP2 | 62 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.orekit.estimation.sequential.UnivariateProcessNoise.getLofCartesianOrbitalParametersEvolution() may expose internal representation by returning UnivariateProcessNoise.lofCartesianOrbitalParametersEvolution | MALICIOUS_CODE | EI_EXPOSE_REP | 129 | Medium |
| org.orekit.estimation.sequential.UnivariateProcessNoise.getPropagationParametersEvolution() may expose internal representation by returning UnivariateProcessNoise.propagationParametersEvolution | MALICIOUS_CODE | EI_EXPOSE_REP | 136 | Medium |
| new org.orekit.estimation.sequential.UnivariateProcessNoise(RealMatrix, LOFType, PositionAngle, UnivariateFunction[], UnivariateFunction[]) may expose internal representation by storing an externally mutable object into UnivariateProcessNoise.lofCartesianOrbitalParametersEvolution | MALICIOUS_CODE | EI_EXPOSE_REP2 | 101 | Medium |
| new org.orekit.estimation.sequential.UnivariateProcessNoise(RealMatrix, LOFType, PositionAngle, UnivariateFunction[], UnivariateFunction[]) may expose internal representation by storing an externally mutable object into UnivariateProcessNoise.propagationParametersEvolution | MALICIOUS_CODE | EI_EXPOSE_REP2 | 102 | Medium |
| Useless object stored in variable MOM of method org.orekit.estimation.sequential.UnivariateProcessNoise.getInertialOrbitalProcessNoiseMatrix(SpacecraftState, SpacecraftState) | STYLE | UC_USELESS_OBJECT | 206 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The field org.orekit.frames.CIRFProvider.xysPxy2Function is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The field org.orekit.frames.EOPHistory.cache is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| The field org.orekit.frames.EOPHistory.tidalCorrection is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The field org.orekit.frames.EclipticProvider.obliquity is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The field org.orekit.frames.GTODProvider.gastFunction is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The field org.orekit.frames.TEMEProvider.nutationFunction is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| The field org.orekit.frames.TEMEProvider.obliquityFunction is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The field org.orekit.frames.TODProvider.nutationFunction is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| The field org.orekit.frames.TODProvider.obliquityFunction is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.orekit.gnss.RinexLoader$Parser$PhaseShiftCorrection.getSatsCorrected() may expose internal representation by returning RinexLoader$Parser$PhaseShiftCorrection.satsPhaseShift | MALICIOUS_CODE | EI_EXPOSE_REP | 1093 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.orekit.gnss.attitude.GPSBlockIIA.END_MARGIN; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 64 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.orekit.gnss.attitude.GPSBlockIIF.END_MARGIN; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 59 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.orekit.gnss.attitude.GPSBlockIIR.END_MARGIN; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 53 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.orekit.gnss.attitude.Galileo.END_MARGIN; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 64 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The field org.orekit.models.earth.Geoid.harmonics is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The field org.orekit.propagation.analytical.Ephemeris.cache is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The field org.orekit.propagation.analytical.KeplerianPropagator.states is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Test for floating point equality in org.orekit.propagation.analytical.tle.TLE.equals(Object) | STYLE | FE_FLOATING_POINT_EQUALITY | 715 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.orekit.propagation.analytical.tle.TLESeries$TLEComparator implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | 425-443 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| instanceof will always return true for all non-null values in org.orekit.propagation.integration.FieldAbstractIntegratedPropagator.propagate(FieldAbsoluteDate), since all org.orekit.errors.OrekitException are instances of org.orekit.errors.OrekitException | STYLE | BC_VACUOUS_INSTANCEOF | 393 | Medium |
| instanceof will always return true for all non-null values in org.orekit.propagation.integration.FieldAbstractIntegratedPropagator.propagate(FieldAbsoluteDate, FieldAbsoluteDate), since all org.orekit.errors.OrekitException are instances of org.orekit.errors.OrekitException | STYLE | BC_VACUOUS_INSTANCEOF | 424 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The field org.orekit.propagation.semianalytical.dsst.forces.DSSTTesseral$TesseralShortPeriodicCoefficients.slots is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The field org.orekit.propagation.semianalytical.dsst.forces.DSSTZonal$ZonalShortPeriodicCoefficients.slots is transient but isn't set by deserialization | BAD_PRACTICE | SE_TRANSIENT_FIELD_NOT_RESTORED | Not available | Medium |