ViewHints I/O (was Lights I/O)

[Stefan H. <st...@to...>]

I did some testing tonight. Below I'm listing the changes that I've 
made in order to write QD3D compatible viewhints. Now camera, lights, 
attributes, color, and dimensions will be written.

>>QD3D always writes the whole ViewHints data. In
>>   e3viewhints_new (file E3IOData.c)
>>this behaviour is partly implemented (if paramData...): Camera and 
>>lights are ok, renderer is basically ok (as you mentioned above: 
>>depends on the renderer type), default attributes should use 
>>Q3View_GetDefaultAttributeSet (not Q3View_GetAttributeSetState, I 
>>suppose), and clear color/dimension should be initialized from the 
>>draw context data.
>
>Why should we use Q3View_GetDefaultAttributeSet instead of 
>Q3View_GetAttributeSetState ?

Q3View_GetAttributeSetState does not work. There are two functions to 
set/get the attribute to/from a view called 
Q3View_Set/GetDefaultAttributeSet. That' why I tried to use 
Q3View_GetDefaultAttributeSet. I wrote two files under QD3D with 
different attribute values, read and rewrote them under Quesa. Well, 
it works!

>>>I do not need backward compatibility but I would like to sort this out
>>>for you if you do need it. Any chance of patching the file to find
>>>out which particular aspect is tripping up QD3D?
>>
>>Currently I have not found a particular aspect; ignoring the 
>>viewhints as well as reading the complete viewhints seems to work. 
>>Certainly, I'm missing something, but what?
>>
>>My suggestions are:
>>- upload the "kQ3LightData as 'lhgt'" hack

Could you upload the "kQ3LightData as 'lhgt'" in E3FFW_3DMFBin_Writer.c?

>>- change e3viewhints_new (the "if paramData" section)

The clear image color works well.
The dimension should be max - min as you mentioned in the code 
(Q3DrawContext_GetPane/Q3DrawContext_GetPaneState).

>
>
>>- Quesa renderers: write the renderer settings out
>
>This is a job for the someone familiar with the interactive renderer 
>code. Its not part of View Hints, I am asking the Renderer to write 
>itself out.

It would be nice to do this for interactive and wireframe renderer... 
But it's not that important for me.

Stefan