diff options
author | Gary Gregory <garydgregory@gmail.com> | 2022-06-13 23:02:14 -0400 |
---|---|---|
committer | Gary Gregory <garydgregory@gmail.com> | 2022-06-13 23:02:14 -0400 |
commit | dceed1be7f279b29702e7a88b2449c88d7bcdb2f (patch) | |
tree | 3054f0228a0eddb300f298fcdab28a9e6e915e98 /src | |
parent | bd7956949eff64db275b209429221c816eca9571 (diff) | |
download | apache-commons-lang-dceed1be7f279b29702e7a88b2449c88d7bcdb2f.tar.gz |
Better way to deal with ignored exceptions and PMD checks.
Gets rid of output like:
<suppressedviolation
filename="...\src\main\java\org\apache\commons\lang3\reflect\MemberUtils.java"
suppressiontype="nopmd" msg="Avoid empty catch blocks" usermsg="
"></suppressedviolation>
Diffstat (limited to 'src')
-rw-r--r-- | src/main/java/org/apache/commons/lang3/text/StrLookup.java | 4 | ||||
-rw-r--r-- | src/main/java/org/apache/commons/lang3/time/DateUtils.java | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/src/main/java/org/apache/commons/lang3/text/StrLookup.java b/src/main/java/org/apache/commons/lang3/text/StrLookup.java index fb009e66e..aef5268b4 100644 --- a/src/main/java/org/apache/commons/lang3/text/StrLookup.java +++ b/src/main/java/org/apache/commons/lang3/text/StrLookup.java @@ -172,8 +172,8 @@ public abstract class StrLookup<V> { if (!key.isEmpty()) { try { return System.getProperty(key); - } catch (final SecurityException scex) { - // Squelched. All lookup(String) will return null. + } catch (final SecurityException ignored) { + // ignored, all lookup(String) will return null. } } return null; diff --git a/src/main/java/org/apache/commons/lang3/time/DateUtils.java b/src/main/java/org/apache/commons/lang3/time/DateUtils.java index 33c61dcad..78fb6b964 100644 --- a/src/main/java/org/apache/commons/lang3/time/DateUtils.java +++ b/src/main/java/org/apache/commons/lang3/time/DateUtils.java @@ -374,7 +374,7 @@ public class DateUtils { if (fdp.parse(str, pos, calendar) && pos.getIndex() == str.length()) { return calendar.getTime(); } - } catch (final IllegalArgumentException ignore) { + } catch (final IllegalArgumentException ignored) { // leniency is preventing calendar from being set } pos.setIndex(0); |