
<html><head></head><body><div>Hi All,</div><div><br></div><div>On Wed, 2022-10-26 at 02:52 +0200, Michalis Kamburelis wrote:</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div>Forgot to attach: error log from FreeWRL when reading<br></div><div>CADGeometryPROTO.wrl.gz (before fixing with backslashes).<br></div><div><br></div><div>śr., 26 paź 2022 o 01:50 Michalis Kamburelis<br></div><div><<a href="mailto:michalis.kambi@gmail.com">michalis.kambi@gmail.com</a>> napisał(a):<br></div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div><br></div><div>In this case, the VRML file<br></div><div>/usr/share/doc/whitedune-docs/docs/x3d/CADGeometryPROTO.wrl is really<br></div><div>invalid.</div></blockquote></blockquote><div>OK, makes sens to not support buggy files as Paul said also.</div><div>I'll drop this file from tests.</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div>While I can imagine how to "tolerate" such files, but it would be<br></div><div>quite a hack in CGE (tovrmlx3d just uses Castle Game Engine to read<br></div><div>X3D). We would effectively guess too much "what was the intention of<br></div><div>the author of this VRML file". I would need to encode that, where a<br></div><div>single string is expected in VRML, we allow a sequence of "string +<br></div><div>(identifier + string)*" and we just glue them all together, assuming<br></div><div>that it's actually one long string but someone forgot backslashes.<br></div><div>That's too much guessing, and too big hack, just to support a single<br></div><div>invalid file :)<br></div><div><br></div><div>I wanted to look at what others do:<br></div><div><br></div><div>- FreeWRL: <a href="https://freewrl.sourceforge.io/">https://freewrl.sourceforge.io/</a> (4.0) also fails to read<br></div><div>it. Adding backslashes, as I suggested above, makes FreeWRL also read<br></div><div>it OK. So basically FreeWRL behaves the same as<br></div><div>view3dscene/tovrmlx3d/CGE in this case. And it's a completely<br></div><div>independent codebase to read VRML/X3D files.<br></div><div><br></div><div>- I wanted to test whitedune itself, alas it fails for me with a<br></div><div>SEGFAULT now (just running it without any parameters makes a<br></div><div>SEGFAULT). Tested on Debian (whitedune 0.30.10-2.2) and another<br></div><div>machine with Ubuntu (whitedune 0.30.10-2.2build1).<br></div><div><br></div><div>The part of the CADGeometryPROTO.wrl file that misses backslashes<br></div><div>contains a shader code inside a VRML string. While this is a comment<br></div><div>for shader (GLSL), but first the VRML browser must read the string<br></div><div>with shader code, and the "Script.url" must contain something that can<br></div><div>be parsed.<br></div><div><br></div><div>All in all, we cannot easily ignore this problem on tovrmlx3d side --<br></div><div>it would be a too big hack, just for this file. And I have yet to see<br></div><div>any VRML browser that actually opens this file :)<br></div><div><br></div><div>My suggestion would be to:<br></div><div><br></div><div>- Report this to whitedune-docs, as error in<br></div><div>/usr/share/doc/whitedune-docs/docs/x3d/CADGeometryPROTO.wrl.gz<br></div><div><br></div><div>- While you're at this, check can you run whitedune at all. If it<br></div><div>SEGFAULTs for others, then this is likely even more critical bug we<br></div><div>should submit to whitedune package.<br></div><div><br></div><div>- In view3dscene: either stop testing on whitedune-docs, or at least<br></div><div>omit this particular faulty file CADGeometryPROTO.wrl.gz .<br></div><div><br></div><div>- If you're looking for files to test, we maintain a big repository of<br></div><div>test models on <a href="https://github.com/castle-engine/demo-models">https://github.com/castle-engine/demo-models</a> . All on<br></div><div>open-source licenses. Though I understand you would want something<br></div><div>that is already part of Debian.</div></blockquote></blockquote><div>I can't download files from the internet, so I should package these and I don't think it make sense.</div><div>So will try to filter out buggy files from whitedune.</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"></blockquote><div style="font-size: 14.666667px;"><span><pre>-- <br></pre><pre>Cheers,

Abou Al Montacir</pre></span></div></body></html>