Author: mturk
Date: Mon May  4 06:13:05 2009
New Revision: 771190

URL: http://svn.apache.org/viewvc?rev=771190&view=rev
Log:
Use package macros

Modified:
    commons/sandbox/runtime/trunk/src/main/native/include/acr_private.h
    commons/sandbox/runtime/trunk/src/main/native/os/unix/file.c
    commons/sandbox/runtime/trunk/src/main/native/os/unix/syslog.c
    commons/sandbox/runtime/trunk/src/main/native/os/unix/uuid.c
    commons/sandbox/runtime/trunk/src/main/native/os/win32/file.c
    commons/sandbox/runtime/trunk/src/main/native/os/win32/syslog.c
    commons/sandbox/runtime/trunk/src/main/native/os/win32/uuid.c
    commons/sandbox/runtime/trunk/src/main/native/shared/error.c
    commons/sandbox/runtime/trunk/src/main/native/shared/memory.c

Modified: commons/sandbox/runtime/trunk/src/main/native/include/acr_private.h
URL: 
http://svn.apache.org/viewvc/commons/sandbox/runtime/trunk/src/main/native/include/acr_private.h?rev=771190&r1=771189&r2=771190&view=diff
==============================================================================
--- commons/sandbox/runtime/trunk/src/main/native/include/acr_private.h 
(original)
+++ commons/sandbox/runtime/trunk/src/main/native/include/acr_private.h Mon May 
 4 06:13:05 2009
@@ -31,9 +31,20 @@
  *
  */
 
-#define ACR_CLASS_PATH                    "org/apache/commons/runtime/"
+#define ACR_CLASS_PATH              "org/apache/commons/runtime/"
+#define ACR_EXCEPTION_CLASS_PATH    "org/apache/commons/runtime/exception/"
+#define ACR_IO_CLASS_PATH           "org/apache/commons/runtime/io/"
+#define ACR_NET_CLASS_PATH          "org/apache/commons/runtime/net/"
+#define ACR_UTIL_CLASS_PATH         "org/apache/commons/runtime/util/"
+
 #define ACR_JNI_EXPORT_DECLARE(RT, CL, MN)  \
     JNIEXPORT RT JNICALL Java_org_apache_commons_runtime_##CL##_##MN
+#define ACR_IO_EXPORT_DECLARE(RT, CL, MN)  \
+    JNIEXPORT RT JNICALL Java_org_apache_commons_runtime_io_##CL##_##MN
+#define ACR_NET_EXPORT_DECLARE(RT, CL, MN)  \
+    JNIEXPORT RT JNICALL Java_org_apache_commons_runtime_net_##CL##_##MN
+#define ACR_UTIL_EXPORT_DECLARE(RT, CL, MN)  \
+    JNIEXPORT RT JNICALL Java_org_apache_commons_runtime_util_##CL##_##MN
 
 
 /* ACR_ALIGN() is only to be used to align on a power of 2 boundary */

Modified: commons/sandbox/runtime/trunk/src/main/native/os/unix/file.c
URL: 
http://svn.apache.org/viewvc/commons/sandbox/runtime/trunk/src/main/native/os/unix/file.c?rev=771190&r1=771189&r2=771190&view=diff
==============================================================================
--- commons/sandbox/runtime/trunk/src/main/native/os/unix/file.c (original)
+++ commons/sandbox/runtime/trunk/src/main/native/os/unix/file.c Mon May  4 
06:13:05 2009
@@ -108,7 +108,7 @@
     return -1;
 }
 
-ACR_JNI_EXPORT_DECLARE(int, io_File, ftype0)(ACR_JNISTDARGS, jstring pathname)
+ACR_IO_EXPORT_DECLARE(int, File, ftype0)(ACR_JNISTDARGS, jstring pathname)
 {
     int type = ACR_FT_UNKFILE;
 
@@ -120,9 +120,9 @@
     return type;
 }
 
