Marvin Fröhlich
|
 |
«
Posted
2007-11-16 13:50:45 » |
|
Hey, guys. What is the current SVN (or CVS?) branch/trunk, that I should use with ODEJava? Is is the trunk or one of the branches? I hope, it is one, that already uses vecmath. I would want to port it to vecmath2.
Marvin
|
|
|
|
|
|
irrisor
|
|
«
Reply #1 - Posted
2007-11-17 16:07:27 » |
|
The "with-odejava-jni" cvs branch uses vecmath and a newer version of xith than the head. I think we should merge it into the head in the near future. The only active developer besides myself seems to be a user named harsens
- so if you are around, harsens - any objections?
|
|
|
|
|
|
Marvin Fröhlich
|
|
«
Reply #2 - Posted
2007-11-18 02:19:15 » |
|
Where do I get the org.odejava.ode.* classes from? There doesn't seem to be a jar, nor a src folder in the branch containing these classes.
Marvin
|
|
|
|
|
Games published by our own members! Check 'em out!
|
|
|
Marvin Fröhlich
|
|
«
Reply #3 - Posted
2007-11-18 02:22:13 » |
|
Oh, and how about Java-5 syntax? You seem to be using Java 1.4 syntax. Is it a problem, if I port the lib to use the nice generics and @Override annotations?
Marvin
|
|
|
|
|
|
Marvin Fröhlich
|
|
«
Reply #4 - Posted
2007-11-18 02:38:07 » |
|
Where do I get the org.odejava.ode.* classes from? There doesn't seem to be a jar, nor a src folder in the branch containing these classes.
OK, found it. |
|
|
|
|
|
Marvin Fröhlich
|
|
«
Reply #5 - Posted
2007-11-18 03:05:04 » |
|
The methods Ode.setWorldID() and Ode.setContactGroupID() don't seem to be applicable anymore. What should I use instead?
Marvin
|
|
|
|
|
|
irrisor
|
|
«
Reply #6 - Posted
2007-11-18 11:30:11 » |
|
The methods Ode.setWorldID() and Ode.setContactGroupID() don't seem to be applicable anymore. What should I use instead?
they're parameters to the collision function now (setting them globally caused errors with multiple worlds) |
|
|
|
|
|
irrisor
|
|
«
Reply #7 - Posted
2007-11-18 11:33:33 » |
|
Oh, and how about Java-5 syntax?
I think there might be people who would like to use it with 1.4 - but not myself. They could use retroweaver if they really need 1.4, though... so... why not? |
|
|
|
|
|
Marvin Fröhlich
|
|
«
Reply #8 - Posted
2007-11-18 15:17:39 » |
|
they're parameters to the collision function now (setting them globally caused errors with multiple worlds)
Could you maybe fix that in the branch and commit? I don't know, how to do that. Marvin |
|
|
|
|
|
irrisor
|
|
«
Reply #9 - Posted
2007-11-18 16:31:33 » |
|
ok, fixed in branch
|
|
|
|
|
Games published by our own members! Check 'em out!
|
|
|
Marvin Fröhlich
|
|
«
Reply #10 - Posted
2007-11-18 16:42:49 » |
|
ok, fixed in branch
Cool. Thanks. I have finished the port to the latest Xith3D and vecmath2. Please tell me, if and when I can commit it. Marvin |
|
|
|
|
|
Marvin Fröhlich
|
|
«
Reply #11 - Posted
2007-11-18 16:52:47 » |
|
There's one thing left, that I don't know, how to fix. In org.odejava.GeomTriMesh.init() the method getVoidPointer() is undefined for the SWIG types. What should I do here? Well I could simply commit all the stuff and let you fix it. Is this ok?
Marvin
|
|
|
|
|
|
Marvin Fröhlich
|
|
«
Reply #12 - Posted
2007-11-18 16:58:30 » |
|
What's this? 1
2
3
4
5
6
7
8
9
10
11
| 18.11.2007 16:53:45 org.odejava.Odejava <clinit>
SCHWERWIEGEND: Wrong native version! Expected 0.3.1, but found 0.3
Exception in thread "main" java.lang.UnsupportedOperationException: Failed to load natives
at org.odejava.Odejava.init(Odejava.java:125)
at org.odejava.test.car.CarTerrain.initWorld(CarTerrain.java:142)
at org.odejava.test.car.CarTerrain.<init>(CarTerrain.java:131)
at org.odejava.xith3d.test.CarTerrainExample.<init>(CarTerrainExample.java:124)
at org.odejava.xith3d.test.CarTerrainExample.main(CarTerrainExample.java:105)
Caused by: java.lang.UnsatisfiedLinkError: Wrong native version! Expected 0.3.1, but found 0.3
at org.odejava.Odejava.<clinit>(Odejava.java:105)
... 4 more |
Marvin |
|
|
|
|
|
irrisor
|
|
«
Reply #13 - Posted
2007-11-18 17:07:25 » |
|
There's one thing left, that I don't know, how to fix. In org.odejava.GeomTriMesh.init() the method getVoidPointer() is undefined for the SWIG types. What should I do here? Well I could simply commit all the stuff and let you fix it. Is this ok?
you should use the head in odejava-jni module, sorry, forgot to point that out - the branch was joined there already What's this?
...
As printed: you use an old native version of odejava-jni  (same problem as above probably) OS? (osx might be out of date, win/lin should be fine) |
|
|
|
|
|
Marvin Fröhlich
|
|
«
Reply #14 - Posted
2007-11-18 17:20:23 » |
|
you should use the head in odejava-jni module, sorry, forgot to point that out - the branch was joined there already
 . Well, if it's ok, I will commit the changes to the with-odejava-jni branch now. Could you than please merge it to where it should be? And then please tell me, where I can checkout the latest. I am a bit confused now. As printed: you use an old native version of odejava-jni (same problem as above probably) OS? (osx might be out of date, win/lin should be fine) I am using Linux. Marvin |
