Fix logs in LTI-CIVIL library loader

sarxos · sarxos · commit a04d1caabd45 · 2017-09-13T18:51:36.000+02:00
diff --git a/webcam-capture-drivers/driver-lti-civil/src/main/java/com/github/sarxos/webcam/ds/civil/LtiCivilLoader.java b/webcam-capture-drivers/driver-lti-civil/src/main/java/com/github/sarxos/webcam/ds/civil/LtiCivilLoader.java
@@ -16,7 +16,7 @@ public class LtiCivilLoader {
 	private static final Logger LOG = LoggerFactory.getLogger(LtiCivilLoader.class);
 
 	/**
-	 * Will be called until JVM shutdown.
+	 * Will be called on JVM shutdown.
 	 * 
 	 * @author Bartosz Firyn (SarXos)
 	 */
@@ -34,7 +34,7 @@ public void run() {
 			super.run();
 			if (file.exists()) {
 				if (!file.delete()) {
-					LOG.warn(String.format("JVM was not able to remove file %s", file));
+					LOG.warn("JVM was not able to remove file {}", file);
 				}
 			}
 		}
@@ -65,10 +65,10 @@ private static long copy(InputStream input, OutputStream output) throws IOExcept
 	}
 
 	protected static void load(String lib) {
-		LOG.info("Loading native library: " + lib);
+		LOG.info("Loading native library: {}", lib);
 		try {
 			System.loadLibrary(lib);
-			LOG.info("DLL has been loaded from memory: " + lib);
+			LOG.info("DLL has been loaded from memory: {}", lib);
 		} catch (UnsatisfiedLinkError e) {
 			try {
 				load("civil-" + System.currentTimeMillis(), lib);

Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ public class LtiCivilLoader {`
`16`	`16`	`private static final Logger LOG = LoggerFactory.getLogger(LtiCivilLoader.class);`
`17`	`17`
`18`	`18`	`/**`
`19`		`- * Will be called until JVM shutdown.`
	`19`	`+ * Will be called on JVM shutdown.`
`20`	`20`	`*`
`21`	`21`	`* @author Bartosz Firyn (SarXos)`
`22`	`22`	`*/`
`@@ -34,7 +34,7 @@ public void run() {`
`34`	`34`	`super.run();`
`35`	`35`	`if (file.exists()) {`
`36`	`36`	`if (!file.delete()) {`
`37`		`- LOG.warn(String.format("JVM was not able to remove file %s", file));`
	`37`	`+ LOG.warn("JVM was not able to remove file {}", file);`
`38`	`38`	`}`
`39`	`39`	`}`
`40`	`40`	`}`
`@@ -65,10 +65,10 @@ private static long copy(InputStream input, OutputStream output) throws IOExcept`
`65`	`65`	`}`
`66`	`66`
`67`	`67`	`protected static void load(String lib) {`
`68`		`- LOG.info("Loading native library: " + lib);`
	`68`	`+ LOG.info("Loading native library: {}", lib);`
`69`	`69`	`try {`
`70`	`70`	`System.loadLibrary(lib);`
`71`		`- LOG.info("DLL has been loaded from memory: " + lib);`
	`71`	`+ LOG.info("DLL has been loaded from memory: {}", lib);`
`72`	`72`	`} catch (UnsatisfiedLinkError e) {`
`73`	`73`	`try {`
`74`	`74`	`load("civil-" + System.currentTimeMillis(), lib);`