-ACR_JNI_EXPORT_DECLARE(jboolean, io_File, mkslink0)(ACR_JNISTDARGS,
-                                                    jstring target,
-                                                    jstring lnkname)
+ACR_IO_EXPORT_DECLARE(jboolean, File, mkslink0)(ACR_JNISTDARGS,
+                                                jstring target,
+                                                jstring lnkname)
 {
     jboolean rc = JNI_FALSE;
 
@@ -146,9 +146,9 @@
     return rc;
 }
 
-ACR_JNI_EXPORT_DECLARE(jboolean, io_File, mkhlink0)(ACR_JNISTDARGS,
-                                                    jstring target,
-                                                    jstring lnkname)
+ACR_IO_EXPORT_DECLARE(jboolean, File, mkhlink0)(ACR_JNISTDARGS,
+                                                jstring target,
+                                                jstring lnkname)
 {
     jboolean rc = JNI_FALSE;
 
@@ -172,8 +172,8 @@
     return rc;
 }
 
-ACR_JNI_EXPORT_DECLARE(jstring, io_File, target0)(ACR_JNISTDARGS,
-                                                  jstring lnkname)
+ACR_IO_EXPORT_DECLARE(jstring, File, target0)(ACR_JNISTDARGS,
+                                              jstring lnkname)
 {
     jstring rv = NULL;
 
@@ -288,7 +288,7 @@
     return -1;
 }
 
-ACR_JNI_EXPORT_DECLARE(int, io_File, fprot0)(ACR_JNISTDARGS, jstring pathname)
+ACR_IO_EXPORT_DECLARE(int, File, fprot0)(ACR_JNISTDARGS, jstring pathname)
 {
     int prot = 0;
 

Modified: commons/sandbox/runtime/trunk/src/main/native/os/unix/syslog.c
URL: 
http://svn.apache.org/viewvc/commons/sandbox/runtime/trunk/src/main/native/os/unix/syslog.c?rev=771190&r1=771189&r2=771190&view=diff
==============================================================================
--- commons/sandbox/runtime/trunk/src/main/native/os/unix/syslog.c (original)
+++ commons/sandbox/runtime/trunk/src/main/native/os/unix/syslog.c Mon May  4 
06:13:05 2009
@@ -34,15 +34,15 @@
 
 #define LOG_MSG_DOMAIN  "runtime"
 
-ACR_JNI_EXPORT_DECLARE(void, util_Syslog, init0)(ACR_JNISTDARGS)
+ACR_UTIL_EXPORT_DECLARE(void, Syslog, init0)(ACR_JNISTDARGS)
 {
 
     UNREFERENCED_STDARGS;
     openlog(LOG_MSG_DOMAIN, LOG_CONS | LOG_PID, LOG_LOCAL0);
 }
 
-ACR_JNI_EXPORT_DECLARE(void, util_Syslog, init1)(ACR_JNISTDARGS,
-                                                 jstring domain)
+ACR_UTIL_EXPORT_DECLARE(void, Syslog, init1)(ACR_JNISTDARGS,
+                                             jstring domain)
 {
 
     UNREFERENCED_O;
@@ -53,15 +53,15 @@
     } END_WITH_CSTR(domain);
 }
 
-ACR_JNI_EXPORT_DECLARE(void, util_Syslog, close0)(ACR_JNISTDARGS)
+ACR_UTIL_EXPORT_DECLARE(void, Syslog, close0)(ACR_JNISTDARGS)
 {
     UNREFERENCED_STDARGS;
     closelog();
 }
 
