Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch to slf4j #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
import java.util.*;
import java.util.concurrent.atomic.AtomicReference;

import org.apache.log4j.Logger;
import org.mapleir.app.service.ClassTree.InheritanceEdge;
import org.mapleir.stdlib.collections.graph.FastDirectedGraph;
import org.mapleir.stdlib.collections.graph.FastGraphEdge;
Expand All @@ -12,6 +11,8 @@
import org.mapleir.stdlib.collections.graph.algorithms.SimpleDfs;
import org.mapleir.stdlib.util.TabbedStringWriter;
import org.mapleir.asm.ClassNode;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

/**
* A graph to represent the inheritance tree.
Expand All @@ -22,7 +23,7 @@
// edge = (c, super)
// so a dfs goes through edges towards the root
public class ClassTree extends FastDirectedGraph<ClassNode, InheritanceEdge> {
private static final Logger LOGGER = Logger.getLogger(ClassTree.class);
private static final Logger LOGGER = LoggerFactory.getLogger(ClassTree.class);
private static final boolean ALLOW_PHANTOM_CLASSES = true;

private final ApplicationClassSource source;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,13 +3,14 @@
import java.io.IOException;
import java.util.HashSet;

import org.apache.log4j.Logger;
import org.mapleir.asm.ClassHelper;
import org.objectweb.asm.ClassReader;
import org.mapleir.asm.ClassNode;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

public class InstalledRuntimeClassSource extends LibraryClassSource {
protected final Logger LOGGER = Logger.getLogger(InstalledRuntimeClassSource.class);
protected final Logger LOGGER = LoggerFactory.getLogger(InstalledRuntimeClassSource.class);

private final HashSet<String> notContains;

Expand Down Expand Up @@ -77,8 +78,7 @@ private LocateableClassNode __resolve(String name) {
LocateableClassNode node = new LocateableClassNode(this, cn, true);
return node;
} catch(IOException e) {
LOGGER.error(String.format("Could not load class from calling classloader: %s", name));
LOGGER.error(e);
LOGGER.error(String.format("Could not load class from calling classloader: %s", name), e);
notContains.add(name);
return null;
}
Expand Down
Original file line number Diff line number Diff line change
@@ -1,16 +1,10 @@
package org.mapleir.ir.cfg.builder;

import org.apache.log4j.Logger;
import org.mapleir.ir.cfg.BasicBlock;
import org.mapleir.ir.cfg.ControlFlowGraph;
import org.mapleir.ir.code.CodeUnit;
import org.mapleir.ir.code.Stmt;
import org.mapleir.ir.code.expr.VarExpr;
import org.mapleir.ir.code.stmt.copy.CopyVarStmt;
import org.mapleir.ir.locals.Local;
import org.mapleir.ir.locals.impl.StaticMethodLocalsPool;
import org.mapleir.ir.locals.impl.VirtualMethodLocalsPool;
import org.mapleir.ir.utils.CFGUtils;
import org.mapleir.stdlib.collections.map.NullPermeableHashMap;
import org.mapleir.asm.MethodNode;

Expand All @@ -20,8 +14,6 @@

public class ControlFlowGraphBuilder {

private static final Logger LOGGER = Logger.getLogger(ControlFlowGraph.class);

protected final MethodNode method;
protected final ControlFlowGraph graph;
protected final Set<Local> locals;
Expand Down
5 changes: 3 additions & 2 deletions org.mapleir.main/src/main/java/org/mapleir/Boot.java
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
package org.mapleir;

import org.apache.log4j.Logger;
import org.mapleir.app.client.SimpleApplicationContext;
import org.mapleir.app.service.ApplicationClassSource;
import org.mapleir.app.service.CompleteResolvingJarDumper;
Expand All @@ -22,6 +21,8 @@
import org.mapleir.ir.codegen.ControlFlowGraphDumper;
import org.mapleir.asm.ClassNode;
import org.mapleir.asm.MethodNode;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import org.topdank.byteengineer.commons.data.JarInfo;
import org.topdank.byteio.in.SingleJarDownloader;

Expand All @@ -32,7 +33,7 @@

public class Boot {

private static final Logger LOGGER = Logger.getLogger(Boot.class);
private static final Logger LOGGER = LoggerFactory.getLogger(Boot.class);

public static boolean logging = false;
private static long timer;
Expand Down
5 changes: 3 additions & 2 deletions org.mapleir.main/src/main/java/org/mapleir/Boot2.java
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
package org.mapleir;

import org.apache.log4j.Logger;
import org.mapleir.app.client.SimpleApplicationContext;
import org.mapleir.app.service.ApplicationClassSource;
import org.mapleir.app.service.CompleteResolvingJarDumper;
Expand All @@ -24,6 +23,8 @@
import org.mapleir.ir.codegen.ControlFlowGraphDumper;
import org.mapleir.asm.ClassNode;
import org.mapleir.asm.MethodNode;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import org.topdank.byteengineer.commons.data.JarInfo;
import org.topdank.byteio.in.SingleJarDownloader;

Expand All @@ -34,7 +35,7 @@

public class Boot2 {

private static final Logger LOGGER = Logger.getLogger(Boot2.class);
private static final Logger LOGGER = LoggerFactory.getLogger(Boot2.class);

public static boolean logging = false;

Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
package org.mapleir;

import org.apache.log4j.Logger;
import org.mapleir.app.client.SimpleApplicationContext;
import org.mapleir.app.service.ApplicationClassSource;
import org.mapleir.app.service.LibraryClassSource;
Expand All @@ -21,6 +20,8 @@
import org.mapleir.stdlib.util.JavaDescUse;
import org.mapleir.asm.ClassNode;
import org.mapleir.asm.MethodNode;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import org.topdank.byteengineer.commons.data.JarInfo;
import org.topdank.byteio.in.SingleJarDownloader;

Expand All @@ -31,7 +32,7 @@

public class DataFlowDemoBoot {

private static final Logger LOGGER = Logger.getLogger(DataFlowDemoBoot.class);
private static final Logger LOGGER = LoggerFactory.getLogger(DataFlowDemoBoot.class);

public static boolean logging = false;
private static long timer;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,21 +1,22 @@
package org.mapleir;

import org.apache.log4j.Logger;
import org.mapleir.app.service.ApplicationClassSource;
import org.mapleir.app.service.InvocationResolver;
import org.mapleir.stdlib.collections.map.NullPermeableHashMap;
import org.objectweb.asm.Opcodes;
import org.mapleir.asm.ClassNode;
import org.mapleir.asm.FieldNode;
import org.mapleir.asm.MethodNode;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

import java.lang.reflect.Modifier;
import java.util.*;
import java.util.Map.Entry;

public class DefaultInvocationResolver implements InvocationResolver {

private static final Logger LOGGER = Logger.getLogger(DefaultInvocationResolver.class);
private static final Logger LOGGER = LoggerFactory.getLogger(DefaultInvocationResolver.class);

/* trick for no IDE warning */
private static int __debug_level() { return 0; }
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,20 +2,21 @@

