loader package hierarchy

[croft]

@croft

I will probably put
the Whoola, Java 5, JAXB 1, COLLADA 1.4 implementation in package org.xith3d.loaders.collada.whoola1.*.

The Whoola, Java 5, JAXB 2, COLLADA 1.4 implementation will go in org.xith3d.loaders.collada.whoola2.*.

The Whoola, Java 5, JAXB 2, COLLADA 1.5 implementation will go in org.xith3d.loaders.collada.whoola3.*, etc.

OK, I changed my mind on this. I notice the current structure is like this:

org.xith3d.loaders.ase
org.xith3d.loaders.ext
org.xith3d.loaders.loader3ds
org.xith3d.loaders.md2
org.xith3d.loaders.obj
org.xith3d.loaders.tds

There is almost a pattern of 3 letters there based on filename extension. The filename extension for COLLADA is .dae (Digital Asset Exchange).

There are two 3DS loaders. If we move the two 3DS loaders under one package and then distinguish them by a subpackage name based on the contributors, we could have something like this:

org.xith3d.loaders.ase
org.xith3d.loaders.dae.whoola
org.xith3d.loaders.ext
org.xith3d.loaders.md2
org.xith3d.loaders.obj
org.xith3d.loaders.tds.kinostudios
org.xith3d.loaders.tds.newdawn

I’ve distinguished the Whoola loader in a subpackage as I am fairly certain a new COLLADA loader for Xith from a different source will emerge within the next couple of years.

Whether we move the 3DS packages or not, are we OK with org.xith3d.loaders.dae.whoola for COLLADA?

[eMKa]

@croft

I’ve distinguished the Whoola loader in a subpackage as I am fairly certain a new COLLADA loader for Xith from a different source will emerge within the next couple of years.

Whether we move the 3DS packages or not, are we OK with org.xith3d.loaders.dae.whoola for COLLADA?

+1.

And I’ll move the 3DS packages myself.

[eMKa]

Committed.

And also, croft, I saw you made alternative OBJ loader.
Then we should split again the OBJ package to :

org.xith3d.loaders.obj.newdawn
org.xith3d.loaders.obj.whoola

And same thing with the md2 loaders.

[croft]

[quote="<MagicSpark.org [ BlueSky ]>,post:3,topic:27199"]
And also, croft, I saw you made alternative OBJ loader.
Then we should split again the OBJ package to :

org.xith3d.loaders.obj.newdawn
org.xith3d.loaders.obj.whoola

And same thing with the md2 loaders.
[/quote]
No, Whoola did not make those. I just modified the New Dawn OBJ and MD2 loaders to implement the Loader interface. I gave them names like OBJLoader2 where the “2” was supposed to be temporary until the changes could be reviewed by William Denniss. The intent was that once reviewed they would replace the original classes.
http://www.java-gaming.org/forums/index.php?topic=9911.msg84348#msg84348

Let’s merge the files in org.xith3d.loaders.obj.whoola into org.xith3d.loaders.obj.newdawn, possibly overwriting the old classes by the same name. I think this is reasonable since New Dawn provided the bulk of the work and my OBJ and MD2 classes are just minor derivatives.

I am going to start migrating classes gradually to package org.xith3d.loaders.dae.whoola.

I see you already updated DefaultLoader to point to the new package names. If you modify my code, please add your @author tag so that people know I am not solely responsible for the work.

[eMKa]

@croft

[quote="<MagicSpark.org [ BlueSky ]>,post:3,topic:27199"]
And also, croft, I saw you made alternative OBJ loader.
Then we should split again the OBJ package to :

org.xith3d.loaders.obj.newdawn
org.xith3d.loaders.obj.whoola

And same thing with the md2 loaders.

No, Whoola did not make those. I just modified the New Dawn OBJ and MD2 loaders to implement the Loader interface. I gave them names like OBJLoader2 where the “2” was supposed to be temporary until the changes could be reviewed by William Denniss. The intent was that once reviewed they would replace the original classes.
http://www.java-gaming.org/forums/index.php?topic=9911.msg84348#msg84348

Let’s merge the files in org.xith3d.loaders.obj.whoola into org.xith3d.loaders.obj.newdawn, possibly overwriting the old classes by the same name. I think this is reasonable since New Dawn provided the bulk of the work and my OBJ and MD2 classes are just minor derivatives.

I am going to start migrating classes gradually to package org.xith3d.loaders.dae.whoola.

I see you already updated DefaultLoader to point to the new package names. If you modify my code, please add your @author tag so that people know I am not solely responsible for the work.
[/quote]
All Done.