-ACR_JNI_EXPORT_DECLARE(void, util_Syslog, log0)(ACR_JNISTDARGS,
-                                                jint level,
-                                                jstring msg)
+ACR_UTIL_EXPORT_DECLARE(void, Syslog, log0)(ACR_JNISTDARGS,
+                                            jint level,
+                                            jstring msg)
 {
     UNREFERENCED_O;
 

Modified: commons/sandbox/runtime/trunk/src/main/native/os/unix/uuid.c
URL: 
http://svn.apache.org/viewvc/commons/sandbox/runtime/trunk/src/main/native/os/unix/uuid.c?rev=771190&r1=771189&r2=771190&view=diff
==============================================================================
--- commons/sandbox/runtime/trunk/src/main/native/os/unix/uuid.c (original)
+++ commons/sandbox/runtime/trunk/src/main/native/os/unix/uuid.c Mon May  4 
06:13:05 2009
@@ -22,7 +22,7 @@
 #if HAVE_UUID_UUID_H
 #include <uuid/uuid.h>
 
-ACR_JNI_EXPORT_DECLARE(jbyteArray, util_UUID, getb0)(ACR_JNISTDARGS)
+ACR_UTIL_EXPORT_DECLARE(jbyteArray, UUID, getb0)(ACR_JNISTDARGS)
 {
     jbyteArray rv;
     uuid_t id;
@@ -43,7 +43,7 @@
  * a decent random number generator.
  */
 
-ACR_JNI_EXPORT_DECLARE(jbyteArray, util_UUID, getb0)(ACR_JNISTDARGS)
+ACR_UTIL_EXPORT_DECLARE(jbyteArray, UUID, getb0)(ACR_JNISTDARGS)
 {
     UNREFERENCED_O;
     ACR_ThrowException(_E, THROW_NMARK, ACR_EX_ENOTIMPL, 0);

Modified: commons/sandbox/runtime/trunk/src/main/native/os/win32/file.c
URL: 
http://svn.apache.org/viewvc/commons/sandbox/runtime/trunk/src/main/native/os/win32/file.c?rev=771190&r1=771189&r2=771190&view=diff
==============================================================================
--- commons/sandbox/runtime/trunk/src/main/native/os/win32/file.c (original)
+++ commons/sandbox/runtime/trunk/src/main/native/os/win32/file.c Mon May  4 
06:13:05 2009
@@ -90,7 +90,7 @@
     return -1;
 }
 
-ACR_JNI_EXPORT_DECLARE(int, io_File, ftype0)(ACR_JNISTDARGS, jstring pathname)
+ACR_IO_EXPORT_DECLARE(int, File, ftype0)(ACR_JNISTDARGS, jstring pathname)
 {
     int type = ACR_FT_UNKFILE;
 
@@ -102,9 +102,9 @@
     return type;
 }
 
-ACR_JNI_EXPORT_DECLARE(jboolean, io_File, mkslink0)(ACR_JNISTDARGS,
-                                                    jstring target,
-                                                    jstring lnkname)
+ACR_IO_EXPORT_DECLARE(jboolean, File, mkslink0)(ACR_JNISTDARGS,
+                                                jstring target,
+                                                jstring lnkname)
 {
     jboolean rc = JNI_FALSE;
 
@@ -136,9 +136,9 @@
     return rc;
 }
 
-ACR_JNI_EXPORT_DECLARE(jboolean, io_File, mkhlink0)(ACR_JNISTDARGS,
-                                                    jstring target,
-                                                    jstring lnkname)
+ACR_IO_EXPORT_DECLARE(jboolean, File, mkhlink0)(ACR_JNISTDARGS,
+                                                jstring target,
+                                                jstring lnkname)
 {
     jboolean rc = JNI_FALSE;
 
@@ -162,8 +162,8 @@
     return rc;
 }
 
-ACR_JNI_EXPORT_DECLARE(jstring, io_File, target0)(ACR_JNISTDARGS,
-                                                  jstring lnkname)
+ACR_IO_EXPORT_DECLARE(jstring, File, target0)(ACR_JNISTDARGS,
+                                              jstring lnkname)
 {
     jstring rv = NULL;
 
@@ -352,7 +352,7 @@
     return 0;
 }
 
-ACR_JNI_EXPORT_DECLARE(int, io_File, fprot0)(ACR_JNISTDARGS, jstring pathname)
+ACR_IO_EXPORT_DECLARE(int, File, fprot0)(ACR_JNISTDARGS, jstring pathname)
 {
     int prot = 0;
 

Modified: commons/sandbox/runtime/trunk/src/main/native/os/win32/syslog.c
URL: 
http://svn.apache.org/viewvc/commons/sandbox/runtime/trunk/src/main/native/os/win32/syslog.c?rev=771190&r1=771189&r2=771190&view=diff
==============================================================================
--- commons/sandbox/runtime/trunk/src/main/native/os/win32/syslog.c (original)
+++ commons/sandbox/runtime/trunk/src/main/native/os/win32/syslog.c Mon May  4 
06:13:05 2009
@@ -91,15 +91,15 @@
     }
 }
 
-ACR_JNI_EXPORT_DECLARE(void, util_Syslog, init0)(ACR_JNISTDARGS)
+ACR_UTIL_EXPORT_DECLARE(void, Syslog, init0)(ACR_JNISTDARGS)
 {
     UNREFERENCED_STDARGS;
 
     acr_init_log_source(LOG_MSG_DOMAIN);
 }
 
-ACR_JNI_EXPORT_DECLARE(void, util_Syslog, init1)(ACR_JNISTDARGS,
-                                                 jstring domain)
+ACR_UTIL_EXPORT_DECLARE(void, Syslog, init1)(ACR_JNISTDARGS,
+                                             jstring domain)
 {
     UNREFERENCED_O;
     WITH_CSTR(domain) {
@@ -107,9 +107,9 @@
     } END_WITH_CSTR(domain);
 }
 
-ACR_JNI_EXPORT_DECLARE(void, util_Syslog, log0)(ACR_JNISTDARGS,
-                                                jint level,
-                                                jstring msg)
+ACR_UTIL_EXPORT_DECLARE(void, Syslog, log0)(ACR_JNISTDARGS,
+                                            jint level,
+                                            jstring msg)
 {
     UNREFERENCED_O;
     WITH_WSTR(msg) {

Modified: commons/sandbox/runtime/trunk/src/main/native/os/win32/uuid.c
URL: 
http://svn.apache.org/viewvc/commons/sandbox/runtime/trunk/src/main/native/os/win32/uuid.c?rev=771190&r1=771189&r2=771190&view=diff
==============================================================================
--- commons/sandbox/runtime/trunk/src/main/native/os/win32/uuid.c (original)
+++ commons/sandbox/runtime/trunk/src/main/native/os/win32/uuid.c Mon May  4 
06:13:05 2009
@@ -19,7 +19,7 @@
 #include "acr_arch.h"
 #include "acr_error.h"
 
-ACR_JNI_EXPORT_DECLARE(jbyteArray, util_UUID, getb0)(ACR_JNISTDARGS)
+ACR_UTIL_EXPORT_DECLARE(jbyteArray, UUID, getb0)(ACR_JNISTDARGS)
 {
     jbyteArray rv;
     jbyte id[16];
@@ -34,4 +34,3 @@
     }
     return rv;
 }
-

Modified: commons/sandbox/runtime/trunk/src/main/native/shared/error.c
URL: 
http://svn.apache.org/viewvc/commons/sandbox/runtime/trunk/src/main/native/shared/error.c?rev=771190&r1=771189&r2=771190&view=diff
==============================================================================
--- commons/sandbox/runtime/trunk/src/main/native/shared/error.c (original)
+++ commons/sandbox/runtime/trunk/src/main/native/shared/error.c Mon May  4 
06:13:05 2009
@@ -37,10 +37,10 @@
     "java/io/IOException",
     "java/io/SyncFailedException",
     "java/net/SocketException",
-    ACR_CLASS_PATH "exception/ClosedDescriptorException.java",
-    ACR_CLASS_PATH "exception/NoSuchObjectException.java",
-    ACR_CLASS_PATH "exception/OperatingSystemException.java",
-    ACR_CLASS_PATH "exception/UnsupportedOperatingSystemException.java",
+    ACR_EXCEPTION_CLASS_PATH "ClosedDescriptorException.java",
+    ACR_EXCEPTION_CLASS_PATH "NoSuchObjectException.java",
+    ACR_EXCEPTION_CLASS_PATH "OperatingSystemException.java",
+    ACR_EXCEPTION_CLASS_PATH "UnsupportedOperatingSystemException.java",
     NULL
 };
 
@@ -105,7 +105,7 @@
 ACR_DECLARE(void) ACR_ThrowExceptionA(JNIEnv *env,
                                       const char *file, int line,
                                       acr_trowclass_e clazz, const char *msg)
-{    
+{
     ACR_ThrowClassEx(env, file, line, throw_classes[clazz], msg);
 }
 

Modified: commons/sandbox/runtime/trunk/src/main/native/shared/memory.c
URL: 
http://svn.apache.org/viewvc/commons/sandbox/runtime/trunk/src/main/native/shared/memory.c?rev=771190&r1=771189&r2=771190&view=diff
==============================================================================
--- commons/sandbox/runtime/trunk/src/main/native/shared/memory.c (original)
+++ commons/sandbox/runtime/trunk/src/main/native/shared/memory.c Mon May  4 
06:13:05 2009
@@ -236,7 +236,7 @@
 }
 
 
-static int ptr_cleanup(void *mem)
+static int memory_pointer_cleanup(void *mem)
 {
     if (mem) {
         free(mem);
@@ -268,7 +268,9 @@
     if (mem) {
         /* Create the Pointer class with default cleanup.
          */
-        ptr = ACR_PointerCreate(_E, mem, (size_t)siz, ptr_cleanup);
+        ptr = ACR_PointerCreate(_E, mem, (size_t)siz, memory_pointer_cleanup);
+        if (!ptr)
+            free(mem);
     }
     return ptr;
 }
@@ -288,7 +290,9 @@
     if (mem) {
         /* Create the Pointer class with default cleanup.
          */
-        ptr = ACR_PointerCreate(_E, mem, (size_t)siz, ptr_cleanup);
+        ptr = ACR_PointerCreate(_E, mem, (size_t)siz, memory_pointer_cleanup);
+        if (!ptr)
+            free(mem);
     }
     return ptr;
 }
@@ -348,6 +352,7 @@
                                              jobject src, jlong off,
                                              jlong siz)
 {
+    jobject po;
     size_t  so  = (size_t)off;
     size_t  ss  = (size_t)siz;
     size_t  sl;
@@ -376,7 +381,14 @@
     }
     /* Copy the original content */
     memcpy(dp, sp + so, ss);
