Split getException into method and function, so it can be used in native functions.

diff --git a/src/bind/dobinding.cpp b/src/bind/dobinding.cpp
index 29d61b722d0810af3aa2bebfee0e2c949c7c2944..e2157de0b3f69b584e9b6c2640496e1ac95c3a9a 100644 (file)
--- a/src/bind/dobinding.cpp
+++ b/src/bind/dobinding.cpp
@@ -74,6 +74,11 @@ typedef struct
 static jlong getPointer(JNIEnv *env, jobject obj)
 {
     jfieldID id = env->GetFieldID(env->GetObjectClass(obj), "_pointer", "J");
+    if (!id)
+        {
+        err("getPointer: %s", getException(env).c_str());
+        return 0;
+               }
     jlong val = env->GetLongField(obj, id);
     return val;
 }
@@ -81,9 +86,49 @@ static jlong getPointer(JNIEnv *env, jobject obj)
 static void setPointer(JNIEnv *env, jobject obj, jlong val)
 {
     jfieldID id = env->GetFieldID(env->GetObjectClass(obj), "_pointer", "J");
+    if (!id)
+        {
+        err("setPointer: %s", getException(env).c_str());
+        return;
+               }
     env->SetLongField(obj, id, val);
 }
 
+static void JNICALL BaseObject_construct
+  (JNIEnv *env, jobject obj)
+{
+    setPointer(env, obj, 0L);
+}
+
+static void JNICALL BaseObject_destruct
+  (JNIEnv *env, jobject obj)
+{
+    NodePtr *ptr = (NodePtr *)getPointer(env, obj);
+    if (ptr)
+        {
+        delete ptr;
+        }
+    setPointer(env, obj, 0L);
+}
+
+
+static JNINativeMethod nm_BaseObject[] =
+{
+{ (char *)"construct", (char *)"()V", (void *)BaseObject_construct },
+{ (char *)"destruct",  (char *)"()V", (void *)BaseObject_destruct  },
+{ NULL,  NULL, NULL }
+};
+
+static NativeClass nc_BaseObject =
+{
+    "org/inkscape/cmn/BaseObject",
+    nm_BaseObject
+};
+
+//########################################################################
+//# BASE OBJECT
+//########################################################################
+
 static void JNICALL DOMBase_construct
   (JNIEnv *env, jobject obj)
 {
@@ -114,6 +159,8 @@ static NativeClass nc_DOMBase =
     "org/inkscape/dom/DOMBase",
     nm_DOMBase
 };
+
+
 //########################################################################
 //# DOMImplementation
 //########################################################################
@@ -157,6 +204,7 @@ static NativeClass nc_DOMImplementation =
  */   
 static NativeClass *allClasses[] =
 {
+    &nc_BaseObject,
     &nc_DOMBase,
     &nc_DOMImplementation,
     NULL

diff --git a/src/bind/javabind-private.h b/src/bind/javabind-private.h
index 5a810480ba6dfcf51eaf7f07b61fbe4ebc0a9fbd..c4dbd5e9a8530aee1a0a6d4f64a3aa807980117b 100644 (file)
--- a/src/bind/javabind-private.h
+++ b/src/bind/javabind-private.h
@@ -91,6 +91,13 @@ void msg(const char *fmt, ...);
 //# UTILITY
 //########################################################################
 
+String normalizePath(const String &str);
+
+String getExceptionString(JNIEnv *env);
+
+String getException(JNIEnv *env)
+    { return getExceptionString(env); }
+
 jint getInt(JNIEnv *env, jobject obj, const char *name);
 
 void setInt(JNIEnv *env, jobject obj, const char *name, jint val);

diff --git a/src/bind/javabind.cpp b/src/bind/javabind.cpp
index b03d562ceea3da7807669a1b4805e2f2e6144b12..1424a527b457cc60ce67519c42c95b071d6b5018 100644 (file)
--- a/src/bind/javabind.cpp
+++ b/src/bind/javabind.cpp
@@ -102,6 +102,21 @@ String normalizePath(const String &str)
        return buf;
 }
 
+String getExceptionString(JNIEnv *env)
+{
+    String buf;
+    jthrowable exc = env->ExceptionOccurred();
+    if (!exc)
+        return buf;
+    jclass cls = env->GetObjectClass(exc);
+    jmethodID mid = env->GetMethodID(cls, "toString", "()Ljava/lang/String;");
+    jstring jstr = (jstring) env->CallObjectMethod(exc, mid);
+    const char *str = env->GetStringUTFChars(jstr, JNI_FALSE);
+    buf.append(str);
+    env->ReleaseStringUTFChars(jstr, str);
+    env->ExceptionClear();
+       return buf;
+}
 
 jint getInt(JNIEnv *env, jobject obj, const char *name)
 {
@@ -901,6 +916,7 @@ bool JavaBinderyImpl::callStatic(int type,
 
 
 
+
 /**
  * Fetch the last exception from the JVM, if any.  Clear it to
  * continue processing
@@ -909,18 +925,7 @@ bool JavaBinderyImpl::callStatic(int type,
  */  
 String JavaBinderyImpl::getException()
 {
-    String buf;
-    jthrowable exc = env->ExceptionOccurred();
-    if (!exc)
-        return buf;
-    jclass cls = env->GetObjectClass(exc);
-    jmethodID mid = env->GetMethodID(cls, "toString", "()Ljava/lang/String;");
-    jstring jstr = (jstring) env->CallObjectMethod(exc, mid);
-    const char *str = env->GetStringUTFChars(jstr, JNI_FALSE);
-    buf.append(str);
-    env->ReleaseStringUTFChars(jstr, str);
-    env->ExceptionClear();
-       return buf;
+    return getExceptionString(env);
 }