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 Work | 69 | todo, FIXME |
Each tag is detailed below:
Number of occurrences found in the code: 69
| com.puppycrawl.tools.checkstyle.Checker | Line |
|---|---|
| i18n | 183 |
| com.puppycrawl.tools.checkstyle.ConfigurationLoader | Line |
| debug logging for support purposes | 128 |
| , this check could be modified to go if pos>prev; seems like this current version could stick empty strings into the list | 513 |
| com.puppycrawl.tools.checkstyle.DefaultConfiguration | Line |
| i18n | 75 |
| com.puppycrawl.tools.checkstyle.InputSimple | Line |
| --No comment-- | 161 |
| a | 162 |
| --No comment-- | 163 |
| --No comment-- | 164 |
| x */ /* YES!! | 167 |
| 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 | 167 |
| improve the error handing | 179 |
| com.puppycrawl.tools.checkstyle.annotation.AnnotationUseWithTrailingComma | Line |
| CheckStyle's ANTL grammar cannot handle this syntax | 4 |
| CheckStyle's ANTL grammar cannot handle this syntax | 28 |
| CheckStyle's ANTL grammar cannot handle this syntax | 36 |
| CheckStyle's ANTL grammar cannot handle this syntax | 47 |
| com.puppycrawl.tools.checkstyle.annotation.InputCorrectAnnotationLocation | Line |
| Auto-generated constructor stub | 29 |
| Auto-generated constructor stub | 62 |
| com.puppycrawl.tools.checkstyle.annotation.InputIncorrectAnnotationLocation | Line |
| Auto-generated constructor stub | 58 |
| com.puppycrawl.tools.checkstyle.api.AutomaticBean | Line |
| is there a smarter way to tell beanutils not to use defaults? | 66 |
| debug log messages | 130 |
| debug log messages | 201 |
| log.debug("The bean " + this.getClass() + " is not interested in " + value) | 211 |
| com.puppycrawl.tools.checkstyle.api.DetailAST | Line |
| should be private but that breaks the DetailASTTest until we manage parent in DetailAST instead of externally | 194 |
| Check visibility, could be private if set in setFirstChild() and friends | 198 |
| com.puppycrawl.tools.checkstyle.api.FullIdent | Line |
| make a function | 91 |
| 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 | 59 |
| com.puppycrawl.tools.checkstyle.checks.blocks.EmptyBlockCheck | Line |
| does this handle TokenTypes.LITERAL_SYNCHRONIZED? | 97 |
| does this handle TokenTypes.LITERAL_SYNCHRONIZED? | 120 |
| com.puppycrawl.tools.checkstyle.checks.blocks.LeftCurlyCheck | Line |
| replace this ugly hack * | 99 |
| need to handle.... TokenTypes.STATIC_INIT, | 144 |
| com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck | Line |
| is there better way to check is ast in left part of assignment? | 191 |
| 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 | 82 |
| com.puppycrawl.tools.checkstyle.checks.design.HideUtilityClassConstructorCheck | Line |
| check for "extends java.lang.Object" and "extends Object" consider "import org.omg.CORBA.*" | 113 |
| com.puppycrawl.tools.checkstyle.checks.header.RegexpHeaderCheck | Line |
| Not sure if cache in Utils is still necessary | 138 |
| com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck | Line |
| allow preset indentation styles (IE... GNU style, Sun style, etc...)? | 25 |
| optionally make imports (and other?) statements required to start line? -- but maybe this should be a different check | 27 |
| optionally allow array children, throws clause, etc... to be of any indentation > required, for emacs-style indentation | 30 |
| this is not illegal, but probably should be: myfunc3(11, 11, Integer. getInteger("mytest").intValue(), // this should be in 4 more 11); | 33 |
| 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 | 38 |
| 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. | 205 |
| com.puppycrawl.tools.checkstyle.checks.whitespace.SeparatorWrapCheck | Line |
| It is a copy/paste from OperatorWrapCheck. It should be fixed in another issue | 132 |
| com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck | Line |
| i18n | 91 |
| com.puppycrawl.tools.checkstyle.doclets.CheckDocsDoclet | Line |
| introduce a "CheckstyleModule" interface so we can do better in the next line... | 178 |
| throw exception here ??? | 240 |
| 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 | 291 |
| perhaps we should notify user somehow? | 441 |
| com.puppycrawl.tools.checkstyle.filters.SuppressionCommentFilter | Line |
| Investigate performance improvement with array | 282 |
| perhaps we should notify user somehow? | 421 |
| try binary search if sequential search becomes a performance problem. | 444 |
| 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. | 306 |
| com.puppycrawl.tools.checkstyle.gui.ParseTreeModel | Line |
| Need to resolve if need a fake root node.... | 48 |
| com.puppycrawl.tools.checkstyle.indentation //indent:0 exp:0.FromGuava | Line |
| (kevinb): If we strongly reference the value and aren't loading, we needn't wrap the //indent:5 exp:5 value. This could save ~8 bytes per entry. //indent:5 exp:5 / //indent:5 exp:5 | 29 |
| com.puppycrawl.tools.checkstyle.indentation //indent:0 exp:0.InputInvalidAssignIndent | Line |
| this should be illegal. //indent:8 exp:8 | 30 |
| add more testing //indent:8 exp:8 | 33 |
| com.puppycrawl.tools.checkstyle.indentation //indent:0 exp:0.InputInvalidIfIndent | Line |
| bother to support this style? //indent:8 exp:8 | 221 |
| com.puppycrawl.tools.checkstyle.indentation //indent:0 exp:0.InputInvalidMethodIndent | Line |
| this is not illegal, but probably should be //indent:0 exp:0 myfunc3(11, 11, Integer. //indent:0 exp:0 getInteger("mytest").intValue(), //indent:0 exp:0 11); //indent:0 exp:0 | 151 |
| com.puppycrawl.tools.checkstyle.indentation //indent:0 exp:0.InputInvalidWhileIndent | Line |
| this is allowed //indent:8 exp:8 | 52 |
| com.puppycrawl.tools.checkstyle.indentation //indent:0 exp:0.InputValidAssignIndent | Line |
| add more testing //indent:8 exp:8 | 55 |
| com.puppycrawl.tools.checkstyle.indentation //indent:0 exp:0.InputValidClassDefIndent | Line |
| this is broken right now: //indent:8 exp:8 1) this is both an expression and an OBJBLOCK //indent:8 exp:8 2) methods aren't yet parsed //indent:8 exp:8 3) only CLASSDEF is handled now, not OBJBLOCK //indent:8 exp:8 | 92 |
| com.puppycrawl.tools.checkstyle.indentation //indent:0 exp:0.InputValidIfIndent | Line |
| bother to support this style? //indent:8 exp:8 | 203 |