Fix module-info.java for agents; up SLF4J version (to beta4)

Author: vsilaev

README.md

@@ -123,7 +123,7 @@ Please note, that CDI-specific agent neither requires javaflow.instrument-contin
          -javaagent:<path-to-jar>/javaflow.instrument-cdi-proxy.jar \
 	 <rest-of arguments>
 ```
-CDI functionality is tested with JBoss Weld 2.x - 3.1.0 and Apache OpenWebBeans 1.6.x - 2.0.10. Contribution for other CDI/CDI-like containers (Spring, Google Guice, etc) is welcome.
+CDI functionality is tested with JBoss Weld 2.x - 3.1.1 and Apache OpenWebBeans 1.6.x - 2.0.10. Contribution for other CDI/CDI-like containers (Spring, Google Guice, etc) is welcome.
 
 # More documentation & exmaples
 Examples of the library usage may be found in the [Tascalate JavaFlow Examples](https://github.com/vsilaev/tascalate-javaflow-examples) project. The covered topics are common tasks, inheritance, lambdas support, proxies, usage with CDI containers like JBoss Weld and Apache OpenWebBeans.

net.tascalate.javaflow.tools.cdi-javaagent/src/main/java/module-info.java

@@ -25,6 +25,8 @@
 
     requires net.tascalate.javaflow.spi;
     requires net.tascalate.javaflow.providers.asmx;
+
+    requires transitive org.apache.commons.javaflow.instrumentation.common;
 
     exports org.apache.commons.javaflow.instrumentation.cdi;
 }

net.tascalate.javaflow.tools.instrument/src/main/java/org/apache/commons/javaflow/instrumentation/common/ConfigurableClassFileTransformer.java

@@ -112,7 +112,7 @@ private boolean isSystemClassLoaderParent(ClassLoader maybeParent) {
     private String resolveClassName(String className,
                                     Class<?> classBeingRedefined,
                                     byte[] classfileBuffer) {
-        if (null != className) {
+        if (className != null) {
             return className;
         } else if (classBeingRedefined != null) {
             return classBeingRedefined.getName().replace('.', '/');

net.tascalate.javaflow.tools.javaagent/src/main/java/module-info.java

@@ -21,5 +21,7 @@
     requires net.tascalate.javaflow.spi;
     requires net.tascalate.javaflow.providers.asmx;
 
+    requires transitive org.apache.commons.javaflow.instrumentation.common;
+
     exports org.apache.commons.javaflow.instrumentation; 
 }

pom.xml

@@ -80,12 +80,12 @@
 			<dependency>
 				<groupId>org.slf4j</groupId>
 				<artifactId>slf4j-api</artifactId>
-				<version>1.8.0-beta2</version>
+				<version>1.8.0-beta4</version>
 			</dependency>
 			<dependency>
 				<groupId>org.slf4j</groupId>
 				<artifactId>slf4j-simple</artifactId>
-				<version>1.8.0-beta2</version>
+				<version>1.8.0-beta4</version>
 				<scope>runtime</scope>
 			</dependency>
 			<dependency>