Commit Graph

575 Commits

Author SHA1 Message Date
4808cb089f Merge branch 'issue-199'
Closes #203 (rebased onto pmd/5.4.x)
2017-01-27 17:10:56 +01:00
420c6d45a3 Fixes #199 - False positives with ternary operator
- I deected a couple of extra related issues with ternary operators which are now fixed.
2017-01-27 16:42:24 +01:00
137b76a561 Merge branch 'issue-185'
Closes #196 (rebased onto pmd/5.5.x)
2017-01-27 11:42:38 +01:00
36fcdd252c Fix NPE error in CPD reported under #185 2017-01-27 11:26:51 +01:00
756c2f6774 Fix build 2017-01-21 14:46:35 +01:00
2efe75fac5 Merge branch 'unit-test-for-pr-170' 2017-01-21 13:53:57 +01:00
12f2431bb4 Fix build 2017-01-21 13:43:04 +01:00
620fed2732 Merge branch 'unit-test-for-pr-170' into pmd/5.5.x 2017-01-21 12:13:47 +01:00
ba260e753c Adjust unit test to verify console encoding of XML formatter 2017-01-21 11:58:59 +01:00
4c1890d56c Add unit test to verify encoding of XML formatter 2017-01-21 11:58:59 +01:00
056554c4e3 Merge branch 'bug-1556' 2017-01-08 14:03:37 +01:00
923aab8065 Merge branch 'bug-1556' into pmd/5.5.x 2017-01-08 13:58:13 +01:00
c0e1c3e731 Slightly improve performance of XPath Rule UseLocaleWithCaseConversions 2017-01-08 13:51:44 +01:00
af3ad2bc02 Fixes #1556 [java] UseLocaleWithCaseConversions does not works with ResultSet (false negative) 2017-01-08 12:12:56 +01:00
5b3142433b Merge branch 'doclint' 2017-01-07 17:06:05 +01:00
e1d12dd4f8 Further Javadoc doclint fixes
References #1516 [build] doclint error during build
2017-01-07 17:05:03 +01:00
62e33b8e03 Merge branch 'bug-177'
Fixes #177
2017-01-06 10:42:21 +01:00
c9514e563a Merge branch 'bug-177' into pmd/5.5.x
Fixes #177
2017-01-06 10:35:45 +01:00
af9d78c01a Update changelog 2017-01-06 10:16:45 +01:00
767c9aae79 Bugfix singular field + lambda 2017-01-06 10:07:19 +01:00
c9d3bd3e9c [java] Javadoc doclint fixes
References #1516 [build] doclint error during build
2017-01-05 21:54:48 +01:00
2c714e1fbc Merge branch 'pr-164'
Closes #164 (rebased onto pmd/5.5.x)
2017-01-03 12:19:45 +01:00
a499711990 Fix compile error 2017-01-03 12:12:11 +01:00
932ad7dd2b Reduce memory allocations during symbol table 2017-01-03 12:07:30 +01:00
79d7869782 Merge branch 'pr-162'
Closes #162 (rebased onto pmd/5.4.x)
2017-01-03 11:54:26 +01:00
992e5547bb Merge branch 'pr-162' into pmd/5.5.x
Closes #162 (rebased onto pmd/5.4.x)
2017-01-03 11:45:47 +01:00
6283316e51 Avoid redundant method calls and improve codebase
- Avoid making calls within for loops, specially costly ones
    such as `getQualifiedTypeNames`
 - Don't create lists when they are empty.
 - Create lists of proper size to avoid resizing / oversizing
 - I'm seeing a ~5% improvement. We are reaching the point were noise
    makes it hard to detect improvements. We should attack GC cycles soon.
2017-01-03 11:41:33 +01:00
c9ada390db Merge branch 'pr-161'
Closes #161 (rebased onto pmd/5.5.x)
2017-01-03 11:33:24 +01:00
b950929b7c Improve symboltable codebase
- Move shared code to pmd-core
 - Allow search methods to stop searching when they want to
 - If we are looking for a variable declaration, just search among those and not all name declarations
 - This is roughtly another 10% improvement on symbol table performance