import java.util.Set;

import org.apache.log4j.Logger;
import org.mapleir.context.AnalysisContext;
import org.mapleir.ir.cfg.ControlFlowGraph;
import org.mapleir.ir.code.Expr;
import org.mapleir.ir.code.Stmt;
import org.mapleir.ir.code.expr.invoke.Invocation;
import org.mapleir.asm.MethodNode;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

/**
* CallTracer which implements a DFS on the callgraph based on IR instructions.
*/
public class IRCallTracer extends CallTracer {

private static final Logger LOGGER = Logger.getLogger(IRCallTracer.class);
private static final Logger LOGGER = LoggerFactory.getLogger(IRCallTracer.class);

protected final AnalysisContext context;

Expand Down
23 changes: 0 additions & 23 deletions org.mapleir.main/src/main/resources/log4j.properties

This file was deleted.

6 changes: 3 additions & 3 deletions org.mapleir.parent/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,9 @@
<scope>test</scope>
</dependency>
<dependency>
<groupId>log4j</groupId>
<artifactId>log4j</artifactId>
<version>1.2.17</version>
<groupId>org.slf4j</groupId>
<artifactId>slf4j-api</artifactId>
<version>1.7.32</version>
</dependency>
<dependency>
<groupId>com.google.guava</groupId>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
import java.util.List;
import java.util.Map;

import org.apache.log4j.Logger;
import org.mapleir.propertyframework.api.IPropertyDictionary;
import org.mapleir.serviceframework.api.IServiceContext;
import org.mapleir.serviceframework.api.IServiceFactory;
Expand All @@ -16,11 +15,13 @@
import org.mapleir.serviceframework.api.IServiceReferenceHandlerFactory;
import org.mapleir.serviceframework.api.IServiceRegistry;
import org.mapleir.serviceframework.util.ClassServiceQuery;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

// FIXME: synchronise data structures
public class ServiceRegistryImpl implements IServiceRegistry {

private static final Logger LOGGER = Logger.getLogger(ServiceRegistryImpl.class);
private static final Logger LOGGER = LoggerFactory.getLogger(ServiceRegistryImpl.class);

private final ContextLookupHelper map = new ContextLookupHelper();
private final IServiceReferenceHandlerFactory serviceReferenceFactoryHandler = new ServiceReferenceHandlerFactoryImpl();
Expand Down