From 4800ef85d0039dddc5af15481a3b6b2335eb2335 Mon Sep 17 00:00:00 2001
From: Paul Campbell <pcampbell@kemitix.net>
Date: Mon, 5 Sep 2016 17:56:58 +0100
Subject: [PATCH] EitherLogOfThrow: add support for java.util.logging.Logger

---
 README.md                                     | 2 +-
 src/main/resources/net/kemitix/checkstyle.xml | 4 ++++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 4f6ae77..486dc5b 100644
--- a/README.md
+++ b/README.md
@@ -2255,7 +2255,7 @@ Map<Long, String> idTable = new HashMap<Long, String>();
 
 Checks that when an exception is caught, that if it is logged then it is not also re-thrown. Log or throw; one or the other or neither, but not both.
 
-> TODO: loggingMethodNames: add 'log'
+Accepts `java.util.logging.Logger` and `org.slf4j.Logger`.
 
 #### [EnumValueName](http://sevntu-checkstyle.github.io/sevntu.checkstyle/apidocs/com/github/sevntu/checkstyle/checks/naming/EnumValueNameCheck.html)
 
diff --git a/src/main/resources/net/kemitix/checkstyle.xml b/src/main/resources/net/kemitix/checkstyle.xml
index 8a49ad6..6a77cab 100644
--- a/src/main/resources/net/kemitix/checkstyle.xml
+++ b/src/main/resources/net/kemitix/checkstyle.xml
@@ -219,6 +219,10 @@
         <module name="ConfusingConditionCheck"/> <!-- prevents negation within an "if" expression if "else" is present -->
         <module name="DiamondOperatorForVariableDefinitionCheck"/> <!-- use the diamond operator -->
         <module name="EitherLogOrThrowCheck"/> <!-- log or throw an exception - don't do both -->
+        <module name="EitherLogOrThrowCheck"> <!-- log or throw an exception - don't do both -->
+            <property name="loggerFullyQualifiedClassName" value="java.util.logging.Logger"/>
+            <property name="loggingMethodNames" value="log, severe, warning, info, config, fine, finer, finest, "/>
+        </module>
         <module name="ForbidCCommentsInMethodsCheck"/> <!-- prevent /* C-style */ comments inside methods -->
         <module name="ForbidReturnInFinallyBlockCheck"/> <!-- returns in finally override returns elsewhere in method -->
         <module name="ForbidThrowAnonymousExceptionsCheck"/> <!-- only throw concrete exceptions -->