[Bf-taskforce25] DerivedMesh stuff

Ton Roosendaal ton at blender.org
Tue Jun 2 18:34:54 CEST 2009


Hi Joe,

I noticed you added code in the poll() function for:

screen_ops.c, ED_operator_uvmap(bContext *C)

That stuff really doesn't belong there. The poll() is meant only for 
checking valid context, not for changing it or editing data...

It's in the docs for operators even! I won't have time to check all 
your commits, please carefully at least read what I've written about 
this? And ask me or brecht before committing in this code...

-Ton-

------------------------------------------------------------------------
Ton Roosendaal  Blender Foundation   ton at blender.org    www.blender.org
Blender Institute   Entrepotdok 57A  1018AD Amsterdam   The Netherlands

On 2 Jun, 2009, at 4:19, joe wrote:

> I've decided to make some modifications to DerivedMesh, to handle
> ngons in the bmesh branch.  Basically, I'm going to rename the
> existing face accessors (to make it clear they access tesselations
> now, not real faces) and add in some form of face iterators.
>
> Brecht: the code you did on this a while ago would be nice to have, as
> would any comments you have.
>
> Joe
> _______________________________________________
> Bf-taskforce25 mailing list
> Bf-taskforce25 at blender.org
> http://lists.blender.org/mailman/listinfo/bf-taskforce25
>



More information about the Bf-taskforce25 mailing list