This is an automated email from the ASF dual-hosted git repository. ggregory pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-logging.git
The following commit(s) were added to refs/heads/master by this push: new 59bbadf Remove noisy inline comments. 59bbadf is described below commit 59bbadfc5aa920912d74491eca411b9f38e9838e Author: Gary Gregory <garydgreg...@gmail.com> AuthorDate: Tue May 31 11:29:02 2022 -0400 Remove noisy inline comments. --- .../java/org/apache/commons/logging/impl/Log4JLogger.java | 2 -- .../java/org/apache/commons/logging/log4j/StandardTests.java | 12 ------------ 2 files changed, 14 deletions(-) diff --git a/src/main/java/org/apache/commons/logging/impl/Log4JLogger.java b/src/main/java/org/apache/commons/logging/impl/Log4JLogger.java index bac7364..d41c405 100644 --- a/src/main/java/org/apache/commons/logging/impl/Log4JLogger.java +++ b/src/main/java/org/apache/commons/logging/impl/Log4JLogger.java @@ -46,8 +46,6 @@ public class Log4JLogger implements Log, Serializable { /** Serializable version identifier. */ private static final long serialVersionUID = 5160705895411730424L; - // ------------------------------------------------------------- Attributes - /** The fully qualified name of the Log4JLogger class. */ private static final String FQCN = Log4JLogger.class.getName(); diff --git a/src/test/java/org/apache/commons/logging/log4j/StandardTests.java b/src/test/java/org/apache/commons/logging/log4j/StandardTests.java index 28c4afa..3546f87 100644 --- a/src/test/java/org/apache/commons/logging/log4j/StandardTests.java +++ b/src/test/java/org/apache/commons/logging/log4j/StandardTests.java @@ -51,10 +51,6 @@ public abstract class StandardTests extends TestCase { public Throwable throwable; } - // ------------------------------------------------------------------- - // JUnit Infrastructure Methods - // ------------------------------------------------------------------- - /** * Set up instance variables required by this test case. */ @@ -71,10 +67,6 @@ public abstract class StandardTests extends TestCase { LogFactory.releaseAll(); } - // ----------------------------------------------------------- - // abstract methods - // ----------------------------------------------------------- - /** * Modify log4j's setup so that all messages actually logged get redirected * into the specified list. @@ -84,8 +76,6 @@ public abstract class StandardTests extends TestCase { */ public abstract void setUpTestAppender(List logEvents) throws Exception; - // ----------------------------------------------------------- Test Methods - /** * Test that a LogFactory gets created as expected. */ @@ -146,8 +136,6 @@ public abstract class StandardTests extends TestCase { checkLoggingEvents(logEvents, true); } - // -------------------------------------------------------- Support Methods - /** * Verify that the TestAppender has received the expected * number of messages. This assumes that: