From 31cef1474ace1b37e94ad762ece3c125cb96b1f3 Mon Sep 17 00:00:00 2001 From: Michal Kordas Date: Sat, 17 Oct 2015 20:33:42 +0200 Subject: [PATCH] Fix various typos --- .../src/main/resources/rulesets/java/controversial.xml | 2 +- .../pmd/lang/java/rule/basic/xml/BrokenNullCheck.xml | 2 +- .../java/rule/design/xml/AvoidReassigningParameters.xml | 2 +- .../pmd/lang/java/rule/design/xml/CloseResource.xml | 8 ++++---- .../java/rule/design/xml/SimpleDateFormatNeedsLocale.xml | 2 +- .../pmd/lang/java/rule/junit/xml/JUnitSpelling.xml | 2 +- .../pmd/lang/java/rule/junit/xml/JUnitStaticSuite.xml | 2 +- .../java/rule/loggingjakartacommons/xml/ProperLogger.xml | 2 +- .../migrating/xml/JUnit4TestShouldUseTestAnnotation.xml | 2 +- .../lang/java/rule/unnecessary/xml/UselessParentheses.xml | 2 +- .../lang/java/rule/unusedcode/xml/UnusedPrivateMethod.xml | 2 +- .../pmd/lang/plsql/dfa/StatementAndBraceFinderTest.java | 4 ++-- src/site/markdown/overview/changelog-old.md | 2 +- 13 files changed, 17 insertions(+), 17 deletions(-) diff --git a/pmd-java/src/main/resources/rulesets/java/controversial.xml b/pmd-java/src/main/resources/rulesets/java/controversial.xml index 565a2b79c9..9cf92e4fe9 100644 --- a/pmd-java/src/main/resources/rulesets/java/controversial.xml +++ b/pmd-java/src/main/resources/rulesets/java/controversial.xml @@ -791,7 +791,7 @@ public void checkRequests() { When you write a public method, you should be thinking in terms of an API. If your method is public, it means other class will use it, therefore, you want (or need) to offer a comprehensive and evolutive API. If you pass a lot of information -as a simple series of Strings, you may think of using an Object to represent all those information. You'll get a simplier +as a simple series of Strings, you may think of using an Object to represent all those information. You'll get a simpler API (such as doWork(Workload workload), rather than a tedious series of Strings) and more importantly, if you need at some point to pass extra data, you'll be able to do so by simply modifying or extending Workload without any modification to your API. diff --git a/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/basic/xml/BrokenNullCheck.xml b/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/basic/xml/BrokenNullCheck.xml index 708a751a91..ed5b2ea8b7 100644 --- a/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/basic/xml/BrokenNullCheck.xml +++ b/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/basic/xml/BrokenNullCheck.xml @@ -155,7 +155,7 @@ public class Foo { 0 3 - bug #1131 CloseResource should complain if code betwen declaration of resource and try: Code betwen declaration and try, should fail + bug #1131 CloseResource should complain if code between declaration of resource and try: Code between declaration and try, should fail Connection 1 - bug #1131 CloseResource should complain if code betwen declaration of resource and try: Creation inside try, ok + bug #1131 CloseResource should complain if code between declaration of resource and try: Creation inside try, ok Connection 0 - bug #1131 CloseResource should complain if code betwen declaration of resource and try: No sentences between creation and try, ok + bug #1131 CloseResource should complain if code between declaration of resource and try: No sentences between creation and try, ok Connection 0 - #1259 CloseResource rule ignores conditionnals within finally blocks + #1259 CloseResource rule ignores conditionals within finally blocks 1 1 2 1 0 1 1