|
|
|
|
|
irrisor
|
|
«
Reply #15 - Posted
2007-11-18 17:30:43 » |
|
Due to the weired cvs setup on java.net I was not able to delete the branch for the odejava-jni folder, that's why you need to check out the "HEAD" for the folder odejava-jni, but "with-odejava-jni" branch for all the other dirs.
For me it's ok, if you commit to the "with-odejava-jni" branch (am I the one to decide that?). But do you have commit rights - what's your login?
|
|
|
|
|
|
Marvin Fröhlich
|
|
«
Reply #16 - Posted
2007-11-18 17:46:15 » |
|
I have commit rights. My Login is "qudus". I have committed the changes. Please tell me, if everything is ok.
After you've merged it to where ever it should be I can start the Java-5-Syntax port.
Marvin
|
|
|
|
|
|
Marvin Fröhlich
|
|
«
Reply #17 - Posted
2007-11-18 17:50:51 » |
|
Oh, just to be mentioned: I don't know, if there are any ant-scripts to be updated. I am working with Eclipse and don't have too much knowledge about ant. So please have a look at the .classpath file to know, which jars etc. have to be imported.
Marvin
|
|
|
|
|
|
irrisor
|
|
«
Reply #18 - Posted
2007-11-18 19:32:40 » |
|
After you've merged it to where ever it should be I can start the Java-5-Syntax port.
I won't merge into the head in the next couple of days (no time for that). So just go ahead in the branch. And: I don't use Ant either. |
|
|
|
|
|
Marvin Fröhlich
|
|
«
Reply #19 - Posted
2007-11-18 19:45:07 » |
|
I won't merge into the head in the next couple of days (no time for that). So just go ahead in the branch.
ok And: I don't use Ant either.
So, what are you using? The eclipse project wasn't completely setup in the branch. Marvin |
|
|
|
|
|
irrisor
|
|
«
Reply #20 - Posted
2007-11-18 19:50:55 » |
|
IntelliJ IDEA
|
|
|
|
|
|
Marvin Fröhlich
|
|
«
Reply #21 - Posted
2007-11-19 15:45:49 » |
|
I managed to switch the odejava-jni folder to the HEAD. So there's no need to merge, I guess.
Marvin
|
|
|
|
|
|
Marvin Fröhlich
|
|
«
Reply #22 - Posted
2007-11-19 18:19:33 » |
|
I have finished the Java-5 Syntax port. If you have time, please check my work.
Marvin
|
|
|
|
|
|
irrisor
|
|
«
Reply #23 - Posted
2007-11-19 19:41:05 » |
|
Scanning over the diff, it looks ok. Just one thing I stumbled upon: 1
2
3
4
5
6
7
8
| 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. |
|
|
|
|
|
Marvin Fröhlich
|
|
«
Reply #24 - Posted
2007-11-19 19:55:57 » |
|
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
|
|
|
|
|
|
