Fixed bug abies pointed out in ARBVBOKey where it was necessary to

override hashCode() and therefore equals(). Added caching of
BufferUtils.bufferOffset() buffers.


git-svn-id: file:///usr/local/projects/SUN/JOGL/git-svn/svn-server-sync/jogl/trunk@48 232f8b59-042b-4e1e-8c03-345bb8c30851

2 files changed, 15 insertions, 2 deletions

diff --git a/src/native/jogl/BufferUtils.c b/src/native/jogl/BufferUtils.c
index f10a8ce6d..d5ea5b36e 100644
--- a/src/native/jogl/BufferUtils.c
+++ b/src/native/jogl/BufferUtils.c
@@ -49,6 +49,6 @@
 #endif
 
 JNIEXPORT jobject JNICALL
-Java_net_java_games_jogl_util_BufferUtils_bufferOffset(JNIEnv* env, jclass unused, jint offset) {
+Java_net_java_games_jogl_util_BufferUtils_bufferOffset0(JNIEnv* env, jclass unused, jint offset) {
   return (*env)->NewDirectByteBuffer(env, (void*) (intptr_t) offset, 0);
 }
diff --git a/src/net/java/games/jogl/util/BufferUtils.java b/src/net/java/games/jogl/util/BufferUtils.java
index 05125b75e..88afa1c8f 100644
--- a/src/net/java/games/jogl/util/BufferUtils.java
+++ b/src/net/java/games/jogl/util/BufferUtils.java
@@ -40,6 +40,7 @@
 package net.java.games.jogl.util;
 
 import java.nio.*;
+import java.util.*;
 
 /** Utility routines for dealing with direct buffers. */
 
@@ -70,6 +71,8 @@ public class BufferUtils {
     return dest;
   }
 
+  private static Map bufferOffsetCache = Collections.synchronizedMap(new HashMap());
+
   /** Creates an "offset buffer" for use with the
       ARB_vertex_buffer_object extension. The resulting Buffers are
       suitable for use with routines such as glVertexPointer <em>when
@@ -77,5 +80,15 @@ public class BufferUtils {
       capacity and are not suitable for passing to OpenGL routines
       that do not support buffer offsets, or to non-OpenGL
       routines. */
-  public static native ByteBuffer bufferOffset(int offset);
+  public static ByteBuffer bufferOffset(int offset) {
+    Integer key = new Integer(offset);
+    ByteBuffer buf = (ByteBuffer) bufferOffsetCache.get(key);
+    if (buf == null) {
+      buf = bufferOffset0(offset);
+      bufferOffsetCache.put(key, buf);
+    }
+    return buf;
+  }
+
+  private static native ByteBuffer bufferOffset0(int offset);
 }