Little fix

archive development-discussion-jme2
1

I'm reading and analyzing part of the source-code when I have free time. And maybe I can suggest or do some improvement.

I've found little things like:

  • com.jme.math.Matrix4f.multAcross (line 1029) - is using System.err.println instead of the LoggingSystem
  • com.jme.math.Matrix4f.mult (line 1005) - is using System.err.println instead of the LoggingSystem



    Ok, started to post here. Or should I post my reviews/comments/whatever, in another place? Thanks.
2

Great initiative :slight_smile: I'll be happy no matter where you post it

3

com.jme.renderer.lwjgl.LWJGLRenderer.draw(Curve c) is calling glTranslatef, glRotatef and glScalef.



Some special reason for doing that instead of calling doTransforms() ?

If yes, please, explain it.

4

Curve is a hangover from old code.  Probably could be moved out or at least cleaned up.

5

Oh… danke shon Renanse.