[Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [18086] branches/blender2.5/blender/source /blender: RNA:

[Brecht Van Lommel]

Hi,

joe wrote:
> Could you explain what you need it for?  So I can see where it fits into 
> the big picture :)  It just seems a little klunky to me, having a 
> collection of id properties collections.

I need it to implement an RNA collection of structs. This how I see 
the correspondences:

ID property group ~ C struct        ~ RNA struct
ID property       ~ C struct member ~ RNA property
ID array property ~ C array or list ~ RNA collection property

Now, I guess your interpretation of an ID property group is that it is 
both a struct and a collection. I could try to implement RNA 
collections as ID property groups, but it's a bit messy given that I 
would have to work with string keys which I do not have and would need 
to create with sprintf("%d", index) or so.

Brecht.

> On Sat, Dec 27, 2008 at 12:49 AM, Brecht Van Lommel <brecht at blender.org 
> <mailto:brecht at blender.org>> wrote:
> 
>     Hi,
> 
>     joe wrote:
>      > What's the design rationale of supporting arrays of group properties?
>      >  Seems supporting arrays of id properties in general would be a
>     cleaner
>      > solution, if it's needed. . .
> 
>     Can you explain the difference to me? The array before supported
>     subtypes IDP_INT, ID_FLOAT and IDP_DOUBLE. The logical extension
>     seemed to me to just add IDP_GROUP. Isn't the end result basically the
>     same?
> 
>     Well, it could be interesting to support mixed id property types in a
>     single array, but that is not how the array was implemented here.
>     Doing it that way would have been inconsistent with the way IDP_INT,
>     IDP_FLOAT and IDP_DOUBLE already work.
> 
>     Brecht.
>     _______________________________________________
>     Bf-committers mailing list
>     Bf-committers at blender.org <mailto:Bf-committers at blender.org>
>     http://lists.blender.org/mailman/listinfo/bf-committers
> 
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> Bf-committers mailing list
> Bf-committers at blender.org
> http://lists.blender.org/mailman/listinfo/bf-committers