Skip to content

Commit

Permalink
+ elf4j-engine version bump
Browse files Browse the repository at this point in the history
  • Loading branch information
q3769 committed Nov 10, 2023
1 parent 49fa170 commit 47f6f95
Show file tree
Hide file tree
Showing 3 changed files with 37 additions and 20 deletions.
26 changes: 24 additions & 2 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@

<groupId>io.github.elf4j</groupId>
<artifactId>slf4j-elf4j</artifactId>
<version>6.0.8</version>
<version>6.0.9</version>
<packaging>jar</packaging>
<name>slf4j-elf4j</name>
<description>SLF4J binding to use ELF4J native logging implementation</description>
Expand Down Expand Up @@ -72,7 +72,7 @@
<dependency>
<groupId>io.github.elf4j</groupId>
<artifactId>elf4j-engine</artifactId>
<version>13.0.5</version>
<version>15.0.0</version>
</dependency>
<dependency>
<groupId>org.junit.jupiter</groupId>
Expand Down Expand Up @@ -141,6 +141,28 @@
</execution>
</executions>
</plugin>
<plugin>
<groupId>com.diffplug.spotless</groupId>
<artifactId>spotless-maven-plugin</artifactId>
<version>2.40.0</version>
<executions>
<execution>
<id>spotless-apply-id</id>
<phase>process-sources</phase>
<goals>
<goal>apply</goal>
</goals>
</execution>
</executions>
<configuration>
<java>
<palantirJavaFormat>
<version>2.38.0</version>
</palantirJavaFormat>
<formatAnnotations/>
</java>
</configuration>
</plugin>
</plugins>
</build>

Expand Down
18 changes: 7 additions & 11 deletions src/main/java/elf4j/engine/slf4/Elf4jLogger.java
Original file line number Diff line number Diff line change
Expand Up @@ -27,14 +27,13 @@

import elf4j.Level;
import elf4j.engine.NativeLogger;
import java.util.EnumMap;
import org.slf4j.Logger;
import org.slf4j.Marker;
import org.slf4j.event.LoggingEvent;
import org.slf4j.spi.DefaultLoggingEventBuilder;
import org.slf4j.spi.LoggingEventAware;

import java.util.EnumMap;

/**
*
*/
Expand All @@ -55,8 +54,7 @@ public class Elf4jLogger implements Logger, LoggingEventAware {
private final NativeLogger nativeLogger;

/**
* @param nativeLogger
* wrapped elf4j-impl logger
* @param nativeLogger wrapped elf4j-impl logger
*/
public Elf4jLogger(NativeLogger nativeLogger) {
this.nativeLogger = nativeLogger;
Expand All @@ -68,7 +66,7 @@ private static Level translate(org.slf4j.event.Level slf4jEventLevel) {

@Override
public String getName() {
return nativeLogger.getOwnerClassName();
return nativeLogger.getDeclaringClassName();
}

@Override
Expand Down Expand Up @@ -373,7 +371,8 @@ public void error(Marker marker, String msg, Throwable t) {

@Override
public void log(LoggingEvent event) {
service(nativeLogger.atLevel(translate(event.getLevel())),
service(
nativeLogger.atLevel(translate(event.getLevel())),
LOGGING_SERVICE_CLASS_SLF4J_FLUENT_API,
event.getThrowable(),
event.getMessage(),
Expand All @@ -388,11 +387,8 @@ private boolean isEnabled(Level elf4jLevel) {
return nativeLogger.atLevel(elf4jLevel).isEnabled();
}

private void service(NativeLogger delegate,
Class<?> loggingServiceInterface,
Throwable t,
String msg,
Object... args) {
private void service(
NativeLogger delegate, Class<?> loggingServiceInterface, Throwable t, String msg, Object... args) {
delegate.getLogService().log(delegate, loggingServiceInterface, t, msg, args);
}
}
13 changes: 6 additions & 7 deletions src/main/java/elf4j/engine/slf4/Elf4jLoggerFactory.java
Original file line number Diff line number Diff line change
Expand Up @@ -25,28 +25,27 @@

package elf4j.engine.slf4;

import elf4j.engine.NativeLoggerFactory;
import elf4j.engine.NativeLogServiceProvider;
import java.util.HashMap;
import java.util.Map;
import org.slf4j.ILoggerFactory;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

import java.util.HashMap;
import java.util.Map;

/**
*
*/
public class Elf4jLoggerFactory implements ILoggerFactory {
private static final Class<LoggerFactory> LOGGING_SERVICE_ACCESS_CLASS = LoggerFactory.class;
private final NativeLoggerFactory nativeLoggerFactory;
private final NativeLogServiceProvider nativeLogServiceProvider;
private final Map<String, Elf4jLogger> elf4jLoggerMap = new HashMap<>();

Elf4jLoggerFactory() {
nativeLoggerFactory = new NativeLoggerFactory(LOGGING_SERVICE_ACCESS_CLASS);
nativeLogServiceProvider = new NativeLogServiceProvider(LOGGING_SERVICE_ACCESS_CLASS);
}

@Override
public Logger getLogger(String name) {
return elf4jLoggerMap.computeIfAbsent(name, k -> new Elf4jLogger(nativeLoggerFactory.logger()));
return elf4jLoggerMap.computeIfAbsent(name, k -> new Elf4jLogger(nativeLogServiceProvider.logger()));
}
}

0 comments on commit 47f6f95

Please sign in to comment.