[Bf-committers] Makesdna fails with "wrong" pointer syntax

Campbell Barton ideasman42 at gmail.com
Mon Mar 5 17:24:16 CET 2012


Added check r44651. this is now explicitly disallowed and wont compile
when its used.
Support for this syntax ends up with bigger changes then I'd have
liked so its not really worth it IMHO.

On Mon, Mar 5, 2012 at 11:45 PM, Campbell Barton <ideasman42 at gmail.com> wrote:
> Agree this is horrible and incredibly frustrating when running into
> problems like this (especially with DNA).
> I'll look into a fix.
>
> On Mon, Mar 5, 2012 at 8:04 PM, Tom Edwards <contact at steamreview.org> wrote:
>> I don't mind if it throws an error, just so long as it throws one.
>>
>> On 05/03/2012 12:31, Joshua Leung wrote:
>>> Legal but semantically confusing:
>>> ID* a, b, c;<--- only a is pointer, b and c are not.
>>>
>>> IMO this is bad style, and I'm glad that makesdna actually ends up
>>> enforcing it, albeit with a confusing error message. So, a big -1 to
>>> this idea from me.
>>>
>>> On Mon, Mar 5, 2012 at 1:20 PM, Tom Edwards<contact at steamreview.org>  wrote:
>>>> Hi everyone. I just spent 1hr+ hunting an annoying bug down. The
>>>> following declaration in a DNA header will cause makesdna to fail with a
>>>> generic "strut unknown" error, despite being entirely legal:
>>>>
>>>> ID* data;
>>>>
>>>> Putting the space in front of the asterisk avoids the problem. Fix
>>>> please? :)
>>>> _______________________________________________
>>>> Bf-committers mailing list
>>>> 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
>>>
>> _______________________________________________
>> Bf-committers mailing list
>> Bf-committers at blender.org
>> http://lists.blender.org/mailman/listinfo/bf-committers
>
>
>
> --
> - Campbell



-- 
- Campbell


More information about the Bf-committers mailing list