The following document contains the results of FindBugs Report
FindBugs Version is 1.2.1
Threshold is Low
Effort is Max
Classes | Bugs | Errors | Missing Classes |
---|---|---|---|
180 | 33 | 0 | 0 |
Bug | Category | Details | Line |
---|---|---|---|
Dead store of null to doc in fr.inria.opengve.mascoptLib.io.reader.mgl.dom.MGLDOMReader.parse() | STYLE | DLS_DEAD_LOCAL_STORE_OF_NULL | 346 |
Load of known null value in fr.inria.opengve.mascoptLib.io.reader.mgl.dom.MGLDOMReader.createAndAttachCycle(Node) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 918 |
Possible null pointer dereference of flow in fr.inria.opengve.mascoptLib.io.reader.mgl.dom.MGLDOMReader.createAndAttachFlow(Node) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 1241 |
Redundant nullcheck of ?, which is known to be non-null in fr.inria.opengve.mascoptLib.io.reader.mgl.dom.MGLDOMReader.parse() | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 312 |
Bug | Category | Details | Line |
---|---|---|---|
Should fr.inria.opengve.mascoptLib.io.reader.mgl.dom.MGLDOMReader$InputStreamClone be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 118-189 |
Bug | Category | Details | Line |
---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 22 |
fr.inria.opengve.mascoptLib.io.reader.mgl.dom.MGLNodeFilter.getFilterList() may expose internal representation by returning MGLNodeFilter.interestingtags | MALICIOUS_CODE | EI_EXPOSE_REP | 53 |
Bug | Category | Details | Line |
---|---|---|---|
fr.inria.opengve.mascoptLib.io.reader.mgl.sax.InternalSaxReader.characters(char[], int, int) invokes inefficient new String() constructor | PERFORMANCE | DM_STRING_VOID_CTOR | 103 |
Write to static field fr.inria.opengve.mascoptLib.io.reader.mgl.sax.InternalSaxReader.idrefToDescription_ from instance method fr.inria.opengve.mascoptLib.io.reader.mgl.sax.InternalSaxReader.InternalSaxReader() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 93 |
Write to static field fr.inria.opengve.mascoptLib.io.reader.mgl.sax.InternalSaxReader.idrefToObject_ from instance method fr.inria.opengve.mascoptLib.io.reader.mgl.sax.InternalSaxReader.InternalSaxReader() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 92 |
InternalSaxReader.buffer_ not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Should fr.inria.opengve.mascoptLib.io.reader.mgl.sax.MapDescription$Entry be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 20-49 |
Bug | Category | Details | Line |
---|---|---|---|
Unwritten field: fr.inria.opengve.mascoptLib.io.reader.snd.natives.SNDModelNativeReader.demandModel | CORRECTNESS | UWF_UNWRITTEN_FIELD | 132 |
Unwritten field: fr.inria.opengve.mascoptLib.io.reader.snd.natives.SNDModelNativeReader.fixedChargeModel | CORRECTNESS | UWF_UNWRITTEN_FIELD | 161 |
Unwritten field: fr.inria.opengve.mascoptLib.io.reader.snd.natives.SNDModelNativeReader.linkCapacityModel | CORRECTNESS | UWF_UNWRITTEN_FIELD | 150 |
Unwritten field: fr.inria.opengve.mascoptLib.io.reader.snd.natives.SNDModelNativeReader.linkModel | CORRECTNESS | UWF_UNWRITTEN_FIELD | 141 |
Unwritten field: fr.inria.opengve.mascoptLib.io.reader.snd.natives.SNDModelNativeReader.objectiveType | CORRECTNESS | UWF_UNWRITTEN_FIELD | 209 |
Unwritten field: fr.inria.opengve.mascoptLib.io.reader.snd.natives.SNDModelNativeReader.pathHopLimit | CORRECTNESS | UWF_UNWRITTEN_FIELD | 191 |
Unwritten field: fr.inria.opengve.mascoptLib.io.reader.snd.natives.SNDModelNativeReader.pathModel | CORRECTNESS | UWF_UNWRITTEN_FIELD | 180 |
Unwritten field: fr.inria.opengve.mascoptLib.io.reader.snd.natives.SNDModelNativeReader.routingModel | CORRECTNESS | UWF_UNWRITTEN_FIELD | 170 |
Bug | Category | Details | Line |
---|---|---|---|
Method fr.inria.opengve.mascoptLib.io.reader.snd.natives.SNDNetworkNativeReader.readLinks() concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 360 |
Bug | Category | Details | Line |
---|---|---|---|
Method fr.inria.opengve.mascoptLib.io.validation.mgl.MGLValidator.validateWithXerces(InputSource, boolean) directly allocates a specific implementation of xml interfaces | STYLE | XFB_XML_FACTORY_BYPASS | 158 |
Bug | Category | Details | Line |
---|---|---|---|
Method fr.inria.opengve.mascoptLib.io.validation.mgl.MGLValidator$ErrorRecorder.getErrors() uses Collection.toArray() with zero-length array argument | PERFORMANCE | ITA_INEFFICIENT_TO_ARRAY | 52 |
Method fr.inria.opengve.mascoptLib.io.validation.mgl.MGLValidator$ErrorRecorder.getWarnings() uses Collection.toArray() with zero-length array argument | PERFORMANCE | ITA_INEFFICIENT_TO_ARRAY | 57 |
Bug | Category | Details | Line |
---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 589 |
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 752 |
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 497 |
Method fr.inria.opengve.mascoptLib.io.writer.fig.FigWriter.createOrGetColor(int) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 939 |
Method fr.inria.opengve.mascoptLib.io.writer.fig.FigWriter.createOrGetColor(int) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 943 |
Method fr.inria.opengve.mascoptLib.io.writer.fig.FigWriter.createOrGetColor(int) invokes inefficient Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 944 |
Method fr.inria.opengve.mascoptLib.io.writer.fig.FigWriter.writeUserColors(TreeMap, PrintStream) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 919 |
Bug | Category | Details | Line |
---|---|---|---|
fr.inria.opengve.mascoptLib.io.writer.mgl.dom.MGLWriter.rootElementForXMLFile isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 42 |
fr.inria.opengve.mascoptLib.io.writer.mgl.dom.MGLWriter.relaxNGVersionAttribute isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 65 |