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 0 FIXME
TODO 65 TODO

Each tag is detailed below:

TODO

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