You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2022/05/31 15:29:07 UTC
[commons-logging] branch master updated: Remove noisy inline comments.
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 <ga...@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: