Fixed bugs found by Copilot review

Signed-off-by: Thomas Calmant <thomas.calmant@gmail.com>

Author: tcalmant

javaobj/v1/beans.py

@@ -199,7 +199,7 @@ def __init__(self, classdesc=None):
         self.classdesc = classdesc
 
     def __hash__(self):
-        return list.__hash__(self)
+        return object.__hash__(self)
 
 
 class JavaByteArray(JavaObject):

javaobj/v1/transformers.py

@@ -136,7 +136,7 @@ def __extra_loading__(self, unmarshaller, ident=0):
             """
             # Ignore the blockdata opid
             (opid,) = unmarshaller._readStruct(">B")
-            if opid != ClassDescFlags.SC_BLOCK_DATA:
+            if opid != TerminalCode.TC_BLOCKDATA:
                 raise ValueError("Start of block data not found")
 
             # Read HashMap fields

javaobj/v1/unmarshaller.py

@@ -83,7 +83,7 @@
 # Convertion of a Java type char to its NumPy equivalent
 NUMPY_TYPE_MAP = {
     TypeCode.TYPE_BYTE: "B",
-    TypeCode.TYPE_CHAR: "b",
+    TypeCode.TYPE_CHAR: ">u2",
     TypeCode.TYPE_DOUBLE: ">d",
     TypeCode.TYPE_FLOAT: ">f",
     TypeCode.TYPE_INTEGER: ">i",
@@ -508,7 +508,8 @@ def do_object(self, parent=None, ident=0):
         # classdata[]
 
         if (
-            classdesc.flags & ClassDescFlags.SC_EXTERNALIZABLE
+            classdesc is not None
+            and classdesc.flags & ClassDescFlags.SC_EXTERNALIZABLE
             and not classdesc.flags & ClassDescFlags.SC_BLOCK_DATA
         ):
             # TODO:
@@ -831,8 +832,9 @@ def _oops_dump_state(self, ignore_remaining_data=False):
             "(2nd line is an actual position!):"
         )
 
-        # Do not use a keyword argument
-        self.object_stream.seek(-16, os.SEEK_CUR)
+        # Do not use a keyword argument; clamp to avoid seeking before start
+        current_pos = self.object_stream.tell()
+        self.object_stream.seek(max(0, current_pos - 16))
         position = self.object_stream.tell()
         the_rest = self.object_stream.read()
 

javaobj/v2/transformers.py

@@ -506,7 +506,7 @@ class NumpyArrayTransformer(ObjectTransformer):
     # Convertion of a Java type char to its NumPy equivalent
     NUMPY_TYPE_MAP = {
         TypeCode.TYPE_BYTE: "B",
-        TypeCode.TYPE_CHAR: "b",
+        TypeCode.TYPE_CHAR: ">u2",
         TypeCode.TYPE_DOUBLE: ">d",
         TypeCode.TYPE_FLOAT: ">f",
         TypeCode.TYPE_INTEGER: ">i",