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 | 0 | @todo |
| FIXME | 4 | FIXME |
| TODO | 13 | TODO |
Each tag is detailed below:
FIXME
Number of occurrences found in the code: 4
| 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... | 87 |
| 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 |
TODO
Number of occurrences found in the code: 13
| org.codehaus.plexus.util.FileUtilsTest | Line |
|---|---|
| I'm not sure what is supposed to happen here | 139 |
| Figure out how to test this | 148 |
| Should comparing 2 directories throw an Exception instead of returning false? Directories | 176 |
| StringIndexOutOfBoundsException thrown if file doesn't contain slash. Is this acceptable? assertEquals("", FileUtils.catPath("a", "b")); | 227 |
| 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 |