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 |
|---|---|---|
| @todo | 1 | @todo |
| FIXME | 8 | FIXME |
| TODO | 40 | TODO |
Each tag is detailed below:
@todo
Number of occurrences found in the code: 1
| org.codehaus.plexus.util.xml.SerializerXMLWriter | Line |
|---|---|
| Maybe the interface should allow IOExceptions on each? | 94 |
FIXME
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) | 1123 |
| 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... | 88 |
| org.codehaus.plexus.util.ReflectionUtils | Line |
| needed /required? | 185 |
| 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... | 126 |
| org.codehaus.plexus.util.xml.pull.MXParser | Line |
| --No comment-- | 1596 |
| --No comment-- | 1686 |
| --No comment-- | 1729 |
| --No comment-- | 1740 |
TODO
Number of occurrences found in the code: 40
| org.codehaus.plexus.util.FileUtilsTest | Line |
|---|---|
| I'm not sure what is supposed to happen here | 147 |
| Figure out how to test this | 156 |
| Should comparing 2 directories throw an Exception instead of returning false? Directories | 184 |
| StringIndexOutOfBoundsException thrown if file doesn't contain slash. Is this acceptable? assertEquals("", FileUtils.catPath("a", "b")); | 235 |
| org.codehaus.plexus.util.LineOrientedInterpolatingReader | Line |
| Fix this! It should report, but not interrupt. | 284 |
| 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(); } } | 186 |
| org.codehaus.plexus.util.cli.Commandline | Line |
| Provided only for backward compat. with <= 1.4 | 465 |
| Provided only for backward compat. with <= 1.4 | 572 |
| 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 | 165 |
| org.codehaus.plexus.util.introspection.ReflectionValueExtractorTest | Line |
| assert exception message | 224 |
| assert exception message | 230 |
| org.codehaus.plexus.util.reflection.Reflector | Line |
| Check if exception is the right action! Field exists, but contains null | 371 |
| 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. | 284 |
| org.codehaus.plexus.util.xml.Xpp3Dom | Line |
| WARNING! Later versions of plexus-utils psit out an <?xml ?> header due to thinking this is a new document - not the desired behaviour! | 320 |
| share this as some sort of assembler, implement a walk interface? | 367 |
| WARNING! Later versions of plexus-utils psit out an <?xml ?> header due to thinking this is a new document - not the desired behaviour! | 534 |
| WARNING! Later versions of plexus-utils psit out an <?xml ?> header due to thinking this is a new document - not the desired behaviour! | 543 |
| 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! | 69 |
| share this as some sort of assembler, implement a walk interface? | 118 |
| org.codehaus.plexus.util.xml.Xpp3DomWriter | Line |
| move to XMLWriter? | 39 |
| org.codehaus.plexus.util.xml.pull.EntityReplacementMap | Line |
| disallow < or & in entity replacement text (or ]]>???) | 64 |
| keepEntityNormalizedForAttributeValue cached as well ... | 65 |
| org.codehaus.plexus.util.xml.pull.MXParser | Line |
| best handling of interning issues have isAllNewStringInterned ??? | 21 |
| handling surrogate pairs: http://www.unicode.org/unicode/faq/utf_bom.html#6 | 24 |
| review code for use of bufAbsoluteStart when keeping pos between next()/fillBuf() | 26 |
| avoid using element raw name ... | 153 |
| disallow < or & in entity replacement text (or ]]>???) TOOD keepEntityNormalizedForAttributeValue cached as well ... | 642 |
| check FIXME Alek | 701 |
| make check if namespace is interned!!! etc. for names!!! | 982 |
| check for efficient interning and then use elRawNameInterned!!!! | 1767 |
| --No comment-- | 1854 |
| add test for it form spec ... handle EOL normalization ... | 2043 |
| check FIXME Alek | 2090 |
| reconcile with setInput encodingName | 2756 |
| look on trashing //assert bufStart > 0 | 3063 |
| 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 | 139 |
| 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 } | 207 |
| prefix should be already known from matching start tag ... | 758 |
| check if doing char[] text.getChars() would be faster than getCharAt(i) ... | 913 |