On 03/21/2013 20:26, Ivica Ico Bukvic wrote:
> On 03/21/2013 03:12 PM, Ivica Ico Bukvic wrote:
>> IOhannes,

hi

>>
>> Your patch has one regression:

thanks for reporting

>>
>> --- a/src/x_connective.c
>> +++ b/src/x_connective.c
>> @@ -991,6 +991,8 @@ static void trigger_list(t_trigger *x, t_symbol
>> *s, int argc, t_atom *argv)
>>          else if (u->u_type == TR_SYMBOL)
>>              outlet_symbol(u->u_outlet,
>>                  (argc ? atom_getsymbol(argv) : &s_symbol));
>> +        else if (u->u_type == TR_ANYTHING)
>> +            outlet_anything(u->u_outlet, s, argc, argv);
>>          else if (u->u_type == TR_POINTER)
>>          {
>>              if (!argc || argv->a_type != TR_POINTER)
>>
>>
[...]

>> Any thoughts?
>>

yep.
outlet_list() is *not* the same as outlet_anything()....


>>
>>
> As of right now, I cannot see any regressions by removing that part
> since at the end it does output_list call anyhow. Can you test
> this/confirm?


i cannot test right now, but i'm pretty sure that removing that very
line makes my entire patch obsolete and reverts to the old behaviour.

which errors do you get?

fgmadsr
IOhannes

_______________________________________________
[email protected] mailing list
UNSUBSCRIBE and account-management -> 
http://lists.puredata.info/listinfo/pd-list

Reply via email to