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 | 0 | FIXME |
TODO | 65 | TODO |
Each tag is detailed below:
Number of occurrences found in the code: 65
com.puppycrawl.tools.checkstyle.Checker | Line |
---|---|
i18n | 178 |
com.puppycrawl.tools.checkstyle.ConfigurationLoader | Line |
debug logging for support purposes | 129 |
, this check could be modified to go if pos>prev; seems like this current version could stick empty strings into the list | 515 |
com.puppycrawl.tools.checkstyle.DefaultConfiguration | Line |
i18n | 73 |
com.puppycrawl.tools.checkstyle.TreeWalker | Line |
hmmm.. this looks less than elegant we have just parsed the string, now we're recreating it only to parse it again a few moments later | 147 |
improve the error handing | 159 |
else log warning | 281 |
com.puppycrawl.tools.checkstyle.api.AutomaticBean | Line |
is there a smarter way to tell beanutils not to use defaults? | 66 |
debug log messages | 129 |
debug log messages | 199 |
log.debug("The bean " + this.getClass() + " is not interested in " + value) | 209 |
com.puppycrawl.tools.checkstyle.api.DetailAST | Line |
should be private but that breaks the DetailASTTest until we manage parent in DetailAST instead of externally | 148 |
Check visibility, could be private if set in setFirstChild() and friends | 152 |
com.puppycrawl.tools.checkstyle.api.FullIdent | Line |
make a function | 91 |
com.puppycrawl.tools.checkstyle.api.LocalizedMessage | Line |
The cache should be cleared at some point. | 56 |
com.puppycrawl.tools.checkstyle.api.LocalizedMessages | Line |
check that this class is in the right package as soon as architecture has settled. At the time of writing this class is not necessary as a part of the public api | 21 |
com.puppycrawl.tools.checkstyle.checks.TranslationCheckTest | Line |
test with the same resourcebundle name in different packages x/messages.properties key1=x y/messages.properties key2=y should not result in error message about key1 missing in the y bundle | 57 |
com.puppycrawl.tools.checkstyle.checks.blocks.EmptyBlockCheck | Line |
need to handle.... TokenTypes.LITERAL_SWITCH, | 85 |
does this handle TokenTypes.LITERAL_SYNCHRONIZED? | 87 |
com.puppycrawl.tools.checkstyle.checks.blocks.LeftCurlyCheck | Line |
replace this ugly hack * | 73 |
need to handle.... TokenTypes.STATIC_INIT, | 115 |
com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck | Line |
is there better way to check is aAST in left part of assignment? | 137 |
com.puppycrawl.tools.checkstyle.checks.coding.IllegalInstantiationCheck | Line |
Clean up potential duplicate code here and in UnusedImportsCheck | 31 |
com.puppycrawl.tools.checkstyle.checks.coding.ReturnCountCheck | Line |
Test for inside a static block | 37 |
com.puppycrawl.tools.checkstyle.checks.coding.SimplifyBooleanReturnCheck | Line |
Introduce helpers in DetailAST | 69 |
com.puppycrawl.tools.checkstyle.checks.design.HideUtilityClassConstructorCheck | Line |
check for "extends java.lang.Object" and "extends Object" consider "import org.omg.CORBA.*" | 100 |
com.puppycrawl.tools.checkstyle.checks.duplicates.StrictDuplicateCodeCheck | Line |
return IGNORE for lines in the header comment? That would require some simple parsing... | 135 |
better to throw an exception here? it would be best to throw IOException from process(), but interface definition doesn't allow that... | 262 |
Not sure what to do with binary files such as gifs | 291 |
com.puppycrawl.tools.checkstyle.checks.header.RegexpHeaderCheck | Line |
Not sure if cache in Utils is still necessary | 138 |
com.puppycrawl.tools.checkstyle.checks.indentation.BlockParentHandler | Line |
look for SEMI and check for it here? | 240 |
com.puppycrawl.tools.checkstyle.checks.indentation.ClassDefHandler | Line |
still need to better deal with the modifiers and "class" | 80 |
com.puppycrawl.tools.checkstyle.checks.indentation.DoWhileHandler | Line |
checkWhile(); // while is not in the grammar, why not? | 59 |
com.puppycrawl.tools.checkstyle.checks.indentation.ExpressionHandler | Line |
this whole checking of consecuitive/expression line indents is smelling pretty bad... and is in serious need of pruning. But, I want to finish the invalid tests before I start messing around with it. | 217 |
shouldIncreaseIndent() is a hack, should be removed after complete rewriting of checkExpressionSubtree() | 290 |
allowNesting either shouldn't be allowed with firstLineMatches, or I should change the firstLineMatches logic so it doesn't match if the first line is nested | 373 |
getLineNo should probably not return < 0, but it is for the interface methods... I should ask about this | 442 |
getLineNo should probably not return < 0, but it is for the interface methods... I should ask about this | 489 |
the problem with this is that not all tree tokens actually have the right column number -- I should get a list of these and verify that checking nesting this way won't cause problems if (aAllowNesting && aTree.getColumnNo() != getLineStart(aTree)) { return; } | 498 |
com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck | Line |
allow preset indentation styles (IE... GNU style, Sun style, etc...)? | 27 |
optionally make imports (and other?) statements required to start line? -- but maybe this should be a different check | 29 |
optionally allow array children, throws clause, etc... to be of any indentation > required, for emacs-style indentation | 32 |
this is not illegal, but probably should be: myfunc3(11, 11, Integer. getInteger("mytest").intValue(), // this should be in 4 more 11); | 35 |
any dot-based indentation doesn't work (at least not yet...) the problem is that we don't know which way an expression tree will be built and with dot trees, they are built backwards. This means code like | 40 |
com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheckTest | Line |
needs to be finished | 296 |
extends and implements need to be added | 348 |
should be 16, not 12 | 457 |
why we get this message (it's duplicate of previous). | 460 |
com.puppycrawl.tools.checkstyle.checks.metrics.NPathComplexityCheck | Line |
For every or: _value += (_orCount * (nestedValue - 1)); | 35 |
For every and: ??? | 36 |
com.puppycrawl.tools.checkstyle.checks.whitespace.OperatorWrapCheck | Line |
Handle comments before and after operator Check if rest of line is whitespace, and not just the operator by itself. This last bit is to handle the operator on a line by itself. | 191 |
com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck | Line |
i18n | 75 |
com.puppycrawl.tools.checkstyle.doclets.CheckDocsDoclet | Line |
introduce a "CheckstyleModule" interface so we can do better in the next line... | 176 |
throw exception here ??? | 238 |
com.puppycrawl.tools.checkstyle.filters.FilterSetTest | Line |
this test should be removed/rewritten | 27 |
com.puppycrawl.tools.checkstyle.filters.SuppressWithNearbyCommentFilter | Line |
Investigate performance improvement with array | 290 |
perhaps we should notify user somehow? | 440 |
com.puppycrawl.tools.checkstyle.filters.SuppressionCommentFilter | Line |
Investigate performance improvement with array | 281 |
perhaps we should notify user somehow? | 420 |
try binary search if sequential search becomes a performance problem. | 443 |
com.puppycrawl.tools.checkstyle.grammars.GeneratedJavaLexer | Line |
Check visibility of this method one parsing is done in central utility method | 46 |
com.puppycrawl.tools.checkstyle.gui.FileDrop | Line |
Not sure that changing borders is a good idea. At least we should make sure that the border insets are preserved so that the panel layout does not change during the DnD operation. | 77 |
Blue is not a nice color in all LookAndFeels | 84 |
com.puppycrawl.tools.checkstyle.gui.ParseTreeInfoPanel | Line |
creating an object for the side effect of the constructor and then ignoring the object looks strange. | 281 |
com.puppycrawl.tools.checkstyle.gui.ParseTreeModel | Line |
Need to resolve if need a fake root node.... | 48 |