Current ODEJava branch

IntelliJ IDEA

I managed to switch the odejava-jni folder to the HEAD. So there’s no need to merge, I guess.

Marvin

I have finished the Java-5 Syntax port. If you have time, please check my work.

Marvin

Scanning over the diff, it looks ok. Just one thing I stumbled upon:

public class GeomBox extends PlaceableGeom {
 
+    @SuppressWarnings("unused")
     private float sizeX;
+    @SuppressWarnings("unused")
     private float sizeY;
+    @SuppressWarnings("unused")
     private float sizeZ;

Does that make sense? Are they really used - what for? If you are unsure you should not suppress the warnings. If you are sure they are not use, they should rather be removed.

You’re right. I shouldn’t have suppressed these warnings. If they are unused (up to you, but I think, they are), just remove them.

Marvin