Code cleanup and updated version

Author: jonatino

build.gradle

@@ -2,7 +2,7 @@ apply plugin: 'java'
 apply plugin: 'maven'
 
 group = 'com.beaudoin.jmm'
-version = '1.5'
+version = '1.6'
 
 repositories {
     mavenCentral()

src/main/java/com/beaudoin/jmm/misc/Cacheable.java

@@ -25,6 +25,7 @@
 package com.beaudoin.jmm.misc;
 
 import com.sun.jna.Pointer;
+import com.sun.jna.platform.win32.WinDef;
 import com.sun.jna.ptr.IntByReference;
 
 import java.util.HashMap;
@@ -34,16 +35,17 @@
 public final class Cacheable {
 
 	private static final Map<Integer, MemoryBuffer> bufferCache = new HashMap<>();
-	private static final Function<Integer, MemoryBuffer> butterCreate = MemoryBuffer::new;
+	private static final Function<Integer, MemoryBuffer> bufferCreate = MemoryBuffer::new;
 
 	private static final Map<Integer, byte[]> arrayCache = new HashMap<>();
 	private static final Function<Integer, byte[]> arrayCreate = byte[]::new;
 
 	private static final Pointer cachedPointer = new Pointer(0);
 	public static final IntByReference INT_BY_REF = new IntByReference();
+	public static final WinDef.DWORD DWORD_ZERO = new WinDef.DWORD();
 
 	public static MemoryBuffer buffer(int size) {
-		return bufferCache.computeIfAbsent(size, butterCreate);
+		return bufferCache.computeIfAbsent(size, bufferCreate);
 	}
 
 	public static byte[] array(int size) {

src/main/java/com/beaudoin/jmm/misc/Strings.java

@@ -34,20 +34,20 @@
  */
 public final class Strings {
 
-    private static Map<Long, String> map = new HashMap<>(16_982);
+    private static Map<Long, String> stringCache = new HashMap<>(16_982);
 
     public static String transform(byte[] bytes) {
         long hash = LongHashFunction.xx_r39().hashBytes(bytes);
-        if (map.containsKey(hash)) {
-            return map.get(hash);
+        if (stringCache.containsKey(hash)) {
+            return stringCache.get(hash);
         }
         for (int i = 0; i < bytes.length; i++) {
             if (bytes[i] == 0) {
                 bytes[i] = 32;
             }
         }
         String string = new String(bytes).split(" ")[0].trim().intern();
-        map.put(hash, string);
+        stringCache.put(hash, string);
         return string;
     }
 

src/main/java/com/beaudoin/jmm/process/AbstractProcess.java

@@ -23,7 +23,7 @@ public int id() {
 
 	@Override
 	public Module findModule(String moduleName) {
-		Module module = modules.get(moduleName);
+		Module module = modules.isEmpty() ? null : modules.get(moduleName);
 		if (module == null) {
 			initModules();
 		}

src/main/java/com/beaudoin/jmm/process/DataSource.java

@@ -51,7 +51,7 @@ default double readDouble(long address) {
     }
 
     default String readString(long address, int length) {
-        byte[] bytes = Cacheable.array(length);
+        byte[] bytes = new byte[length];
         read(address, bytes.length).get(bytes);
         return Strings.transform(bytes);
     }

src/main/java/com/beaudoin/jmm/process/impl/win32/Win32Process.java

@@ -45,7 +45,6 @@ public final class Win32Process extends AbstractProcess {
     public Win32Process(int id, Pointer handle) {
         super(id);
         this.handle = handle;
-        initModules();
     }
 
     public Pointer pointer() {
@@ -59,7 +58,7 @@ public void initModules() {
         try {
             while (Kernel32.Module32NextW(snapshot, entry)) {
                 String name = entry.szModule();
-                modules.putIfAbsent(name, new Module(this, name, entry.getPointer(), entry.modBaseSize.intValue()));
+                modules.putIfAbsent(name, new Module(this, name, entry.hModule.getPointer(), entry.modBaseSize.intValue()));
             }
         } finally {
             Kernel32.CloseHandle(snapshot);