-    return ACR_PointerCreate(_E, dp, ss, ptr_cleanup);
+    po = ACR_PointerCreate(_E, dp, ss, memory_pointer_cleanup);
+    if (!po) {
+        /* Destroy the the memory we failed to attach
+         * to the new Pointer object.
+         */
+        free(dp);
+    }
+    return po;
 }
 
 ACR_JNI_EXPORT_DECLARE(void, Memory, copy)(ACR_JNISTDARGS, jobject src,
@@ -499,7 +511,7 @@
     size_t  pl;
     size_t  dn = (size_t)poff;
     size_t  cs = (size_t)size;
-    char    *p = (char *)ACR_PointerGet(_E, ptr, &pl);
+    jbyte   *p = (jbyte *)ACR_PointerGet(_E, ptr, &pl);
 
     if (!p) {
         ACR_ThrowException(_E, THROW_NMARK, ACR_EX_ENULL, 0);
@@ -509,9 +521,9 @@
         ACR_ThrowException(_E, THROW_NMARK, ACR_EX_EINDEX, 0);
         return NULL;
     }
-    ba = (*_E)->NewByteArray(_E, (jsize)size);
+    ba = (*_E)->NewByteArray(_E, cs);
     if (ba) {
-        (*_E)->SetByteArrayRegion(_E, ba, 0, (jsize)size, (jbyte *)p + dn);
+        (*_E)->SetByteArrayRegion(_E, ba, 0, cs, p + dn);
     }
     return ba;
 }


Reply via email to