The following document contains the results of PMD 6.21.0.
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 148 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 514 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 467 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 560 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 606 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 323 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 502 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 192 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 507 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 570 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 688 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 378 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 114 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 102 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 623 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 561 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 169 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 116 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 831 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 166 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 259 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 511 |
Rule | Violation | Line |
---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 79 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'Format': member interfaces are implicitly static | 94–161 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'Stage(int)': enum constructors are implicitly private | 125–127 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 771–774 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 266–271 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 689–694 |
CollapsibleIfStatements | These nested if statements could be combined | 1273–1276 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 372–376 |
Rule | Violation | Line |
---|---|---|
UnusedFormalParameter | Avoid unused method parameters such as 'state'. | 183 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 124–126 |
Rule | Violation | Line |
---|---|---|
EmptyWhileStmt | Avoid empty while statements | 513–515 |
EmptyIfStmt | Avoid empty if statements | 603–607 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'isFile': private methods cannot be overridden | 1101–1103 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 12–1235 |
EmptyCatchBlock | Avoid empty catch blocks | 711–713 |
Rule | Violation | Line |
---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'fs'. | 336 |
Rule | Violation | Line |
---|---|---|
EmptyIfStmt | Avoid empty if statements | 437–440 |
EmptyIfStmt | Avoid empty if statements | 442–447 |
CollapsibleIfStatements | These nested if statements could be combined | 901–910 |
CollapsibleIfStatements | These nested if statements could be combined | 904–909 |
CollapsibleIfStatements | These nested if statements could be combined | 1220–1226 |
EmptyWhileStmt | Avoid empty while statements | 1640–1642 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 215–221 |
Rule | Violation | Line |
---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'otp'. | 30 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 29–31 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 101–108 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 130–132 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 139–141 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 148–150 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 157–159 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 166–168 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 175–177 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 188–190 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 23–25 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 20–22 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'readIncludeFile': the method is declared in an interface type | 78–79 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'sha1': the method is declared in an interface type | 124 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'readFile': the method is declared in an interface type | 144–147 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'readFileWithMode': the method is declared in an interface type | 171–172 |
EmptyCatchBlock | Avoid empty catch blocks | 559–561 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 168–170 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifiers 'public static final' on field 'NO_MATCH': the field is declared in an interface type | 26–38 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'createPathMatcher': the method is declared in an interface type | 53–57 |
Rule | Violation | Line |
---|---|---|
SimplifiedTernary | Ternary operators that can be simplified with || or && | 33 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 10–310 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 302–304 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 31–35 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 39–41 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 73–75 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–604 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–125 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–758 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–320 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–151 |
EmptyCatchBlock | Avoid empty catch blocks | 116–118 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–204 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–415 |
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'State(boolean)': enum constructors are implicitly private | 81–83 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–293 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–214 |
Rule | Violation | Line |
---|---|---|
EmptyIfStmt | Avoid empty if statements | 48–50 |
Rule | Violation | Line |
---|---|---|
EmptyIfStmt | Avoid empty if statements | 61–63 |
Rule | Violation | Line |
---|---|---|
EmptyIfStmt | Avoid empty if statements | 56–58 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–244 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–747 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 215–217 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–490 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'isGarbage': the method is already in a final class | 182–184 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 150–152 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 326–328 |
EmptyCatchBlock | Avoid empty catch blocks | 346–348 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–149 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 341–343 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 143–145 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 89–91 |
Rule | Violation | Line |
---|---|---|
OverrideBothEqualsAndHashcode | Ensure you override both equals() and hashCode() | 108 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'clear': the method is already in a final class | 29–31 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'set': the method is already in a final class | 33–41 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'clear': the method is already in a final class | 43–47 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'get': the method is already in a final class | 49–52 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'toEWAHCompressedBitmap': the method is already in a final class | 54–79 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'block': private methods cannot be overridden | 81–83 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'mask': private methods cannot be overridden | 85–87 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'ofObjectType': private methods cannot be overridden | 316–318 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 79–81 |
CollapsibleIfStatements | These nested if statements could be combined | 166–168 |
Rule | Violation | Line |
---|---|---|
UselessOverridingMethod | Overriding method merely calls super | 164–166 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 196–199 |
EmptyCatchBlock | Avoid empty catch blocks | 315–317 |
CollapsibleIfStatements | These nested if statements could be combined | 716–718 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 202–205 |
EmptyCatchBlock | Avoid empty catch blocks | 275–277 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 389–391 |
EmptyCatchBlock | Avoid empty catch blocks | 426–428 |
EmptyCatchBlock | Avoid empty catch blocks | 544–547 |
EmptyCatchBlock | Avoid empty catch blocks | 942–944 |
EmptyCatchBlock | Avoid empty catch blocks | 967–969 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 78–80 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'maybeContains': the method is already in a final class | 43–47 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'contains': the method is already in a final class | 49–91 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'get': private methods cannot be overridden | 93–96 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'block': private methods cannot be overridden | 98–100 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'mask': private methods cannot be overridden | 102–104 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'isEmpty': private methods cannot be overridden | 106–108 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'or': the method is already in a final class | 110–114 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'andNot': the method is already in a final class | 116–120 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'xor': the method is already in a final class | 122–129 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'getBitmap': the method is already in a final class | 131–133 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 65–67 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 293–295 |
CollapsibleIfStatements | These nested if statements could be combined | 309–311 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 187–189 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'decompress': private methods cannot be overridden | 364–385 |
EmptyCatchBlock | Avoid empty catch blocks | 700–704 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 349–351 |
EmptyCatchBlock | Avoid empty catch blocks | 356–358 |
EmptyCatchBlock | Avoid empty catch blocks | 361–363 |
Rule | Violation | Line |
---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'fs'. | 35 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–514 |
EmptyCatchBlock | Avoid empty catch blocks | 414–416 |
EmptyCatchBlock | Avoid empty catch blocks | 435–449 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 14–1477 |
CollapsibleIfStatements | These nested if statements could be combined | 524–525 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'LooseRef': member interfaces are implicitly static | 1373–1377 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 184–186 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 13–255 |
Rule | Violation | Line |
---|---|---|
EmptyIfStmt | Avoid empty if statements | 36–40 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'StatsRecorder': member interfaces are implicitly static | 113–182 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'bits': private methods cannot be overridden | 328–334 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'PageRef': member interfaces are implicitly static | 774–834 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'CleanupQueue': member interfaces are implicitly static | 959–962 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 103–105 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on class 'PackInfo': members of interface types are implicitly public | 42–84 |
UnnecessaryModifier | Unnecessary modifier 'static' on class 'PackInfo': types nested within an interface type are implicitly static | 42–84 |
Rule | Violation | Line |
---|---|---|
ForLoopShouldBeWhileLoop | This for loop could be simplified to a while loop | 106–116 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'size': the method is already in a final class | 229–231 |
SimplifiedTernary | Ternary operators that can be simplified with || or && | 317 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 276–278 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'set': the method is already in a final class | 24–28 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'depth': the method is already in a final class | 31–33 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'type': the method is already in a final class | 36–38 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'size': the method is already in a final class | 41–43 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'empty': the method is already in a final class | 46–48 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'makeNext': the method is already in a final class | 50–60 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'write': the method is already in a final class | 77–81 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'write': the method is already in a final class | 85–105 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'writeFileHeader': the method is already in a final class | 113–120 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'writeObject': the method is already in a final class | 137–139 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'writeHeader': the method is already in a final class | 158–173 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'objectHeader': private methods cannot be overridden | 175–184 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'ofsDelta': private methods cannot be overridden | 186–193 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'ofsDeltaVarIntLength': private methods cannot be overridden | 195–200 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'getCopyBuffer': the method is already in a final class | 207–209 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'length': the method is already in a final class | 220–222 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'getDigest': the method is already in a final class | 225–227 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 12–2493 |
EmptyWhileStmt | Avoid empty while statements | 1918–1920 |
CollapsibleIfStatements | These nested if statements could be combined | 2258–2259 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'have': private methods cannot be overridden | 2273–2276 |
Rule | Violation | Line |
---|---|---|
EmptyWhileStmt | Avoid empty while statements | 261–263 |
CollapsibleIfStatements | These nested if statements could be combined | 273–277 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–567 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–572 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 281–284 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–382 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–216 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–694 |
UnusedNullCheckInEquals | Invoke equals() on the object you've already ensured is not null | 677 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–840 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–309 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–384 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–194 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–254 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 10–174 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 84–86 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 339–341 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 203–205 |
EmptyCatchBlock | Avoid empty catch blocks | 413–415 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'isId': the method is already in a final class | 44–54 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromString': the method is already in a final class | 68–76 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromObjectId': the method is already in a final class | 89–92 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromString': the method is already in a final class | 101–106 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromHexString': private methods cannot be overridden | 108–123 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'hexUInt32': private methods cannot be overridden | 125–136 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prefixCompare': the method is already in a final class | 217–237 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prefixCompare': the method is already in a final class | 253–273 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prefixCompare': the method is already in a final class | 289–309 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'getFirstByte': the method is already in a final class | 316–318 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'name': the method is already in a final class | 346–367 |
Rule | Violation | Line |
---|---|---|
SimplifiedTernary | Ternary operators that can be simplified with || or && | 266 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–753 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on interface 'Bitmap': members of interface types are implicitly public | 46–94 |
UnnecessaryModifier | Unnecessary modifier 'public' on interface 'BitmapBuilder': members of interface types are implicitly public | 100–188 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'BranchRebaseMode(String)': enum constructors are implicitly private | 42–44 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'SectionParser': member interfaces are implicitly static | 1454–1463 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'ConfigEnum': member interfaces are implicitly static | 1491–1507 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 95–97 |
Rule | Violation | Line |
---|---|---|
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 35 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 38 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 41 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 44 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 47 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 50 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 75 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 80 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 85 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 90 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 130 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'GpgFormat(String)': enum constructors are implicitly private | 31–33 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'StageState(int)': enum constructors are implicitly private | 116–118 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getWorkingTreeIterator': the method is declared in an interface type | 319 |
CollapsibleIfStatements | These nested if statements could be combined | 505–508 |
CollapsibleIfStatements | These nested if statements could be combined | 550–554 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 12–1244 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromHexString': private methods cannot be overridden | 208–222 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'newSegment': private methods cannot be overridden | 307–309 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'computeGrowAt': private methods cannot be overridden | 311–313 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'equals': private methods cannot be overridden | 315–322 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'createArray': private methods cannot be overridden | 225–227 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'Action(String, String)': enum constructors are implicitly private | 54–57 |
CollapsibleIfStatements | These nested if statements could be combined | 158–161 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'Storage(boolean, boolean)': enum constructors are implicitly private | 75–78 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 749–751 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 872–873 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'Key': member interfaces are implicitly static | 311–332 |
Rule | Violation | Line |
---|---|---|
UselessOverridingMethod | Overriding method merely calls super | 62–64 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'FetchRecurseSubmodulesMode(String)': enum constructors are implicitly private | 41–43 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–370 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 39–41 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 14–1328 |
CollapsibleIfStatements | These nested if statements could be combined | 375–377 |
CollapsibleIfStatements | These nested if statements could be combined | 787–823 |
CollapsibleIfStatements | These nested if statements could be combined | 812–814 |
Rule | Violation | Line |
---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'inCore'. | 53 |
Rule | Violation | Line |
---|---|---|
UselessOverridingMethod | Overriding method merely calls super | 119–121 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–620 |
EmptyCatchBlock | Avoid empty catch blocks | 215–218 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–359 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 164–166 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 133–136 |
EmptyCatchBlock | Avoid empty catch blocks | 152–167 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'dontFollow': private methods cannot be overridden | 141–145 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on class 'Commit': members of interface types are implicitly public | 76–112 |
UnnecessaryModifier | Unnecessary modifier 'static' on class 'Commit': types nested within an interface type are implicitly static | 76–112 |
UnnecessaryModifier | Unnecessary modifier 'public' on class 'RevWalk': members of interface types are implicitly public | 115–242 |
UnnecessaryModifier | Unnecessary modifier 'public' on class 'ObjectWalk': members of interface types are implicitly public | 245–360 |
Rule | Violation | Line |
---|---|---|
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 54 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 55 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 56 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 57 |
EmptyWhileStmt | Avoid empty while statements | 810–812 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 160–163 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–306 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getOpenFiles': the method is declared in an interface type | 30–32 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getOpenBytes': the method is declared in an interface type | 39–41 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getStats': the method is declared in an interface type | 47–49 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 12–1222 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 312–314 |
EmptyCatchBlock | Avoid empty catch blocks | 323–325 |
EmptyCatchBlock | Avoid empty catch blocks | 353–355 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 398–400 |
EmptyCatchBlock | Avoid empty catch blocks | 454–456 |
EmptyCatchBlock | Avoid empty catch blocks | 474–477 |
EmptyCatchBlock | Avoid empty catch blocks | 755–757 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 152–154 |
EmptyCatchBlock | Avoid empty catch blocks | 158–160 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 231–233 |
EmptyCatchBlock | Avoid empty catch blocks | 268–270 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on class 'ConnectivityCheckInfo': types nested within an interface type are implicitly static | 51–137 |
UnnecessaryModifier | Unnecessary modifier 'public' on class 'ConnectivityCheckInfo': members of interface types are implicitly public | 51–137 |
Rule | Violation | Line |
---|---|---|
AvoidThreadGroup | Avoid using java.lang.ThreadGroup; it is not thread safe | 81 |
EmptyCatchBlock | Avoid empty catch blocks | 282–284 |
EmptyCatchBlock | Avoid empty catch blocks | 308–310 |
EmptyCatchBlock | Avoid empty catch blocks | 392–394 |
EmptyCatchBlock | Avoid empty catch blocks | 394–396 |
EmptyCatchBlock | Avoid empty catch blocks | 396–398 |
EmptyCatchBlock | Avoid empty catch blocks | 401–403 |
EmptyCatchBlock | Avoid empty catch blocks | 406–408 |
Rule | Violation | Line |
---|---|---|
UselessQualifiedThis | Useless qualified this usage in the same class. | 39 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 12–528 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 62–65 |
EmptyCatchBlock | Avoid empty catch blocks | 74–77 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on class 'FtpException': types nested within an interface type are implicitly static | 31–64 |
EmptyCatchBlock | Avoid empty catch blocks | 188–190 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 159–161 |
EmptyIfStmt | Avoid empty if statements | 383–385 |
EmptyIfStmt | Avoid empty if statements | 386–388 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'HttpRedirectMode(String)': enum constructors are implicitly private | 123–125 |
EmptyIfStmt | Avoid empty if statements | 432–434 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 77–80 |
EmptyCatchBlock | Avoid empty catch blocks | 86–88 |
EmptyCatchBlock | Avoid empty catch blocks | 92–94 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 70–73 |
EmptyCatchBlock | Avoid empty catch blocks | 82–84 |
EmptyCatchBlock | Avoid empty catch blocks | 88–90 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 103–107 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'checkIfTooLarge': private methods cannot be overridden | 698–721 |
EmptyCatchBlock | Avoid empty catch blocks | 1151–1155 |
EmptyCatchBlock | Avoid empty catch blocks | 1167–1171 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 306–308 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 110–115 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on field 'DEFAULT': the field is declared in an interface type | 22–24 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 10–248 |
EmptyIfStmt | Avoid empty if statements | 69–71 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'StringReader': member interfaces are implicitly static | 54–65 |
EmptyCatchBlock | Avoid empty catch blocks | 232–235 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'PushRecurseSubmodulesMode(String)': enum constructors are implicitly private | 43–45 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 263–265 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 747–751 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–2269 |
EmptyCatchBlock | Avoid empty catch blocks | 363–365 |
EmptyCatchBlock | Avoid empty catch blocks | 374–376 |
EmptyCatchBlock | Avoid empty catch blocks | 390–392 |
EmptyCatchBlock | Avoid empty catch blocks | 1048–1050 |
EmptyCatchBlock | Avoid empty catch blocks | 1375–1377 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 521–522 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifiers 'static final' on field 'EMPTY_CONFIG': the field is declared in an interface type | 91–113 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'TagOpt(String)': enum constructors are implicitly private | 54–56 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 129–131 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 231–233 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 85–87 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 121–123 |
EmptyCatchBlock | Avoid empty catch blocks | 181–183 |
EmptyCatchBlock | Avoid empty catch blocks | 220–222 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 327–329 |
EmptyCatchBlock | Avoid empty catch blocks | 334–336 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 14–1679 |
CollapsibleIfStatements | These nested if statements could be combined | 832–834 |
EmptyWhileStmt | Avoid empty while statements | 1130–1132 |
EmptyCatchBlock | Avoid empty catch blocks | 1151–1153 |
EmptyCatchBlock | Avoid empty catch blocks | 1157–1159 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 313–316 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 11–2415 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'IOConsumer': member interfaces are implicitly static | 203–205 |
EmptyWhileStmt | Avoid empty while statements | 1069–1071 |
EmptyWhileStmt | Avoid empty while statements | 1318–1320 |
EmptyWhileStmt | Avoid empty while statements | 1327–1329 |
EmptyCatchBlock | Avoid empty catch blocks | 1374–1376 |
EmptyCatchBlock | Avoid empty catch blocks | 1542–1544 |
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 87–91 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 810–812 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 269–273 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 388–390 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 12–411 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 440–444 |
EmptyWhileStmt | Avoid empty while statements | 480–482 |
CollapsibleIfStatements | These nested if statements could be combined | 864–865 |
EmptyCatchBlock | Avoid empty catch blocks | 1507–1509 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 394–396 |
EmptyIfStmt | Avoid empty if statements | 1292–1295 |
EmptyCatchBlock | Avoid empty catch blocks | 1981–1988 |
EmptyCatchBlock | Avoid empty catch blocks | 2320–2322 |
Rule | Violation | Line |
---|---|---|
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 57 |
CollapsibleIfStatements | These nested if statements could be combined | 142–160 |
CollapsibleIfStatements | These nested if statements could be combined | 143–159 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 131–133 |
CollapsibleIfStatements | These nested if statements could be combined | 147–163 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 309–311 |
SimplifiedTernary | Ternary operators that can be simplified with || or && | 832 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'ParseableSimpleDateFormat(String)': enum constructors are implicitly private | 98–100 |
EmptyCatchBlock | Avoid empty catch blocks | 198–200 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'setRepository': the method is declared in an interface type | 305 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'index': private methods cannot be overridden | 136–140 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'createArray': private methods cannot be overridden | 143–145 |
Rule | Violation | Line |
---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 12–1307 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'match': the method is already in a final class | 98–105 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseBase10': the method is already in a final class | 179–213 |
EmptyCatchBlock | Avoid empty catch blocks | 207–209 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseLongBase10': the method is already in a final class | 233–267 |
EmptyCatchBlock | Avoid empty catch blocks | 261–263 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseHexInt16': the method is already in a final class | 284–297 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseHexInt32': the method is already in a final class | 314–338 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseHexInt64': the method is already in a final class | 356–404 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseHexInt4': the method is already in a final class | 415–420 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseTimeZoneOffset': the method is already in a final class | 434–436 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseTimeZoneOffset': the method is already in a final class | 454–460 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'next': the method is already in a final class | 473–480 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'nextLF': the method is already in a final class | 493–495 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'nextLF': the method is already in a final class | 510–518 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'headerEnd': the method is already in a final class | 531–540 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'headerStart': the method is already in a final class | 555–574 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prev': the method is already in a final class | 587–595 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prevLF': the method is already in a final class | 608–610 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prevLF': the method is already in a final class | 626–635 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'lineMap': the method is already in a final class | 659–667 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'lineMapOrBinary': the method is already in a final class | 685–692 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'author': the method is already in a final class | 730–737 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'committer': the method is already in a final class | 752–761 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'tagger': the method is already in a final class | 776–789 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'encoding': the method is already in a final class | 804–814 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'commitMessage': the method is already in a final class | 1217–1228 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'tagMessage': the method is already in a final class | 1241–1250 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'endOfParagraph': the method is already in a final class | 1266–1276 |
Rule | Violation | Line |
---|---|---|
EmptyWhileStmt | Avoid empty while statements | 67–69 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'neq': private methods cannot be overridden | 84–86 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'lc': private methods cannot be overridden | 88–90 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'isSameObject': the method is declared in an interface type | 31–33 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 372–374 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 38–40 |
Rule | Violation | Line |
---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 116–118 |
EmptyCatchBlock | Avoid empty catch blocks | 121–123 |
Rule | Violation | Line |
---|---|---|
SimplifiedTernary | Ternary operators that can be simplified with || or && | 54 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'init': the method is already in a final class | 562–565 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'save': the method is already in a final class | 567–573 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 179–180 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'FileMode.GITLINK' due to existing static import 'org.eclipse.jgit.lib.FileMode.GITLINK' | 180 |
UselessParentheses | Useless parentheses. | 181–182 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 549 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 171 |
UselessParentheses | Useless parentheses. | 172 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 399–400 |
UselessParentheses | Useless parentheses. | 401 |
UselessParentheses | Useless parentheses. | 535 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.R_HEADS' due to existing static import 'org.eclipse.jgit.lib.Constants.R_HEADS' | 223 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 422 |
UselessParentheses | Useless parentheses. | 422 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 162–164 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 383 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 251 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 756–757 |
UselessParentheses | Useless parentheses. | 756 |
UselessParentheses | Useless parentheses. | 757 |
UselessParentheses | Useless parentheses. | 1720 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 159 |
UselessParentheses | Useless parentheses. | 172 |
UselessParentheses | Useless parentheses. | 185 |
UselessParentheses | Useless parentheses. | 199 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 144 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 287 |
UselessParentheses | Useless parentheses. | 409–410 |
UselessParentheses | Useless parentheses. | 468 |
UselessParentheses | Useless parentheses. | 516 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 50 |
UselessParentheses | Useless parentheses. | 100 |
UselessParentheses | Useless parentheses. | 138 |
UselessParentheses | Useless parentheses. | 173 |
UselessParentheses | Useless parentheses. | 217 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 348 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 651 |
UselessParentheses | Useless parentheses. | 729 |
UselessParentheses | Useless parentheses. | 730 |
UselessParentheses | Useless parentheses. | 731 |
UselessParentheses | Useless parentheses. | 732 |
UselessParentheses | Useless parentheses. | 733 |
UselessParentheses | Useless parentheses. | 734 |
UselessParentheses | Useless parentheses. | 793 |
UselessParentheses | Useless parentheses. | 794 |
UselessParentheses | Useless parentheses. | 794 |
UselessParentheses | Useless parentheses. | 794 |
UselessParentheses | Useless parentheses. | 889 |
UselessParentheses | Useless parentheses. | 1134–1135 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 92 |
UselessParentheses | Useless parentheses. | 261 |
UselessParentheses | Useless parentheses. | 385 |
UselessParentheses | Useless parentheses. | 385 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 461 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 748 |
UselessParentheses | Useless parentheses. | 749 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 97 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Strings.getPathSeparator' due to existing static import 'org.eclipse.jgit.ignore.internal.Strings.getPathSeparator' | 99 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 105 |
UselessParentheses | Useless parentheses. | 363 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 34 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReceiveCommand.Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED' | 394 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.PACK' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.PACK' | 444 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.PACK' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.PACK' | 593 |
UselessParentheses | Useless parentheses. | 600 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 172 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.PACK' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.PACK' | 546 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 436 |
UselessParentheses | Useless parentheses. | 442 |
UselessParentheses | Useless parentheses. | 449 |
UselessParentheses | Useless parentheses. | 455 |
UselessParentheses | Useless parentheses. | 472 |
UselessParentheses | Useless parentheses. | 496 |
UselessParentheses | Useless parentheses. | 582 |
UselessParentheses | Useless parentheses. | 590 |
UselessParentheses | Useless parentheses. | 663 |
UselessParentheses | Useless parentheses. | 713 |
UselessParentheses | Useless parentheses. | 832 |
UselessParentheses | Useless parentheses. | 942 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 729 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Ref.Storage.NEW' due to existing static import 'org.eclipse.jgit.lib.Ref.Storage.NEW' | 245 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Ref.Storage.NEW' due to existing static import 'org.eclipse.jgit.lib.Ref.Storage.NEW' | 247 |
UselessParentheses | Useless parentheses. | 261–262 |
UselessParentheses | Useless parentheses. | 264–265 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Ref.Storage.PACKED' due to existing static import 'org.eclipse.jgit.lib.Ref.Storage.PACKED' | 409 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Ref.Storage.PACKED' due to existing static import 'org.eclipse.jgit.lib.Ref.Storage.PACKED' | 415 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Ref.Storage.NEW' due to existing static import 'org.eclipse.jgit.lib.Ref.Storage.NEW' | 441 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Ref.Storage.NEW' due to existing static import 'org.eclipse.jgit.lib.Ref.Storage.NEW' | 525 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 546 |
UselessParentheses | Useless parentheses. | 546 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 962–964 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.INDEX' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.INDEX' | 1140 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.INDEX' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.INDEX' | 1143 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 621 |
UselessParentheses | Useless parentheses. | 622 |
UselessParentheses | Useless parentheses. | 741 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 130 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 431 |
UselessParentheses | Useless parentheses. | 437 |
UselessParentheses | Useless parentheses. | 444 |
UselessParentheses | Useless parentheses. | 450 |
UselessParentheses | Useless parentheses. | 467 |
UselessParentheses | Useless parentheses. | 500 |
UselessParentheses | Useless parentheses. | 520 |
UselessParentheses | Useless parentheses. | 577 |
UselessParentheses | Useless parentheses. | 585 |
UselessParentheses | Useless parentheses. | 836 |
UselessParentheses | Useless parentheses. | 885 |
UselessParentheses | Useless parentheses. | 1000 |
UselessParentheses | Useless parentheses. | 1091 |
UselessParentheses | Useless parentheses. | 1106 |
UselessParentheses | Useless parentheses. | 1192 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 207 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 198 |
UselessParentheses | Useless parentheses. | 305 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 356 |
UselessParentheses | Useless parentheses. | 357 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReceiveCommand.Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED' | 509 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 1151 |
UselessParentheses | Useless parentheses. | 1166 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.HEAD' due to existing static import 'org.eclipse.jgit.lib.Constants.HEAD' | 1203 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.R_REFS' due to existing static import 'org.eclipse.jgit.lib.Constants.R_REFS' | 1234 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 286 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 35 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 161 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 353 |
UselessParentheses | Useless parentheses. | 2274 |
UselessParentheses | Useless parentheses. | 2275 |
UselessParentheses | Useless parentheses. | 2365 |
UselessParentheses | Useless parentheses. | 2382 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 102 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackBitmapIndex.FLAG_REUSE' due to existing static import 'org.eclipse.jgit.internal.storage.file.PackBitmapIndex.FLAG_REUSE' | 203 |
UselessParentheses | Useless parentheses. | 237 |
UselessParentheses | Useless parentheses. | 238–239 |
UselessParentheses | Useless parentheses. | 239 |
UselessParentheses | Useless parentheses. | 240 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackBitmapIndex.FLAG_REUSE' due to existing static import 'org.eclipse.jgit.internal.storage.file.PackBitmapIndex.FLAG_REUSE' | 252 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 539 |
UselessParentheses | Useless parentheses. | 551 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 105 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 105 |
UselessParentheses | Useless parentheses. | 449 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 204–205 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 81–82 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 1421 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'StringUtils.toLowerCase' due to existing static import 'org.eclipse.jgit.util.StringUtils.toLowerCase' | 248 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 552–553 |
UselessParentheses | Useless parentheses. | 675 |
UselessParentheses | Useless parentheses. | 675–676 |
UselessParentheses | Useless parentheses. | 675–676 |
UselessParentheses | Useless parentheses. | 804–805 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.OBJECT_ID_LENGTH' due to existing static import 'org.eclipse.jgit.lib.Constants.OBJECT_ID_LENGTH' | 538 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.OBJECT_ID_LENGTH' due to existing static import 'org.eclipse.jgit.lib.Constants.OBJECT_ID_LENGTH' | 570 |
UselessParentheses | Useless parentheses. | 843 |
UselessParentheses | Useless parentheses. | 903 |
UselessParentheses | Useless parentheses. | 904 |
UselessParentheses | Useless parentheses. | 982 |
UselessParentheses | Useless parentheses. | 994 |
UselessParentheses | Useless parentheses. | 1075–1076 |
UselessParentheses | Useless parentheses. | 1077 |
UselessParentheses | Useless parentheses. | 1078 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 600 |
UselessParentheses | Useless parentheses. | 612 |
UselessParentheses | Useless parentheses. | 814 |
UselessParentheses | Useless parentheses. | 815 |
UselessParentheses | Useless parentheses. | 816 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 266 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 86 |
UselessParentheses | Useless parentheses. | 94 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 289 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 57 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 108 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 787–788 |
UselessParentheses | Useless parentheses. | 787–788 |
UselessParentheses | Useless parentheses. | 787 |
UselessParentheses | Useless parentheses. | 861–862 |
Rule | Violation | Line |
---|---|---|
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationBundleLoadingException' | 18 |
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationStringMissingException' | 19 |
Rule | Violation | Line |
---|---|---|
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationBundleLoadingException' | 16 |
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationStringMissingException' | 17 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 251 |
UselessParentheses | Useless parentheses. | 253 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'RawParseUtils.decode' due to existing static import 'org.eclipse.jgit.util.RawParseUtils.decode' | 264 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 162 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 91 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.eclipse.jgit.revwalk.RevWalk' due to existing same package import 'org.eclipse.jgit.revwalk.*' | 115 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.eclipse.jgit.revwalk.ObjectWalk' due to existing same package import 'org.eclipse.jgit.revwalk.*' | 245 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 338 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 529 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 647 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 367–368 |
UselessParentheses | Useless parentheses. | 403 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReceiveCommand.Result.OK' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.OK' | 523 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 142 |
UselessParentheses | Useless parentheses. | 143 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 86–104 |
UselessParentheses | Useless parentheses. | 365–366 |
UselessParentheses | Useless parentheses. | 412 |
UselessParentheses | Useless parentheses. | 432 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 818 |
UselessParentheses | Useless parentheses. | 993 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 234 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED' | 246 |
UselessParentheses | Useless parentheses. | 744 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED' | 810 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Result.REJECTED_OTHER_REASON' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.REJECTED_OTHER_REASON' | 838 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Result.REJECTED_OTHER_REASON' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.REJECTED_OTHER_REASON' | 842 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Result.REJECTED_OTHER_REASON' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.REJECTED_OTHER_REASON' | 848 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.HEAD' due to existing static import 'org.eclipse.jgit.lib.Constants.HEAD' | 1347 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.OBJECT_ID_STRING_LENGTH' due to existing static import 'org.eclipse.jgit.lib.Constants.OBJECT_ID_STRING_LENGTH' | 134 |
UselessParentheses | Useless parentheses. | 290 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 322–323 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 121 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 56 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 156 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 82 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 244–245 |
UselessParentheses | Useless parentheses. | 308 |
UselessParentheses | Useless parentheses. | 309 |
UselessParentheses | Useless parentheses. | 333 |
UselessParentheses | Useless parentheses. | 334 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 1387 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.R_TAGS' due to existing static import 'org.eclipse.jgit.lib.Constants.R_TAGS' | 1973 |
Rule | Violation | Line |
---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_OK' due to existing import 'java.net.HttpURLConnection' | 41 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_NOT_AUTHORITATIVE' due to existing import 'java.net.HttpURLConnection' | 47 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_MOVED_PERM' due to existing import 'java.net.HttpURLConnection' | 53 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_MOVED_TEMP' due to existing import 'java.net.HttpURLConnection' | 59 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_SEE_OTHER' due to existing import 'java.net.HttpURLConnection' | 65 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_NOT_FOUND' due to existing import 'java.net.HttpURLConnection' | 90 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_UNAUTHORIZED' due to existing import 'java.net.HttpURLConnection' | 95 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_FORBIDDEN' due to existing import 'java.net.HttpURLConnection' | 100 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 50 |
UselessParentheses | Useless parentheses. | 51 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 1269 |
UselessParentheses | Useless parentheses. | 1406 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 294–295 |
UselessParentheses | Useless parentheses. | 1524–1525 |
UselessParentheses | Useless parentheses. | 1526–1527 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 44 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 83 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 120 |
UselessParentheses | Useless parentheses. | 127 |
UselessParentheses | Useless parentheses. | 134 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 1366 |
UselessParentheses | Useless parentheses. | 1838–1839 |
UselessParentheses | Useless parentheses. | 1854–1855 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 442 |
UselessParentheses | Useless parentheses. | 442 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 31 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 229–230 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 58 |
UselessParentheses | Useless parentheses. | 170 |
UselessParentheses | Useless parentheses. | 173 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 204 |
UselessParentheses | Useless parentheses. | 258 |
UselessParentheses | Useless parentheses. | 711 |
UselessParentheses | Useless parentheses. | 921 |
UselessParentheses | Useless parentheses. | 1269 |
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 89 |
UselessParentheses | Useless parentheses. | 93 |
UselessParentheses | Useless parentheses. | 94 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 179–180 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'FileMode.GITLINK' due to existing static import 'org.eclipse.jgit.lib.FileMode.GITLINK' | 4 | 180 |
UselessParentheses | Useless parentheses. | 4 | 181–182 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'Format': member interfaces are implicitly static | 3 | 94–161 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'Stage(int)': enum constructors are implicitly private | 3 | 125–127 |
UselessParentheses | Useless parentheses. | 4 | 549 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 171 |
UselessParentheses | Useless parentheses. | 4 | 172 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 771–774 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 399–400 |
UselessParentheses | Useless parentheses. | 4 | 401 |
UselessParentheses | Useless parentheses. | 4 | 535 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.R_HEADS' due to existing static import 'org.eclipse.jgit.lib.Constants.R_HEADS' | 4 | 223 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 422 |
UselessParentheses | Useless parentheses. | 4 | 422 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 162–164 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 266–271 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 383 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 251 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 689–694 |
UselessParentheses | Useless parentheses. | 4 | 756–757 |
UselessParentheses | Useless parentheses. | 4 | 756 |
UselessParentheses | Useless parentheses. | 4 | 757 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 1273–1276 |
UselessParentheses | Useless parentheses. | 4 | 1720 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 372–376 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedFormalParameter | Avoid unused method parameters such as 'state'. | 3 | 183 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 159 |
UselessParentheses | Useless parentheses. | 4 | 172 |
UselessParentheses | Useless parentheses. | 4 | 185 |
UselessParentheses | Useless parentheses. | 4 | 199 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 144 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 124–126 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyWhileStmt | Avoid empty while statements | 3 | 513–515 |
EmptyIfStmt | Avoid empty if statements | 3 | 603–607 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'isFile': private methods cannot be overridden | 3 | 1101–1103 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 12–1235 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 711–713 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 287 |
UselessParentheses | Useless parentheses. | 4 | 409–410 |
UselessParentheses | Useless parentheses. | 4 | 468 |
UselessParentheses | Useless parentheses. | 4 | 516 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 50 |
UselessParentheses | Useless parentheses. | 4 | 100 |
UselessParentheses | Useless parentheses. | 4 | 138 |
UselessParentheses | Useless parentheses. | 4 | 173 |
UselessParentheses | Useless parentheses. | 4 | 217 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 348 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'fs'. | 3 | 336 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 148 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyIfStmt | Avoid empty if statements | 3 | 437–440 |
EmptyIfStmt | Avoid empty if statements | 3 | 442–447 |
UselessParentheses | Useless parentheses. | 4 | 651 |
UselessParentheses | Useless parentheses. | 4 | 729 |
UselessParentheses | Useless parentheses. | 4 | 730 |
UselessParentheses | Useless parentheses. | 4 | 731 |
UselessParentheses | Useless parentheses. | 4 | 732 |
UselessParentheses | Useless parentheses. | 4 | 733 |
UselessParentheses | Useless parentheses. | 4 | 734 |
UselessParentheses | Useless parentheses. | 4 | 793 |
UselessParentheses | Useless parentheses. | 4 | 794 |
UselessParentheses | Useless parentheses. | 4 | 794 |
UselessParentheses | Useless parentheses. | 4 | 794 |
UselessParentheses | Useless parentheses. | 4 | 889 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 901–910 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 904–909 |
UselessParentheses | Useless parentheses. | 4 | 1134–1135 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 1220–1226 |
EmptyWhileStmt | Avoid empty while statements | 3 | 1640–1642 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 92 |
UselessParentheses | Useless parentheses. | 4 | 261 |
UselessParentheses | Useless parentheses. | 4 | 385 |
UselessParentheses | Useless parentheses. | 4 | 385 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 215–221 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 461 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'otp'. | 3 | 30 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 29–31 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 101–108 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 130–132 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 139–141 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 148–150 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 157–159 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 166–168 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 175–177 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 188–190 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 23–25 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 20–22 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'readIncludeFile': the method is declared in an interface type | 3 | 78–79 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'sha1': the method is declared in an interface type | 3 | 124 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'readFile': the method is declared in an interface type | 3 | 144–147 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'readFileWithMode': the method is declared in an interface type | 3 | 171–172 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 559–561 |
UselessParentheses | Useless parentheses. | 4 | 748 |
UselessParentheses | Useless parentheses. | 4 | 749 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 168–170 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifiers 'public static final' on field 'NO_MATCH': the field is declared in an interface type | 3 | 26–38 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'createPathMatcher': the method is declared in an interface type | 3 | 53–57 |
Rule | Violation | Priority | Line |
---|---|---|---|
SimplifiedTernary | Ternary operators that can be simplified with || or && | 3 | 33 |
UselessParentheses | Useless parentheses. | 4 | 97 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 10–310 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Strings.getPathSeparator' due to existing static import 'org.eclipse.jgit.ignore.internal.Strings.getPathSeparator' | 4 | 99 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 105 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 302–304 |
UselessParentheses | Useless parentheses. | 4 | 363 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 31–35 |
UselessParentheses | Useless parentheses. | 4 | 34 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'matches': the method is already in a final class | 3 | 39–41 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 73–75 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–604 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–125 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–758 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–320 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–151 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 116–118 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–204 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–415 |
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'State(boolean)': enum constructors are implicitly private | 3 | 81–83 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReceiveCommand.Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED' | 4 | 394 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–293 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–214 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyIfStmt | Avoid empty if statements | 3 | 48–50 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyIfStmt | Avoid empty if statements | 3 | 61–63 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyIfStmt | Avoid empty if statements | 3 | 56–58 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–244 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–747 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.PACK' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.PACK' | 4 | 444 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 215–217 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.PACK' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.PACK' | 4 | 593 |
UselessParentheses | Useless parentheses. | 4 | 600 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–490 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 172 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.PACK' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.PACK' | 4 | 546 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'isGarbage': the method is already in a final class | 3 | 182–184 |
UselessParentheses | Useless parentheses. | 4 | 436 |
UselessParentheses | Useless parentheses. | 4 | 442 |
UselessParentheses | Useless parentheses. | 4 | 449 |
UselessParentheses | Useless parentheses. | 4 | 455 |
UselessParentheses | Useless parentheses. | 4 | 472 |
UselessParentheses | Useless parentheses. | 4 | 496 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 514 |
UselessParentheses | Useless parentheses. | 4 | 582 |
UselessParentheses | Useless parentheses. | 4 | 590 |
UselessParentheses | Useless parentheses. | 4 | 663 |
UselessParentheses | Useless parentheses. | 4 | 713 |
UselessParentheses | Useless parentheses. | 4 | 832 |
UselessParentheses | Useless parentheses. | 4 | 942 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 150–152 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 326–328 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 346–348 |
UselessParentheses | Useless parentheses. | 4 | 729 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–149 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 341–343 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 143–145 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 89–91 |
Rule | Violation | Priority | Line |
---|---|---|---|
OverrideBothEqualsAndHashcode | Ensure you override both equals() and hashCode() | 3 | 108 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'clear': the method is already in a final class | 3 | 29–31 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'set': the method is already in a final class | 3 | 33–41 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'clear': the method is already in a final class | 3 | 43–47 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'get': the method is already in a final class | 3 | 49–52 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'toEWAHCompressedBitmap': the method is already in a final class | 3 | 54–79 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'block': private methods cannot be overridden | 3 | 81–83 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'mask': private methods cannot be overridden | 3 | 85–87 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'ofObjectType': private methods cannot be overridden | 3 | 316–318 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 79–81 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 166–168 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessOverridingMethod | Overriding method merely calls super | 3 | 164–166 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Ref.Storage.NEW' due to existing static import 'org.eclipse.jgit.lib.Ref.Storage.NEW' | 4 | 245 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Ref.Storage.NEW' due to existing static import 'org.eclipse.jgit.lib.Ref.Storage.NEW' | 4 | 247 |
UselessParentheses | Useless parentheses. | 4 | 261–262 |
UselessParentheses | Useless parentheses. | 4 | 264–265 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Ref.Storage.PACKED' due to existing static import 'org.eclipse.jgit.lib.Ref.Storage.PACKED' | 4 | 409 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Ref.Storage.PACKED' due to existing static import 'org.eclipse.jgit.lib.Ref.Storage.PACKED' | 4 | 415 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Ref.Storage.NEW' due to existing static import 'org.eclipse.jgit.lib.Ref.Storage.NEW' | 4 | 441 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Ref.Storage.NEW' due to existing static import 'org.eclipse.jgit.lib.Ref.Storage.NEW' | 4 | 525 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 196–199 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 315–317 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 716–718 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 202–205 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 275–277 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 546 |
UselessParentheses | Useless parentheses. | 4 | 546 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 389–391 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 426–428 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 544–547 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 942–944 |
UselessParentheses | Useless parentheses. | 4 | 962–964 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 967–969 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.INDEX' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.INDEX' | 4 | 1140 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackExt.INDEX' due to existing static import 'org.eclipse.jgit.internal.storage.pack.PackExt.INDEX' | 4 | 1143 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 78–80 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'maybeContains': the method is already in a final class | 3 | 43–47 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'contains': the method is already in a final class | 3 | 49–91 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'get': private methods cannot be overridden | 3 | 93–96 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'block': private methods cannot be overridden | 3 | 98–100 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'mask': private methods cannot be overridden | 3 | 102–104 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'isEmpty': private methods cannot be overridden | 3 | 106–108 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'or': the method is already in a final class | 3 | 110–114 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'andNot': the method is already in a final class | 3 | 116–120 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'xor': the method is already in a final class | 3 | 122–129 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'getBitmap': the method is already in a final class | 3 | 131–133 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 65–67 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 293–295 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 309–311 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 467 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 560 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 606 |
UselessParentheses | Useless parentheses. | 4 | 621 |
UselessParentheses | Useless parentheses. | 4 | 622 |
UselessParentheses | Useless parentheses. | 4 | 741 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 187–189 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 130 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'decompress': private methods cannot be overridden | 3 | 364–385 |
UselessParentheses | Useless parentheses. | 4 | 431 |
UselessParentheses | Useless parentheses. | 4 | 437 |
UselessParentheses | Useless parentheses. | 4 | 444 |
UselessParentheses | Useless parentheses. | 4 | 450 |
UselessParentheses | Useless parentheses. | 4 | 467 |
UselessParentheses | Useless parentheses. | 4 | 500 |
UselessParentheses | Useless parentheses. | 4 | 520 |
UselessParentheses | Useless parentheses. | 4 | 577 |
UselessParentheses | Useless parentheses. | 4 | 585 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 700–704 |
UselessParentheses | Useless parentheses. | 4 | 836 |
UselessParentheses | Useless parentheses. | 4 | 885 |
UselessParentheses | Useless parentheses. | 4 | 1000 |
UselessParentheses | Useless parentheses. | 4 | 1091 |
UselessParentheses | Useless parentheses. | 4 | 1106 |
UselessParentheses | Useless parentheses. | 4 | 1192 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 207 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 198 |
UselessParentheses | Useless parentheses. | 4 | 305 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 349–351 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 356–358 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 361–363 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'fs'. | 3 | 35 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–514 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 323 |
UselessParentheses | Useless parentheses. | 4 | 356 |
UselessParentheses | Useless parentheses. | 4 | 357 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 414–416 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 435–449 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReceiveCommand.Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED' | 4 | 509 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 14–1477 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 502 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 524–525 |
UselessParentheses | Useless parentheses. | 4 | 1151 |
UselessParentheses | Useless parentheses. | 4 | 1166 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.HEAD' due to existing static import 'org.eclipse.jgit.lib.Constants.HEAD' | 4 | 1203 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.R_REFS' due to existing static import 'org.eclipse.jgit.lib.Constants.R_REFS' | 4 | 1234 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'LooseRef': member interfaces are implicitly static | 3 | 1373–1377 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 184–186 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 13–255 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyIfStmt | Avoid empty if statements | 3 | 36–40 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'StatsRecorder': member interfaces are implicitly static | 3 | 113–182 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'bits': private methods cannot be overridden | 3 | 328–334 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'PageRef': member interfaces are implicitly static | 3 | 774–834 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'CleanupQueue': member interfaces are implicitly static | 3 | 959–962 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 286 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 103–105 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on class 'PackInfo': members of interface types are implicitly public | 3 | 42–84 |
UnnecessaryModifier | Unnecessary modifier 'static' on class 'PackInfo': types nested within an interface type are implicitly static | 3 | 42–84 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 35 |
Rule | Violation | Priority | Line |
---|---|---|---|
ForLoopShouldBeWhileLoop | This for loop could be simplified to a while loop | 3 | 106–116 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'size': the method is already in a final class | 3 | 229–231 |
SimplifiedTernary | Ternary operators that can be simplified with || or && | 3 | 317 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 192 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 276–278 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'set': the method is already in a final class | 3 | 24–28 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'depth': the method is already in a final class | 3 | 31–33 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'type': the method is already in a final class | 3 | 36–38 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'size': the method is already in a final class | 3 | 41–43 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'empty': the method is already in a final class | 3 | 46–48 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'makeNext': the method is already in a final class | 3 | 50–60 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'write': the method is already in a final class | 3 | 77–81 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'write': the method is already in a final class | 3 | 85–105 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'writeFileHeader': the method is already in a final class | 3 | 113–120 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'writeObject': the method is already in a final class | 3 | 137–139 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'writeHeader': the method is already in a final class | 3 | 158–173 |
UselessParentheses | Useless parentheses. | 4 | 161 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'objectHeader': private methods cannot be overridden | 3 | 175–184 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'ofsDelta': private methods cannot be overridden | 3 | 186–193 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'ofsDeltaVarIntLength': private methods cannot be overridden | 3 | 195–200 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'getCopyBuffer': the method is already in a final class | 3 | 207–209 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'length': the method is already in a final class | 3 | 220–222 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'getDigest': the method is already in a final class | 3 | 225–227 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 12–2493 |
UselessParentheses | Useless parentheses. | 4 | 353 |
EmptyWhileStmt | Avoid empty while statements | 3 | 1918–1920 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 2258–2259 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'have': private methods cannot be overridden | 3 | 2273–2276 |
UselessParentheses | Useless parentheses. | 4 | 2274 |
UselessParentheses | Useless parentheses. | 4 | 2275 |
UselessParentheses | Useless parentheses. | 4 | 2365 |
UselessParentheses | Useless parentheses. | 4 | 2382 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 102 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackBitmapIndex.FLAG_REUSE' due to existing static import 'org.eclipse.jgit.internal.storage.file.PackBitmapIndex.FLAG_REUSE' | 4 | 203 |
UselessParentheses | Useless parentheses. | 4 | 237 |
UselessParentheses | Useless parentheses. | 4 | 238–239 |
UselessParentheses | Useless parentheses. | 4 | 239 |
UselessParentheses | Useless parentheses. | 4 | 240 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'PackBitmapIndex.FLAG_REUSE' due to existing static import 'org.eclipse.jgit.internal.storage.file.PackBitmapIndex.FLAG_REUSE' | 4 | 252 |
EmptyWhileStmt | Avoid empty while statements | 3 | 261–263 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 273–277 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–567 |
UselessParentheses | Useless parentheses. | 4 | 539 |
UselessParentheses | Useless parentheses. | 4 | 551 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–572 |
UselessParentheses | Useless parentheses. | 4 | 105 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 281–284 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–382 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–216 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–694 |
UselessParentheses | Useless parentheses. | 4 | 105 |
UselessParentheses | Useless parentheses. | 4 | 449 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 507 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 570 |
UnusedNullCheckInEquals | Invoke equals() on the object you've already ensured is not null | 3 | 677 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–840 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 688 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–309 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–384 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–194 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–254 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 10–174 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 84–86 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 204–205 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 339–341 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 203–205 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 413–415 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'isId': the method is already in a final class | 3 | 44–54 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromString': the method is already in a final class | 3 | 68–76 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromObjectId': the method is already in a final class | 3 | 89–92 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromString': the method is already in a final class | 3 | 101–106 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromHexString': private methods cannot be overridden | 3 | 108–123 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'hexUInt32': private methods cannot be overridden | 3 | 125–136 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prefixCompare': the method is already in a final class | 3 | 217–237 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prefixCompare': the method is already in a final class | 3 | 253–273 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prefixCompare': the method is already in a final class | 3 | 289–309 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'getFirstByte': the method is already in a final class | 3 | 316–318 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'name': the method is already in a final class | 3 | 346–367 |
Rule | Violation | Priority | Line |
---|---|---|---|
SimplifiedTernary | Ternary operators that can be simplified with || or && | 3 | 266 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–753 |
UselessParentheses | Useless parentheses. | 4 | 81–82 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on interface 'Bitmap': members of interface types are implicitly public | 3 | 46–94 |
UnnecessaryModifier | Unnecessary modifier 'public' on interface 'BitmapBuilder': members of interface types are implicitly public | 3 | 100–188 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'BranchRebaseMode(String)': enum constructors are implicitly private | 3 | 42–44 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 1421 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'SectionParser': member interfaces are implicitly static | 3 | 1454–1463 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'ConfigEnum': member interfaces are implicitly static | 3 | 1491–1507 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'StringUtils.toLowerCase' due to existing static import 'org.eclipse.jgit.util.StringUtils.toLowerCase' | 4 | 248 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 95–97 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 35 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 38 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 41 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 44 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 47 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 50 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 75 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 80 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 85 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 90 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 130 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'GpgFormat(String)': enum constructors are implicitly private | 3 | 31–33 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'StageState(int)': enum constructors are implicitly private | 3 | 116–118 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getWorkingTreeIterator': the method is declared in an interface type | 3 | 319 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 505–508 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 550–554 |
UselessParentheses | Useless parentheses. | 4 | 552–553 |
UselessParentheses | Useless parentheses. | 4 | 675 |
UselessParentheses | Useless parentheses. | 4 | 675–676 |
UselessParentheses | Useless parentheses. | 4 | 675–676 |
UselessParentheses | Useless parentheses. | 4 | 804–805 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 12–1244 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.OBJECT_ID_LENGTH' due to existing static import 'org.eclipse.jgit.lib.Constants.OBJECT_ID_LENGTH' | 4 | 538 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.OBJECT_ID_LENGTH' due to existing static import 'org.eclipse.jgit.lib.Constants.OBJECT_ID_LENGTH' | 4 | 570 |
UselessParentheses | Useless parentheses. | 4 | 843 |
UselessParentheses | Useless parentheses. | 4 | 903 |
UselessParentheses | Useless parentheses. | 4 | 904 |
UselessParentheses | Useless parentheses. | 4 | 982 |
UselessParentheses | Useless parentheses. | 4 | 994 |
UselessParentheses | Useless parentheses. | 4 | 1075–1076 |
UselessParentheses | Useless parentheses. | 4 | 1077 |
UselessParentheses | Useless parentheses. | 4 | 1078 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'fromHexString': private methods cannot be overridden | 3 | 208–222 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'newSegment': private methods cannot be overridden | 3 | 307–309 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'computeGrowAt': private methods cannot be overridden | 3 | 311–313 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'equals': private methods cannot be overridden | 3 | 315–322 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'createArray': private methods cannot be overridden | 3 | 225–227 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'Action(String, String)': enum constructors are implicitly private | 3 | 54–57 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 158–161 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'Storage(boolean, boolean)': enum constructors are implicitly private | 3 | 75–78 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 749–751 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 600 |
UselessParentheses | Useless parentheses. | 4 | 612 |
UselessParentheses | Useless parentheses. | 4 | 814 |
UselessParentheses | Useless parentheses. | 4 | 815 |
UselessParentheses | Useless parentheses. | 4 | 816 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 872–873 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 266 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'Key': member interfaces are implicitly static | 3 | 311–332 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessOverridingMethod | Overriding method merely calls super | 3 | 62–64 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'FetchRecurseSubmodulesMode(String)': enum constructors are implicitly private | 3 | 41–43 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–370 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 86 |
UselessParentheses | Useless parentheses. | 4 | 94 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 289 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 39–41 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 57 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 108 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 14–1328 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 375–377 |
UselessParentheses | Useless parentheses. | 4 | 787–788 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 787–823 |
UselessParentheses | Useless parentheses. | 4 | 787–788 |
UselessParentheses | Useless parentheses. | 4 | 787 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 812–814 |
UselessParentheses | Useless parentheses. | 4 | 861–862 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'inCore'. | 3 | 53 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationBundleLoadingException' | 4 | 18 |
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationStringMissingException' | 4 | 19 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationBundleLoadingException' | 4 | 16 |
UnusedImports | Avoid unused imports such as 'org.eclipse.jgit.errors.TranslationStringMissingException' | 4 | 17 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 251 |
UselessParentheses | Useless parentheses. | 4 | 253 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessOverridingMethod | Overriding method merely calls super | 3 | 119–121 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–620 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 215–218 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'RawParseUtils.decode' due to existing static import 'org.eclipse.jgit.util.RawParseUtils.decode' | 4 | 264 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–359 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 162 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 164–166 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 133–136 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 152–167 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'dontFollow': private methods cannot be overridden | 3 | 141–145 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 91 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on class 'Commit': members of interface types are implicitly public | 3 | 76–112 |
UnnecessaryModifier | Unnecessary modifier 'static' on class 'Commit': types nested within an interface type are implicitly static | 3 | 76–112 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.eclipse.jgit.revwalk.RevWalk' due to existing same package import 'org.eclipse.jgit.revwalk.*' | 4 | 115 |
UnnecessaryModifier | Unnecessary modifier 'public' on class 'RevWalk': members of interface types are implicitly public | 3 | 115–242 |
UnnecessaryModifier | Unnecessary modifier 'public' on class 'ObjectWalk': members of interface types are implicitly public | 3 | 245–360 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.eclipse.jgit.revwalk.ObjectWalk' due to existing same package import 'org.eclipse.jgit.revwalk.*' | 4 | 245 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 54 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 55 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 56 |
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 57 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 378 |
EmptyWhileStmt | Avoid empty while statements | 3 | 810–812 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 160–163 |
UselessParentheses | Useless parentheses. | 4 | 338 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 114 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 102 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–306 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getOpenFiles': the method is declared in an interface type | 3 | 30–32 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getOpenBytes': the method is declared in an interface type | 3 | 39–41 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getStats': the method is declared in an interface type | 3 | 47–49 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 12–1222 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 529 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 623 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 312–314 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 323–325 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 353–355 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 398–400 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 454–456 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 474–477 |
UselessParentheses | Useless parentheses. | 4 | 647 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 755–757 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 152–154 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 158–160 |
UselessParentheses | Useless parentheses. | 4 | 367–368 |
UselessParentheses | Useless parentheses. | 4 | 403 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 231–233 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 268–270 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on class 'ConnectivityCheckInfo': types nested within an interface type are implicitly static | 3 | 51–137 |
UnnecessaryModifier | Unnecessary modifier 'public' on class 'ConnectivityCheckInfo': members of interface types are implicitly public | 3 | 51–137 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidThreadGroup | Avoid using java.lang.ThreadGroup; it is not thread safe | 3 | 81 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 282–284 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 308–310 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 392–394 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 394–396 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 396–398 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 401–403 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 406–408 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 39 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 12–528 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReceiveCommand.Result.OK' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.OK' | 4 | 523 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 62–65 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 74–77 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on class 'FtpException': types nested within an interface type are implicitly static | 3 | 31–64 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 188–190 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 142 |
UselessParentheses | Useless parentheses. | 4 | 143 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 159–161 |
EmptyIfStmt | Avoid empty if statements | 3 | 383–385 |
EmptyIfStmt | Avoid empty if statements | 3 | 386–388 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 86–104 |
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'HttpRedirectMode(String)': enum constructors are implicitly private | 3 | 123–125 |
UselessParentheses | Useless parentheses. | 4 | 365–366 |
UselessParentheses | Useless parentheses. | 4 | 412 |
EmptyIfStmt | Avoid empty if statements | 3 | 432–434 |
UselessParentheses | Useless parentheses. | 4 | 432 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 77–80 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 86–88 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 92–94 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 70–73 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 82–84 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 88–90 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 103–107 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'checkIfTooLarge': private methods cannot be overridden | 3 | 698–721 |
UselessParentheses | Useless parentheses. | 4 | 818 |
UselessParentheses | Useless parentheses. | 4 | 993 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1151–1155 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1167–1171 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 306–308 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 110–115 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on field 'DEFAULT': the field is declared in an interface type | 3 | 22–24 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 10–248 |
EmptyIfStmt | Avoid empty if statements | 3 | 69–71 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 234 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'StringReader': member interfaces are implicitly static | 3 | 54–65 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 232–235 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'PushRecurseSubmodulesMode(String)': enum constructors are implicitly private | 3 | 43–45 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 263–265 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED' | 4 | 246 |
UselessParentheses | Useless parentheses. | 4 | 744 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 747–751 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Result.NOT_ATTEMPTED' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.NOT_ATTEMPTED' | 4 | 810 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Result.REJECTED_OTHER_REASON' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.REJECTED_OTHER_REASON' | 4 | 838 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Result.REJECTED_OTHER_REASON' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.REJECTED_OTHER_REASON' | 4 | 842 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Result.REJECTED_OTHER_REASON' due to existing static import 'org.eclipse.jgit.transport.ReceiveCommand.Result.REJECTED_OTHER_REASON' | 4 | 848 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–2269 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 363–365 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 374–376 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 390–392 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1048–1050 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.HEAD' due to existing static import 'org.eclipse.jgit.lib.Constants.HEAD' | 4 | 1347 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1375–1377 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.OBJECT_ID_STRING_LENGTH' due to existing static import 'org.eclipse.jgit.lib.Constants.OBJECT_ID_STRING_LENGTH' | 4 | 134 |
UselessParentheses | Useless parentheses. | 4 | 290 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 521–522 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 322–323 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 121 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifiers 'static final' on field 'EMPTY_CONFIG': the field is declared in an interface type | 3 | 91–113 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'TagOpt(String)': enum constructors are implicitly private | 3 | 54–56 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 129–131 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 231–233 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 56 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 85–87 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 121–123 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 181–183 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 220–222 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 156 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 327–329 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 334–336 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 14–1679 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 832–834 |
EmptyWhileStmt | Avoid empty while statements | 3 | 1130–1132 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1151–1153 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1157–1159 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 82 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 313–316 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 244–245 |
UselessParentheses | Useless parentheses. | 4 | 308 |
UselessParentheses | Useless parentheses. | 4 | 309 |
UselessParentheses | Useless parentheses. | 4 | 333 |
UselessParentheses | Useless parentheses. | 4 | 334 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 11–2415 |
UnnecessaryModifier | Unnecessary modifier 'static' on interface 'IOConsumer': member interfaces are implicitly static | 3 | 203–205 |
EmptyWhileStmt | Avoid empty while statements | 3 | 1069–1071 |
EmptyWhileStmt | Avoid empty while statements | 3 | 1318–1320 |
EmptyWhileStmt | Avoid empty while statements | 3 | 1327–1329 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1374–1376 |
UselessParentheses | Useless parentheses. | 4 | 1387 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1542–1544 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Constants.R_TAGS' due to existing static import 'org.eclipse.jgit.lib.Constants.R_TAGS' | 4 | 1973 |
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 87–91 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 561 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 810–812 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 269–273 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 388–390 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_OK' due to existing import 'java.net.HttpURLConnection' | 4 | 41 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_NOT_AUTHORITATIVE' due to existing import 'java.net.HttpURLConnection' | 4 | 47 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_MOVED_PERM' due to existing import 'java.net.HttpURLConnection' | 4 | 53 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_MOVED_TEMP' due to existing import 'java.net.HttpURLConnection' | 4 | 59 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_SEE_OTHER' due to existing import 'java.net.HttpURLConnection' | 4 | 65 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_NOT_FOUND' due to existing import 'java.net.HttpURLConnection' | 4 | 90 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_UNAUTHORIZED' due to existing import 'java.net.HttpURLConnection' | 4 | 95 |
UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.net.HttpURLConnection.HTTP_FORBIDDEN' due to existing import 'java.net.HttpURLConnection' | 4 | 100 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 12–411 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 169 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 50 |
UselessParentheses | Useless parentheses. | 4 | 51 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 116 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 831 |
UselessParentheses | Useless parentheses. | 4 | 1269 |
UselessParentheses | Useless parentheses. | 4 | 1406 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 294–295 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 440–444 |
EmptyWhileStmt | Avoid empty while statements | 3 | 480–482 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 864–865 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1507–1509 |
UselessParentheses | Useless parentheses. | 4 | 1524–1525 |
UselessParentheses | Useless parentheses. | 4 | 1526–1527 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 44 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 83 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 120 |
UselessParentheses | Useless parentheses. | 4 | 127 |
UselessParentheses | Useless parentheses. | 4 | 134 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 166 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 259 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 394–396 |
EmptyIfStmt | Avoid empty if statements | 3 | 1292–1295 |
UselessParentheses | Useless parentheses. | 4 | 1366 |
UselessParentheses | Useless parentheses. | 4 | 1838–1839 |
UselessParentheses | Useless parentheses. | 4 | 1854–1855 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 1981–1988 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 2320–2322 |
Rule | Violation | Priority | Line |
---|---|---|---|
AvoidUsingOctalValues | Do not start a literal by 0 unless its an octal value | 3 | 57 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 142–160 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 143–159 |
UselessParentheses | Useless parentheses. | 4 | 442 |
UselessParentheses | Useless parentheses. | 4 | 442 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 131–133 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 147–163 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 309–311 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 511 |
SimplifiedTernary | Ternary operators that can be simplified with || or && | 3 | 832 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 31 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'ParseableSimpleDateFormat(String)': enum constructors are implicitly private | 3 | 98–100 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 198–200 |
UselessParentheses | Useless parentheses. | 4 | 229–230 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'setRepository': the method is declared in an interface type | 3 | 305 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'final' on method 'index': private methods cannot be overridden | 3 | 136–140 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'createArray': private methods cannot be overridden | 3 | 143–145 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 58 |
UselessParentheses | Useless parentheses. | 4 | 170 |
UselessParentheses | Useless parentheses. | 4 | 173 |
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyStaticImports | Too many static imports may lead to messy code | 3 | 12–1307 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'match': the method is already in a final class | 3 | 98–105 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseBase10': the method is already in a final class | 3 | 179–213 |
UselessParentheses | Useless parentheses. | 4 | 204 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 207–209 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseLongBase10': the method is already in a final class | 3 | 233–267 |
UselessParentheses | Useless parentheses. | 4 | 258 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 261–263 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseHexInt16': the method is already in a final class | 3 | 284–297 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseHexInt32': the method is already in a final class | 3 | 314–338 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseHexInt64': the method is already in a final class | 3 | 356–404 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseHexInt4': the method is already in a final class | 3 | 415–420 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseTimeZoneOffset': the method is already in a final class | 3 | 434–436 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'parseTimeZoneOffset': the method is already in a final class | 3 | 454–460 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'next': the method is already in a final class | 3 | 473–480 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'nextLF': the method is already in a final class | 3 | 493–495 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'nextLF': the method is already in a final class | 3 | 510–518 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'headerEnd': the method is already in a final class | 3 | 531–540 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'headerStart': the method is already in a final class | 3 | 555–574 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prev': the method is already in a final class | 3 | 587–595 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prevLF': the method is already in a final class | 3 | 608–610 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'prevLF': the method is already in a final class | 3 | 626–635 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'lineMap': the method is already in a final class | 3 | 659–667 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'lineMapOrBinary': the method is already in a final class | 3 | 685–692 |
UselessParentheses | Useless parentheses. | 4 | 711 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'author': the method is already in a final class | 3 | 730–737 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'committer': the method is already in a final class | 3 | 752–761 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'tagger': the method is already in a final class | 3 | 776–789 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'encoding': the method is already in a final class | 3 | 804–814 |
UselessParentheses | Useless parentheses. | 4 | 921 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'commitMessage': the method is already in a final class | 3 | 1217–1228 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'tagMessage': the method is already in a final class | 3 | 1241–1250 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'endOfParagraph': the method is already in a final class | 3 | 1266–1276 |
UselessParentheses | Useless parentheses. | 4 | 1269 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyWhileStmt | Avoid empty while statements | 3 | 67–69 |
AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 79 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'neq': private methods cannot be overridden | 3 | 84–86 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'lc': private methods cannot be overridden | 3 | 88–90 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'isSameObject': the method is declared in an interface type | 3 | 31–33 |
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 89 |
UselessParentheses | Useless parentheses. | 4 | 93 |
UselessParentheses | Useless parentheses. | 4 | 94 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 372–374 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 38–40 |
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyCatchBlock | Avoid empty catch blocks | 3 | 116–118 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 121–123 |
Rule | Violation | Priority | Line |
---|---|---|---|
SimplifiedTernary | Ternary operators that can be simplified with || or && | 3 | 54 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'init': the method is already in a final class | 3 | 562–565 |
UnnecessaryModifier | Unnecessary modifier 'final' on method 'save': the method is already in a final class | 3 | 567–573 |