mirror of https://github.com/qos-ch/slf4j
Merge pull request #105 from jmthibault79/jmthibault79-patch-1
Update comments to point to JIRA equivalents of old Bugzilla bugs
This commit is contained in:
commit
4dcada51bd
|
|
@ -36,7 +36,7 @@ import org.slf4j.Logger;
|
|||
import org.slf4j.LoggerFactory;
|
||||
import org.slf4j.spi.LocationAwareLogger;
|
||||
|
||||
// Based on http://bugzilla.slf4j.org/show_bug.cgi?id=38
|
||||
// Based on http://jira.qos.ch/browse/SLF4J-30
|
||||
|
||||
/**
|
||||
* <p>Bridge/route all JUL log records to the SLF4J API.</p>
|
||||
|
|
@ -259,7 +259,7 @@ public class SLF4JBridgeHandler extends Handler {
|
|||
}
|
||||
Object[] params = record.getParameters();
|
||||
// avoid formatting when there are no or 0 parameters. see also
|
||||
// http://bugzilla.slf4j.org/show_bug.cgi?id=212
|
||||
// http://jira.qos.ch/browse/SLF4J-203
|
||||
if (params != null && params.length > 0) {
|
||||
message = MessageFormat.format(message, params);
|
||||
}
|
||||
|
|
@ -289,7 +289,7 @@ public class SLF4JBridgeHandler extends Handler {
|
|||
// this is a check to avoid calling the underlying logging system
|
||||
// with a null message. While it is legitimate to invoke j.u.l. with
|
||||
// a null message, other logging frameworks do not support this.
|
||||
// see also http://bugzilla.slf4j.org/show_bug.cgi?id=108
|
||||
// see also http://jira.qos.ch/browse/SLF4J-99
|
||||
if (message == null) {
|
||||
message = "";
|
||||
}
|
||||
|
|
|
|||
|
|
@ -291,7 +291,7 @@ public class Category {
|
|||
log(FQCN, p, msg, t);
|
||||
}
|
||||
|
||||
// See also http://bugzilla.slf4j.org/show_bug.cgi?id=168
|
||||
// See also http://jira.qos.ch/browse/SLF4J-159
|
||||
public void log(String FQCN, Priority p, Object msg, Throwable t) {
|
||||
int levelInt = priorityToLevelInt(p);
|
||||
differentiatedLog(null, FQCN, levelInt, msg, t);
|
||||
|
|
|
|||
|
|
@ -51,7 +51,7 @@ public class LogManager {
|
|||
|
||||
/**
|
||||
* Returns a logger instance created by loggerFactory. This method was requested in
|
||||
* <a href="http://bugzilla.slf4j.org/show_bug.cgi?id=234">bug #234</a>. Note that
|
||||
* <a href="http://jira.qos.ch/browse/SLF4J-225">SLF4J-225</a>. Note that
|
||||
* log4j-over-slf4j does not ship with a LoggerFactory implementation. If this
|
||||
* method is called, the caller must provide his/her own implementation.
|
||||
*
|
||||
|
|
|
|||
2
pom.xml
2
pom.xml
|
|
@ -222,7 +222,7 @@
|
|||
</configuration>
|
||||
</plugin>
|
||||
|
||||
<!-- as suggested in http://bugzilla.slf4j.org/show_bug.cgi?id=152 -->
|
||||
<!-- as suggested in http://jira.qos.ch/browse/SLF4J-143 -->
|
||||
<plugin>
|
||||
<groupId>org.codehaus.mojo</groupId>
|
||||
<artifactId>build-helper-maven-plugin</artifactId>
|
||||
|
|
|
|||
|
|
@ -339,7 +339,7 @@ public final class LoggerFactory {
|
|||
throw new IllegalStateException(UNSUCCESSFUL_INIT_MSG);
|
||||
case ONGOING_INITIALIZATION:
|
||||
// support re-entrant behavior.
|
||||
// See also http://bugzilla.slf4j.org/show_bug.cgi?id=106
|
||||
// See also http://jira.qos.ch/browse/SLF4J-97
|
||||
return TEMP_FACTORY;
|
||||
}
|
||||
throw new IllegalStateException("Unreachable code");
|
||||
|
|
|
|||
|
|
@ -102,7 +102,7 @@ public class XLogger extends LoggerWrapper implements Logger {
|
|||
public XLogger(Logger logger) {
|
||||
// If class B extends A, assuming B does not override method x(), the caller
|
||||
// of new B().x() is A and not B, see also
|
||||
// http://bugzilla.slf4j.org/show_bug.cgi?id=114
|
||||
// http://jira.qos.ch/browse/SLF4J-105
|
||||
super(logger, LoggerWrapper.class.getName());
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -128,10 +128,11 @@ public class XLoggerTest extends TestCase {
|
|||
verifyWithLevelAndException((LoggingEvent) listAppender.list.get(1), XLogger.Level.DEBUG, "catching", t);
|
||||
}
|
||||
|
||||
// See http://bugzilla.slf4j.org/show_bug.cgi?id=114
|
||||
// See http://jira.qos.ch/browse/SLF4J-105
|
||||
// formerly http://bugzilla.slf4j.org/show_bug.cgi?id=114
|
||||
public void testLocationExtraction_Bug114() {
|
||||
XLogger logger = XLoggerFactory.getXLogger("UnitTest");
|
||||
int line = 135; // requires update if line numbers change
|
||||
int line = 136; // requires update if line numbers change
|
||||
logger.exit();
|
||||
logger.debug("hello");
|
||||
|
||||
|
|
|
|||
|
|
@ -644,7 +644,7 @@ public final class JDK14LoggerAdapter extends MarkerIgnoringBase implements Loca
|
|||
// construction of location data for disabled log
|
||||
// statements. As of 2008-07-31, callers of this method
|
||||
// do not perform this check. See also
|
||||
// http://bugzilla.slf4j.org/show_bug.cgi?id=90
|
||||
// http://jira.qos.ch/browse/SLF4J-81
|
||||
if (logger.isLoggable(julLevel)) {
|
||||
log(callerFQCN, julLevel, message, argArray, t);
|
||||
}
|
||||
|
|
|
|||
|
|
@ -39,7 +39,7 @@ import org.slf4j.Logger;
|
|||
import org.slf4j.LoggerFactory;
|
||||
|
||||
/**
|
||||
* See http://bugzilla.slf4j.org/show_bug.cgi?id=261
|
||||
* See http://jira.qos.ch/browse/SLF4J-252
|
||||
* @author Thorbjorn Ravn Andersen
|
||||
*/
|
||||
public class LoggerSerializationTest extends TestCase {
|
||||
|
|
|
|||
|
|
@ -48,7 +48,7 @@ import org.slf4j.spi.LocationAwareLogger;
|
|||
* The TRACE level was introduced in log4j version 1.2.12. In order to avoid
|
||||
* crashing the host application, in the case the log4j version in use predates
|
||||
* 1.2.12, the TRACE level will be mapped as DEBUG. See also <a
|
||||
* href="http://bugzilla.slf4j.org/show_bug.cgi?id=68">bug 68</a>.
|
||||
* href="http://jira.qos.ch/browse/SLF4J-59">SLF4J-59</a>.
|
||||
*
|
||||
* @author Ceki Gülcü
|
||||
*/
|
||||
|
|
|
|||
|
|
@ -106,7 +106,8 @@ public class InvocationTest extends TestCase {
|
|||
assertEquals(8, listAppender.list.size());
|
||||
}
|
||||
|
||||
// http://bugzilla.slf4j.org/show_bug.cgi?id=78
|
||||
// http://jira.qos.ch/browse/SLF4J-69
|
||||
// formerly http://bugzilla.slf4j.org/show_bug.cgi?id=78
|
||||
public void testNullParameter_BUG78() {
|
||||
Logger logger = LoggerFactory.getLogger("testNullParameter_BUG78");
|
||||
String[] parameters = null;
|
||||
|
|
|
|||
|
|
@ -80,7 +80,8 @@ public class InvocationTest extends TestCase {
|
|||
logger.error("Hello world 4.", e);
|
||||
}
|
||||
|
||||
// http://bugzilla.slf4j.org/show_bug.cgi?id=78
|
||||
// http://jira.qos.ch/browse/SLF4J-69
|
||||
// formerly http://bugzilla.slf4j.org/show_bug.cgi?id=78
|
||||
public void testNullParameter_BUG78() {
|
||||
Logger logger = LoggerFactory.getLogger("testNullParameter_BUG78");
|
||||
String[] parameters = null;
|
||||
|
|
|
|||
Loading…
Reference in New Issue