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 |
| TODO | 40 | TODO |
Each tag is detailed below:
TODO
Number of occurrences found in the code: 40
| org.codehaus.plexus.metadata.DefaultComponentDescriptorWriterTest | Line |
|---|---|
| Verify requirements and configuration too... but I'm too lazy for that right now | 104 |
| copied from PlexusTools.buildConfiguration() - find a better way to do this we have duplication here, but we don't want to depend on plexus-container-default similar code in AnnotationComponentGleaner.glean() and QDoxComponentGleaner.findRequirements() | 175 |
| org.codehaus.plexus.metadata.ann.AnnInvocationHandler | Line |
| conversion for class, array, enum, and annotation types | 50 |
| conversion for class, array, enum, and annotation types | 73 |
| org.codehaus.plexus.metadata.ann.AnnReader | Line |
| good enough for now, but does not cover general case | 130 |
| org.codehaus.plexus.metadata.gleaner.AnnotationComponentGleaner | Line |
| Inspect methods? | 112 |
| See if we need to include interfaces here too? | 170 |
| implement type resolution without loading classes | 189 |
| Auto-generated catch block | 194 |
| See if we can glean any type details out of any generic information from the map or collection | 210 |
| need to read default annotation values if (anno.role()==null || anno.role().isAssignableFrom(Object.class)) { | 218 |
| org.codehaus.plexus.metadata.merge.ComponentsXmlMerger | Line |
| Ideally we don't want to manipulate the original dominant document but use its copy for merge. Document mDoc = (Document) dDocument.clone(); // doesn't merge properly | 40 |
| org.codehaus.plexus.metadata.merge.Driver | Line |
| --No comment-- | 184 |
| --No comment-- | 186 |
| org.codehaus.plexus.metadata.merge.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 ... | 175 |
| disallow < or & in entity replacement text (or ]]>???) TOOD keepEntityNormalizedForAttributeValue cached as well ... | 548 |
| check FIXME Alek | 603 |
| make check if namespace is interned!!! etc. for names!!! | 867 |
| check for efficient interning and then use elRawNameInterned!!!! | 1651 |
| --No comment-- | 1739 |
| add test for it form spec ... handle EOL normalization ... | 1955 |
| check FIXME Alek | 2002 |
| reconcile with setInput encodingName | 2529 |
| look on trashing //assert bufStart > 0 | 2786 |
| org.codehaus.plexus.metadata.merge.PlexusXmlMerger | Line |
| Ideally we don't want to manipulate the original dominant document but use its copy for merge. Document mDoc = (Document) dDocument.clone(); // doesn't merge properly | 40 |
| org.codehaus.plexus.metadata.merge.support.AbstractMergeableElement | Line |
| set up a unit test for this! | 115 |
| log to error | 154 |
| org.codehaus.plexus.metadata.merge.support.AbstractMergeableElementList | Line |
| log to error | 117 |
| better error message | 118 |
| org.codehaus.plexus.metadata.merge.support.ComponentsElement | Line |
| how is this different from getElementNamesForConflictChecks? | 65 |
| org.codehaus.plexus.metadata.merge.support.ConfigurationElement | Line |
| Implement merge for this. | 30 |
| Implement! | 44 |
| org.codehaus.plexus.metadata.merge.support.DescriptorTag | Line |
| set up Logger if (getLogger ().isErrorEnabled ()) getLogger.error ( "Could not create Mergeable instance for specified class '" + this.mergeableClass + "'" ); | 142 |
| org.codehaus.plexus.metadata.merge.support.MergeStrategies | Line |
| Revisit and factor {@link Mergeable#merge(Mergeable)} to use a {@link MergeStrategy}. | 35 |
| org.codehaus.plexus.metadata.merge.support.PlexusRootElement | Line |
| add the managers, etc | 38 |
| org.codehaus.plexus.metadata.merge.support.PlexusXmlTag | Line |
| Might be an idea factor and set up the list of allowed tags here itself.</p> | 9 |
| org.codehaus.plexus.metadata.merge.support.RequirementsElement | Line |
| add this back, but a test will fail (based on a role with no hint, which shouldn't be legal) l.add( ComponentElement.ROLE_HINT.getTagName() ); | 55 |