Misc Patch (Was:Re: [Bf-python] Meeting minutes 24-05 + invitation 31-05)

Jordi Rovira i Bonet jordi.rovira at ima.udg.es
Tue Jun 3 11:57:57 CEST 2003


Michel Selten wrote:

>On Fri, 2003-05-30 at 11:50, Jordi Rovira i Bonet wrote:
>  
>
>>There's the partial Scene wrapper, and the modifications around in 
>>Object to make my script work again. I don't intend that this patch is 
>>applied to CVS, it's only to show the changes i've made.
>>    
>>
>
>I've looked through the patch. Some of the updates I have now
>implemented. Most of these were typos. Others require some updates in
>the NMesh module. That module I rather don't touch without Willian's
>approval ;) Hehe, I know I can probably just update the file, but maybe
>he's already done that.
>
>About the Scene module: It looks like a good start. Do you have time to
>finish it this week?
>
So... shall i assume this work (it wasn't assigned to me initially....)

Notes about the latest commits in the CVS.

BGL.h line 1005, there's a varaible declaration after a function 
instruction which doesn't compile in old C standards (the added line 
with assignation should be moved below the var declaration)

Camera.h it doesn't compile with the declaration of the PyTypeObject 
Camera_Type =... because the callback functions are not declared yet. 
However, moving this declaration to the C file works for me. Is this 
Python object really needed in the header?

There's no way to access object's name (only object's data name) which i 
require to export the scene graph without duplicationg the meshes that 
are referenced twice. I'll add itin Object.c.

The Doxygen documentation is fine, i'll try to use this documentation 
conventions from now on.

Regards,

(bandoler)




More information about the Bf-python mailing list