atom feed9 messages in at.iem.pd-listRe: [PD] Crasher bug found...
FromSent OnAttachments
Luke IanniniFeb 1, 2009 12:43 am.txt, .pd, .pd
Hans-Christoph SteinerFeb 1, 2009 11:55 am 
Mathieu BouchardFeb 1, 2009 6:17 pm 
Miller PucketteFeb 2, 2009 9:05 am 
Mathieu BouchardFeb 2, 2009 9:38 am 
Miller PucketteFeb 2, 2009 6:44 pm 
Luke IanniniFeb 10, 2009 4:06 am 
Mathieu BouchardFeb 10, 2009 6:01 am 
IOhannes m zmölnigFeb 10, 2009 10:14 am 
Subject:Re: [PD] Crasher bug found...
From:Mathieu Bouchard (mat@artengine.ca)
Date:Feb 10, 2009 6:01:16 am
List:at.iem.pd-list

On Tue, 10 Feb 2009, Luke Iannini wrote:

Hi Miller, a little late, but just wanted to say a big thanks for finding this and fixing it! I'm very happy to be able to continue work on my sequencers crash-free : )

Also, there is another thing that looks like a bug, but is about something undocumented. [makefilename] without an argument seems like it was intended to do [makefilename file.%d], but there's this same missing star:

if (!s->s_name) s = gensym("file.%d"); vs if (!*s->s_name) s = gensym("file.%d");

Because Pd uses &s_ as default value for a symbol arg, not a null pointer, so s->s_name is never considered "false"... same bug, really.

I don't really care about that one, nor that anyone would want to use that default value, and the line could also be removed altogether, and it would be as fine, as removing a useless line is good, and removing a misleading useless line is even better.