[news.eclipse.gef3d] Re: A couple of errors in ..draw3d

[Miles Parker <milesparker@xxxxxxxxx>]

On 2009-05-29 14:57:08 -0700, Kristian Duske 
<kristian.duske@xxxxxxxxxxxxxxxx> said:

> Hello Miles,
>
> we are not using Galileo yet, so the six methods you mentioned in your 
> original post are not implemented in GEF3D. I suggest you either 
> implement them yourself or wait until we do it. Either way, there are 
> some methods in the Graphics class that we do not support properly yet. 
> I have just committed a new version of LWJGLGraphics that has support 
> for some of those methods (mostly setters that will be ignored now). 
> Those methods that still throw an exception are now documented in the 
> class javadoc.
>
> I suggest you update to the latest revision and have a look. I hope 
> this helps get GEF3D to run for you. If you have any further problems, 
> let us know.
>
> Best regards
> Kristian

Thanks Kristian, I just updated. FWIW, I had already put these quick fixes in:

LwjglGraphics.

   public float getLineWidthFloat() {
       return getLineWidth();
   }

   public void setLineWidthFloat(float width) {
       setLineWidth((int) width);
   }

   public void setLineMiterLimit(float miterLimit) {
   }

   public void setLineAttributes(LineAttributes attributes) {
       // avoid not implemented exception
   }

   public void setForegroundPattern(Pattern pattern) {
       // avoid not implemented exception
   }

   public void setLineCap(int cap) {
       // avoid not implemented exception
   }

   public void setLineDash(float[] value) {
       // avoid not implemented exception
   }

DummyGraphics and LogGraphics


   public float getLineWidthFloat() {
       return getLineWidth();
   }

   public void setLineWidthFloat(float width) {
       setLineWidth((int) width);
   }

   public void setLineMiterLimit(float miterLimit) {
   }

   public void setLineAttributes(LineAttributes attributes) {
       // avoid not implemented exception
   }

   public void setLineDash(float[] value) {
       // avoid not implemented exception
   }


They should be backward compatible I'd think.. :)