Thank you Bo.  Good catch.  The current version of Xj3D has fixed this
bug.  You can either jump into the current version of CVS or wait a few
days(likely April 20th) and we will be tagging a new release of the
project as stable.  I spent a fair bit of time pouring over the
IndexedFaceSet code so I'm hoping I've cleaned up any remaining bugs.

Other bugs of note that have been fixed as well include:

Problems with handling ccw flag - resulted in weird texturing
Incorrect handling of non-uniform scales - caused incorrect translations

I'd also encourage future bug reports to be placed on the source list.
To subscribe send an email to [EMAIL PROTECTED], in the body put:
subscribe source

"Bo, Jianquan, myview" wrote:
>
> To Alan Hudson, Justin Couch,
>
> I have found an error in Xj3d Vrml-Loader. Please check it again and if it
> is true, please update it in the coming release of the Vrml-loader. I have
> modified the source code in class IndexedFaceSet as follows and it works ok:
>
>     void buildImpl()
>     {
>         gi = null;
>         initSetup();
>         if(implType == 101)
>             gi = new GeometryInfo(1);
>         else
>         if(implType == 102)
>             gi = new GeometryInfo(2);
>         else
> //            gi = new GeometryInfo(3);
>
> // modified by Jianquan Bo, 18.04.2001
> // if it is not triangle or quad array (1 or 2), it should be treated
> // as polygon array (5), but not triangle fan array (3) or triangle
> // strip array (4)
>             gi = new GeometryInfo(5);
>
>         ...
>
>     }
>
> Thanks,
>
> Bo

--
Alan Hudson
President: Yumetech, Inc.                      http://www.yumetech.com/
Web3D Open Source Chair        http://www.web3d.org/TaskGroups/source/

===========================================================================
To unsubscribe, send email to [EMAIL PROTECTED] and include in the body
of the message "signoff JAVA3D-INTEREST".  For general help, send email to
[EMAIL PROTECTED] and include in the body of the message "help".

Reply via email to