is there a scenenode->getmesh?

[Phunk]

As I am using the new .irr scene file loader, I would like to gain control over the mesh that a loaded mesh-scenenode posesses.

But there is only a setMesh, and no getMesh(or something like that). Would it be a problem adding this getter to the IMeshSceneNode (and CMeshSceneNode) interface? It seems to me it was just not useful before the .irr loader, but I think it is now.

Or is there some sort of other way of getting the mesh from a scenenode?

[vitek]

No, there is not. There probably shouldn't be a getMesh() in ISceneNode because all scene nodes do not have meshes.

Maybe the IMeshSceneNode should expose an abstract getMesh() method and the IAnimatedMeshSceneNode should expose a getAnimatedMesh() also.

Travis

[Phunk]

yeah, I know, thats why I said IMeshSceneNode, I guess you could use getType() to see what kind of scenenode you should use, and then create a pointer of that kind to reference to it. But as most of the scene nodes have IMeshSceneNode as their parent, I feel that the getMesh() should be in there.

should I make a bug report of this?

[vitek]

But as most of the scene nodes have IMeshSceneNode as their parent, I feel that the getMesh() should be in there

This is simply not true. The only class that has IMeshSceneNode as a base is CMeshSceneNode. As I said, it would not right for the ISceneNode interface to expose a getMesh() method. Very few scene node types actually have a mesh, and of those that do, some of those meshes are animated meshes.

[Phunk]

yeah, youre right. I thought that every scene node that contains a mesh (so also animatedmesh, terrainnode(allready has a getmesh) and watersurfacenode) would derive from meshscenenode, but I was wrong in that assumption.

I totally agree with you on the matter that getmesh should not be in ISceneNode, I meant with 'should be in there' that there should be a getmesh in IMeshSceneNode and a getanimatedmesh in IAnimatedMeshscenenode, like you said in your first post.

but to get these methods in, should it be made a bug report, or is there an other way?

[vitek]

Yeah, a post in the bug report forum will probably do. Hybrid reads almost every post that comes through, so he's probably already seen this and started to make the appropriate changes.

[hybrid]

I've read all this, yes, but I'm currently doing other things, so I won't do something for this in the near future.

[Phunk]

ill fix it, and post a patch in the bug forum. ok?

Gr. Robin

[TheC]

I added my own getLocalMesh() method. I did think it was odd that there wasn't one in from the start.

it was as simple as virtual IAnimatedMesh* getLocalMesh() { return IAnimatedMesh; }

[Strong99]

where did you add it? in a IScenenode?