[Bf-committers] Re: [Bf-blender-cvs] CVS commit: blender/source/blender/src header_info.c usiblender.c

[Joe Eagar]

Ton Roosendaal wrote:
> Hi,
>
>> Ton Roosendaal wrote:
>>> Hi,
>>> BTW; there's a BLI_dynstr module too, but hardly used and confusing  
>>> design.
>>> Can be much more simple.
>>
>> I took a quick look in my (possibly outdated) CVS BF-Blender tree, and
>> it had only five public functions (create, destroy, append, get, and
>> get_length). Looked very straight-forward, although with such a thin API
>> it's perhaps not very useful (which, of course, might be why it's not
>> being used :).
>
> I would make a compatible system, so traditional strings and 
> 'dynstring' can be used both. Just by still using char pointers. 
> That's why I mentioned our malloc system, you can use that to find out 
> if the string is actually allocated, to prevent errors.
>
> Further it'll fit in SNDA simply, you can then call standard string 
> library functions too, and all other string-eating stuff in Blender.
>
> -Ton-
What if the string is inside a struct?  You'd still need to manually 
specify the size in that case.

Joe