2017-01-03 11:22:42 +01:00
b6bc06d3d2 Avoid comparing strings all time 2017-01-03 11:21:13 +01:00
5d9010672d Merge branch 'pr-159'
Closes #159 (rebased onto pmd/5.4.x)
2017-01-03 10:34:42 +01:00
55b4d81fbf Merge branch 'pr-159' into pmd/5.5.x
Closes #159 (rebased onto pmd/5.4.x)
2017-01-03 10:27:39 +01:00
54db0ffbd9 [java] UnnecessaryFullyQualifiedNameRule handles same file class conflicts
- This resolves [#1555](https://sourceforge.net/p/pmd/bugs/1555/)
 - If a class is defined in the same file that conflicts with an import,
    it's not considered a violation
2017-01-03 10:18:29 +01:00
68fd556c9c Merge branch 'add-clone-test' of https://github.com/Monits/pmd into pr-157 2017-01-03 10:09:02 +01:00
d6ef39f4d4 Recycle the test cases of AvoidConstantsInInterface 2017-01-03 09:59:29 +01:00
7a46613043 Merge branch 'issue-1512' of https://github.com/Monits/pmd into pr-156 2017-01-03 09:29:56 +01:00
544d22f9b8 Merge branch 'checksum-aware' of https://github.com/Monits/pmd into pr-145 2016-12-23 18:21:50 +01:00
e280151987 pmd-cpp/pmd-java: checkstyle fixes 2016-12-16 15:48:53 +01:00
eb2d942ed4 pmd-core: checkstyle fixes 2016-12-16 12:32:18 +01:00
36c34eee81 Merge branch 'pr-141'
Closes #141 (rebased for pmd/5.4.x)
2016-12-10 10:29:56 +01:00
5ded17c5b5 Merge branch 'pr-141' into pmd/5.5.x
Closes #141 (rebased for pmd/5.4.x)
2016-12-10 10:20:39 +01:00
28a24dec2a [java] Speedup PreserveStackTrace by over 7X
- By avoiding to look at all variable declarators we can greatly improve
    analysis speed without loosing accurancy
2016-12-10 10:13:37 +01:00
21a012484d Merge branch 'pr-140'
Closes #140 (rebased onto pmd/5.4.x)
2016-12-09 09:59:06 +01:00
bb8b604152 Merge branch 'pr-140' into pmd/5.5.x
Closes #140 (rebased onto pmd/5.4.x)
2016-12-09 09:50:36 +01:00
f4e727d274 [java] Make CloneMethodMustImplementCloneable over 500x faster
- This was an excruciatingly slow rule, the slowest by far of all PMD
 - We speed it up by:
    * checking early if we are analyzing a `clone()` method to cut the AST tree
    * making better use of type information available
    * only performing additional checks when we know we can't rely on
       available type information
2016-12-09 09:44:23 +01:00
b5b9ed7bc7 Merge branch 'symboltable-tidyup' of https://github.com/Monits/pmd into pr-139 2016-12-09 09:29:55 +01:00
df0b52fa62 Uncomment the test scenario for issue 681
- This asserts the issue has actually been fixed and makes sure it won't return
2016-12-08 18:37:29 -03:00
efebb1ef6b Unify AvoidConstantsInterface and ConstantsInInterface
- We stick to ConstantsInInterface since it covers all scenarios the other one did,
    and allows greater configuration.
 - The existing test cases were already covered, so we can safely drop them.
2016-12-08 18:04:37 -03:00
a9a6859b34 Merge branch 'pr-154' 2016-12-07 14:34:07 -03:00
aa7a1f42f5 Merge branch 'pr-154' into pmd/5.5.x 2016-12-07 14:32:30 -03:00