Fix buffer usage bug.

android_15/src/org/ros/android/view/DistancePoints.java

@@ -27,7 +27,7 @@ import javax.microedition.khronos.opengles.GL10;
 /**
  * Helper function for the DistanceRenderer that creates the polygons, lines,
  * points, etc based on the received data.
- * 
+ *
  * @author munjaldesai@google.com (Munjal Desai)
  */
 class DistancePoints {
@@ -91,6 +91,7 @@ class DistancePoints {
     // Update the buffers with the latest coordinates.
     initRangeVertexBuffer();
     rangeVertexBuffer.put(rangeVertexArray);
+    // TODO(hrapp): This should probably be flip().
     rangeVertexBuffer.position(0);
   }
 
@@ -107,7 +108,7 @@ class DistancePoints {
   /**
    * Draws the open region in light gray and the objects seen by the laser in
    * red.
-   * 
+   *
    * @param gl
    *          The GL interface.
    */
@@ -132,7 +133,7 @@ class DistancePoints {
 
   /**
    * Draws the reference markers that show the current scale or zoom.
-   * 
+   *
    * @param gl
    *          The GL interface.
    */
@@ -150,7 +151,7 @@ class DistancePoints {
 
   /**
    * Draws the robot.
-   * 
+   *
    * @param gl
    *          The GL interface.
    */
@@ -193,6 +194,7 @@ class DistancePoints {
     vertexByteBuffer.order(ByteOrder.nativeOrder());
     robotVertexBuffer = vertexByteBuffer.asFloatBuffer();
     robotVertexBuffer.put(robotVertices);
+    // TODO(hrapp): This should probably be flip().
     robotVertexBuffer.position(0);
   }
 
@@ -245,6 +247,7 @@ class DistancePoints {
     referenceVertexByteBuffer.order(ByteOrder.nativeOrder());
     referenceVertexBuffer = referenceVertexByteBuffer.asFloatBuffer();
     referenceVertexBuffer.put(referenceVertices);
+    // TODO(hrapp): This should probably be flip().
     referenceVertexBuffer.position(0);
   }
 }

android_15/src/org/ros/android/view/visualization/OpenGlTransform.java

@@ -24,7 +24,7 @@ import javax.microedition.khronos.opengles.GL10;
 
 /**
  * An adapter for applying {@link Transform}s in an OpenGL context.
- * 
+ *
  * @author damonkohler@google.com (Damon Kohler)
  * @author moesenle@google.com (Lorenz Moesenlechner)
  */
@@ -50,7 +50,7 @@ public class OpenGlTransform {
 
   /**
    * Applies a {@link Transform} to an OpenGL context.
-   * 
+   *
    * @param gl
    *          the context
    * @param transform
@@ -61,6 +61,7 @@ public class OpenGlTransform {
     for (double value : transform.toMatrix()) {
       matrix.put((float) value);
     }
+    // TODO(hrapp): This should probably be flip().
     matrix.position(0);
     gl.glMultMatrixf(matrix);
   }

android_15/src/org/ros/android/view/visualization/Vertices.java

@@ -44,6 +44,7 @@ public class Vertices {
   public static FloatBuffer toFloatBuffer(float[] floats) {
     FloatBuffer floatBuffer = allocateBuffer(floats.length);
     floatBuffer.put(floats);
+    // TODO(hrapp): This should probably be flip().
     floatBuffer.position(0);
     return floatBuffer;
   }

android_15/src/org/ros/android/view/visualization/layer/LaserScanLayer.java

@@ -32,7 +32,7 @@ import sensor_msgs.LaserScan;
 
 /**
  * A {@link SubscriberLayer} that visualizes sensor_msgs/LaserScan messages.
- * 
+ *
  * @author munjaldesai@google.com (Munjal Desai)
  * @author damonkohler@google.com (Damon Kohler)
  */
@@ -114,7 +114,7 @@ public class LaserScanLayer extends SubscriberLayer<sensor_msgs.LaserScan> imple
       }
       angle += angleIncrement * stride;
     }
-    vertexBackBuffer.position(0);
+    vertexBackBuffer.flip();
     synchronized (mutex) {
       FloatBuffer tmp = vertexFrontBuffer;
       vertexFrontBuffer = vertexBackBuffer;

android_15/src/org/ros/android/view/visualization/layer/PathLayer.java

@@ -31,7 +31,7 @@ import javax.microedition.khronos.opengles.GL10;
 
 /**
  * Renders a nav_msgs/Path as a dotted line.
- * 
+ *
  * @author moesenle@google.com (Lorenz Moesenlechner)
  * @author damonkohler@google.com (Damon Kohler)
  */
@@ -99,6 +99,7 @@ public class PathLayer extends SubscriberLayer<nav_msgs.Path> implements TfLayer
         i++;
       }
     }
+    // TODO(hrapp): This should probably be flip().
     vertexBuffer.position(0);
   }
 

android_15/src/org/ros/android/view/visualization/layer/PointCloud2DLayer.java

@@ -122,7 +122,7 @@ public class PointCloud2DLayer extends SubscriberLayer<PointCloud2> implements T
       // Discard intensity.
       buffer.readFloat();
     }
-    vertexBackBuffer.position(0);
+    vertexBackBuffer.flip();
     synchronized (mutex) {
       FloatBuffer tmp = vertexFrontBuffer;
       vertexFrontBuffer = vertexBackBuffer;