Tag List Report
The following document contains the listing of user tags found in the code. Below is the summary of the occurrences per tag.
| Tag Class | Total number of occurrences | Tag strings used by tag class |
|---|---|---|
| Deprecated Work | 27 | @deprecated |
| FIXME Work | 8 | fixme, @fixme |
| Todo Work | 28 | todo, @todo |
Each tag is detailed below:
Deprecated Work
Number of occurrences found in the code: 27
| org.codehaus.plexus.util.FileUtils | Line |
|---|---|
| use {@code java.nio.files.Files.write(filename, data.getBytes(encoding), StandardOpenOption.APPEND, StandardOpenOption.CREATE)} | 392 |
| use {@code java.nio.files.Files.write(filename, data.getBytes(encoding), StandardOpenOption.APPEND, StandardOpenOption.CREATE)} | 408 |
| org.codehaus.plexus.util.IOUtil | Line |
| Buffering streams is actively harmful! See the class description as to why. Use {@link #copy(InputStream, OutputStream)} instead. | 525 |
| use try-with-resources instead | 771 |
| use try-with-resources instead | 795 |
| use try-with-resources instead | 819 |
| use try-with-resources instead | 843 |
| use try-with-resources instead | 867 |
| org.codehaus.plexus.util.StringInputStream | Line |
| As of version 1.5.2 this class should no longer be used because it does not properly handle character encoding. Instead, wrap the output from {@link String#getBytes(String)} into a {@link java.io.ByteArrayInputStream}. | 68 |
| org.codehaus.plexus.util.StringOutputStream | Line |
| As of version 1.5.2 this class should no longer be used because it does not properly handle character encoding. Instead, use {@link java.io.ByteArrayOutputStream#toString(String)}. | 27 |
| org.codehaus.plexus.util.cli.CommandLineUtils | Line |
| Use {@link StringUtils#quoteAndEscape(String, char, char[], char[], char, boolean)}, {@link StringUtils#quoteAndEscape(String, char, char[], char, boolean)}, or {@link StringUtils#quoteAndEscape(String, char)} instead. | 506 |
| Use {@link StringUtils#quoteAndEscape(String, char, char[], char[], char, boolean)}, {@link StringUtils#quoteAndEscape(String, char, char[], char, boolean)}, or {@link StringUtils#quoteAndEscape(String, char)} instead. | 530 |
| Use {@link StringUtils#quoteAndEscape(String, char, char[], char[], char, boolean)}, {@link StringUtils#quoteAndEscape(String, char, char[], char, boolean)}, or {@link StringUtils#quoteAndEscape(String, char)} instead. | 549 |
| org.codehaus.plexus.util.cli.Commandline | Line |
| Use {@link org.codehaus.plexus.util.Os} class instead. | 101 |
| Use {@link org.codehaus.plexus.util.Os} class instead. | 107 |
| Use {@link Commandline#setExecutable(String)} instead. | 123 |
| Use {@link Commandline#setWorkingDirectory(File)} or {@link Commandline#setWorkingDirectory(String)} instead. | 129 |
| Use {@link Commandline#createArg()} instead | 296 |
| Use {@link Commandline#createArg(boolean)} instead | 311 |
| Remove once backward compat with plexus-utils <= 1.4 is no longer a consideration | 698 |
| Use {@link CommandLineUtils#translateCommandline(String)} instead. | 748 |
| Use {@link CommandLineUtils#quote(String)} instead. | 761 |
| Use {@link CommandLineUtils#toString(String[])} instead. | 771 |
| org.codehaus.plexus.util.dag.DAG | Line |
| instead use {@link #getVertices()} | 77 |
| org.codehaus.plexus.util.xml.XmlReader | Line |
| use XmlStreamReader | 57 |
| org.codehaus.plexus.util.xml.pull.XmlPullParserException | Line |
| use generic getCause() method | 15 |
| Use the generic <code>getCause()</code> method | 53 |
FIXME Work
Number of occurrences found in the code: 8
| org.codehaus.plexus.util.FileUtils | Line |
|---|---|
| I'm almost certain this // removal is unnecessary, as getAbsoluteFile() strips them. However, I'm not sure about this UNC stuff. (JT) | 1331 |
| org.codehaus.plexus.util.MatchPattern | Line |
| ICK! But we can't do partial matches for regex, so we have to reserve judgement until we have a file to deal with, or we can definitely say this is an exclusion... | 91 |
| org.codehaus.plexus.util.ReflectionUtils | Line |
| needed /required? | 205 |
| org.codehaus.plexus.util.SelectorUtils | Line |
| ICK! But we can't do partial matches for regex, so we have to reserve judgement until we have a file to deal with, or we can definitely say this is an exclusion... | 133 |
| org.codehaus.plexus.util.xml.pull.MXParser | Line |
| --No comment-- | 2004 |
| --No comment-- | 2136 |
| --No comment-- | 2176 |
| --No comment-- | 2189 |
Todo Work
Number of occurrences found in the code: 28
| org.codehaus.plexus.util.LineOrientedInterpolatingReader | Line |
|---|---|
| Fix this! It should report, but not interrupt. | 337 |
| org.codehaus.plexus.util.cli.CommandLineUtils | Line |
| Find out if waitUntilDone needs to be called using a try-finally construct. The method may throw an InterruptedException so that calls to waitUntilDone may be skipped. try { if ( inputFeeder != null ) { inputFeeder.waitUntilDone(); } } finally { try { outputPumper.waitUntilDone(); } finally { errorPumper.waitUntilDone(); } } | 206 |
| org.codehaus.plexus.util.cli.Commandline | Line |
| Provided only for backward compat. with <= 1.4 | 533 |
| Provided only for backward compat. with <= 1.4 | 656 |
| org.codehaus.plexus.util.dag.DAG | Line |
| this class should be renamed from DAG to Dag | 31 |
| org.codehaus.plexus.util.introspection.ReflectionValueExtractor | Line |
| don't throw Exception | 182 |
| org.codehaus.plexus.util.reflection.Reflector | Line |
| Check if exception is the right action! Field exists, but contains null | 455 |
| org.codehaus.plexus.util.xml.PrettyPrintXMLWriter | Line |
| change the below code to a more efficient expression when the library be ready to target Java 8. | 315 |
| org.codehaus.plexus.util.xml.Xpp3DomUtils | Line |
| WARNING! Later versions of plexus-utils psit out an <?xml ?> header due to thinking this is a new document - not the desired behaviour! | 73 |
| share this as some sort of assembler, implement a walk interface? | 126 |
| org.codehaus.plexus.util.xml.Xpp3DomWriter | Line |
| move to XMLWriter? | 44 |
| org.codehaus.plexus.util.xml.pull.EntityReplacementMap | Line |
| disallow < or & in entity replacement text (or ]]>???) | 71 |
| keepEntityNormalizedForAttributeValue cached as well ... | 72 |
| org.codehaus.plexus.util.xml.pull.MXParser | Line |
| best handling of interning issues have isAllNewStringInterned ??? | 23 |
| handling surrogate pairs: http://www.unicode.org/unicode/faq/utf_bom.html#6 | 26 |
| review code for use of bufAbsoluteStart when keeping pos between next()/fillBuf() | 28 |
| avoid using element raw name ... | 172 |
| disallow < or & in entity replacement text (or ]]>???) TOOD keepEntityNormalizedForAttributeValue cached as well ... | 755 |
| make check if namespace is interned!!! etc. for names!!! | 1197 |
| check for efficient interning and then use elRawNameInterned!!!! | 2219 |
| --No comment-- | 2335 |
| add test for it form spec ... handle EOL normalization ... | 2572 |
| reconcile with setInput encodingName | 3418 |
| look on trashing //assert bufStart > 0 | 3831 |
| org.codehaus.plexus.util.xml.pull.MXSerializer | Line |
| how to prevent from reporting this namespace? this is special namespace declared for consistency with XML infoset | 147 |
| use hashes for quick namespace->prefix lookups if( ! allStringsInterned ) { int[] newNamespacePrefixHash = new int[newSize]; if(namespacePrefixHash != null) { System.arraycopy( namespacePrefixHash, 0, newNamespacePrefixHash, 0, namespaceEnd); } namespacePrefixHash = newNamespacePrefixHash; } prefixesSize = newSize; ////assert nsPrefixes.length > size && nsPrefixes.length == newSize } | 225 |
| prefix should be already known from matching start tag ... | 947 |
| check if doing char[] text.getChars() would be faster than getCharAt(i) ... | 1164 |