🎨 Remove some compiler warnings

xfeep · xfeep · commit 76fadc200c41 · 2019-11-27T14:20:53.000+08:00
diff --git a/src/java/nginx/clojure/NginxClojureRT.java b/src/java/nginx/clojure/NginxClojureRT.java
@@ -560,7 +560,11 @@ public static void initStringAddrMapsByNativeAddr(Map<String, Long> map, long ad
 	
 	private static synchronized void initMemIndex(long idxpt) {
 		getLog();
-		initUnsafe();	
+		initUnsafe();
+		
+		if (log.isDebugEnabled()) {
+			log.debug("jvm classpath:\n " + System.getProperty("java.class.path"));
+		}
 	    
 		NGINX_MAIN_THREAD = Thread.currentThread();
 		
diff --git a/src/java/nginx/clojure/util/NginxPubSubTopic.java b/src/java/nginx/clojure/util/NginxPubSubTopic.java
@@ -138,7 +138,7 @@ public void publish(String message) {
 		NginxClojureRT.broadcastEvent(MiniConstants.POST_EVENT_TYPE_PUB, id);
 	}
 	
-	@SuppressWarnings({"rawtypes", "unchecked"})
+	@SuppressWarnings({"rawtypes"})
 	public <T> PubSubListenerData<T> subscribe(T data, NginxPubSubListener<T> listener) {
 		Set<PubSubListenerData> pds = topicSubs.get(topicId);
 		if (pds == null) {
diff --git a/src/java/nginx/clojure/util/NginxSharedHashMap.java b/src/java/nginx/clojure/util/NginxSharedHashMap.java
@@ -67,7 +67,7 @@ public class NginxSharedHashMap<K, V> implements ConcurrentMap<K, V>{
 
 	private native static long natomicAddNumber(long ctx, int ktype, Object keyBuf, long offset, long len, int vtype, long delta);
 	
-	private native static long nvisit(long ctx, SharedMapSimpleVisitor visitor);
+	private native static long nvisit(long ctx, @SuppressWarnings("rawtypes") SharedMapSimpleVisitor visitor);
 
 	private NginxSharedHashMap() {
 	}
@@ -109,7 +109,6 @@ private final static int visit(int ktype, long kaddr, long ksize, int vtype, lon
 		return visitor.visit(native2JavaObject(ktype, kaddr, ksize), native2JavaObject(vtype, vaddr, vsize));
 	}
 	
-	@SuppressWarnings({"rawtypes", "unchecked"})
 	public static <KT, VT> NginxSharedHashMap<KT, VT> build(String name) {
 		return new NginxSharedHashMap<>(name);
 	}
@@ -327,7 +326,6 @@ public void clear() {
 	}
 
 
-	@SuppressWarnings({"rawtypes", "unchecked"})
 	@Override
 	public Set<K> keySet() {
 		NginxClojureRT.getLog().warn("NginxSharedHashMap.keySet is quite expensive operation DO NOT use it at non-debug case!!!");
@@ -340,7 +338,6 @@ public Set<K> keySet() {
 	}
 
 
-	@SuppressWarnings({"rawtypes", "unchecked"})
 	@Override
 	public Collection<V> values() {
 		NginxClojureRT.getLog().warn("NginxSharedHashMap.values is quite expensive operation DO NOT use it at non-debug case!!!");
@@ -353,7 +350,6 @@ public Collection<V> values() {
 	}
 
 
-	@SuppressWarnings({"rawtypes", "unchecked"})
 	@Override
 	public Set<java.util.Map.Entry<K, V>> entrySet() {
 		NginxClojureRT.getLog().warn("NginxSharedHashMap.entrySet is quite expensive operation DO NOT use it at non-debug case!!!");

Original file line number	Diff line number	Diff line change
`@@ -138,7 +138,7 @@ public void publish(String message) {`
`138`	`138`	`NginxClojureRT.broadcastEvent(MiniConstants.POST_EVENT_TYPE_PUB, id);`
`139`	`139`	`}`
`140`	`140`
`141`		`- @SuppressWarnings({"rawtypes", "unchecked"})`
	`141`	`+ @SuppressWarnings({"rawtypes"})`
`142`	`142`	`public <T> PubSubListenerData<T> subscribe(T data, NginxPubSubListener<T> listener) {`
`143`	`143`	`Set<PubSubListenerData> pds = topicSubs.get(topicId);`
`144`	`144`	`if (pds == null) {`
Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ public class NginxSharedHashMap<K, V> implements ConcurrentMap<K, V>{`
`67`	`67`
`68`	`68`	`private native static long natomicAddNumber(long ctx, int ktype, Object keyBuf, long offset, long len, int vtype, long delta);`
`69`	`69`
`70`		`- private native static long nvisit(long ctx, SharedMapSimpleVisitor visitor);`
	`70`	`+ private native static long nvisit(long ctx, @SuppressWarnings("rawtypes") SharedMapSimpleVisitor visitor);`
`71`	`71`
`72`	`72`	`private NginxSharedHashMap() {`
`73`	`73`	`}`
`@@ -109,7 +109,6 @@ private final static int visit(int ktype, long kaddr, long ksize, int vtype, lon`
`109`	`109`	`return visitor.visit(native2JavaObject(ktype, kaddr, ksize), native2JavaObject(vtype, vaddr, vsize));`
`110`	`110`	`}`
`111`	`111`
`112`		`- @SuppressWarnings({"rawtypes", "unchecked"})`
`113`	`112`	`public static <KT, VT> NginxSharedHashMap<KT, VT> build(String name) {`
`114`	`113`	`return new NginxSharedHashMap<>(name);`
`115`	`114`	`}`
`@@ -327,7 +326,6 @@ public void clear() {`
`327`	`326`	`}`
`328`	`327`
`329`	`328`
`330`		`- @SuppressWarnings({"rawtypes", "unchecked"})`
`331`	`329`	`@Override`
`332`	`330`	`public Set<K> keySet() {`
`333`	`331`	`NginxClojureRT.getLog().warn("NginxSharedHashMap.keySet is quite expensive operation DO NOT use it at non-debug case!!!");`
`@@ -340,7 +338,6 @@ public Set<K> keySet() {`
`340`	`338`	`}`
`341`	`339`
`342`	`340`
`343`		`- @SuppressWarnings({"rawtypes", "unchecked"})`
`344`	`341`	`@Override`
`345`	`342`	`public Collection<V> values() {`
`346`	`343`	`NginxClojureRT.getLog().warn("NginxSharedHashMap.values is quite expensive operation DO NOT use it at non-debug case!!!");`
`@@ -353,7 +350,6 @@ public Collection<V> values() {`
`353`	`350`	`}`
`354`	`351`
`355`	`352`
`356`		`- @SuppressWarnings({"rawtypes", "unchecked"})`
`357`	`353`	`@Override`
`358`	`354`	`public Set<java.util.Map.Entry<K, V>> entrySet() {`
`359`	`355`	`NginxClojureRT.getLog().warn("NginxSharedHashMap.entrySet is quite expensive operation DO NOT use it at non-debug case!!!");`