Patches item #1841002, was opened at 2007-11-29 06:11
Message generated for change (Settings changed) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1841002&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: bugfix
>Status: Closed
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: Russell Bryant (russellbryant)
Assigned to: Miller Puckette (millerpuckette)
Summary: Add format string error checking, and fix related errors
Initial Comment:
This patch adds format string error checking for bug(), error(), verbose(), and pd_error(). It also fixes the errors in the pd core that were pointed out by this change.
There are some other functions that take printf style format strings that would benefit from this same change, but I wanted to address pieces of it at a time.
----------------------------------------------------------------------
Comment By: Russell Bryant (russellbryant)
Date: 2007-11-30 05:33
Message:
Logged In: YES
user_id=1942915
Originator: YES
File Added: format_string.patch2.txt
----------------------------------------------------------------------
Comment By: Russell Bryant (russellbryant)
Date: 2007-11-30 05:32
Message:
Logged In: YES
user_id=1942915
Originator: YES
I assume you meant seems "not" very likely to do any harm. :)
Good point about MSVC. I am used to only caring about gcc. I have
provided an updated patch that only uses the attribute if GNUC is in use.
It is also implemented as a macro in one spot so that addressing it for
other compilers in the future can be done in one spot.
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2007-11-29 10:15
Message:
Logged In: YES
user_id=27104
Originator: NO
Looks like the right thing to do, and seems very likely to do any harm.
I'm going to try it in Pd-extended. The only concern I can think of is
whether the __attribute__ stuff causes any problems when building with
MSVC, since some people still build Pd with MSVC. This site seems to claim
it does not:
http://ocliteracy.com/techtips/gnu-c-attributes.html
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1841002&group_…
Patches item #1838621, was opened at 2007-11-26 03:11
Message generated for change (Settings changed) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1838621&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: feature
>Status: Closed
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: Cyrille Henry (nusmuk)
Assigned to: Miller Puckette (millerpuckette)
Summary: setable [route], just like [select]
Initial Comment:
this patch allow [route] to react just like [select] :
when route is create with 1 argument, a 2nd inlet is created in order to change this argument.
this ca be very usefull, and the patch is very small.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1838621&group_…
Patches item #1838468, was opened at 2007-11-25 21:57
Message generated for change (Settings changed) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1838468&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: feature
>Status: Closed
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Miller Puckette (millerpuckette)
Summary: made [print] display all arguments in its posts to pd window
Initial Comment:
Just like the subject says, I made [print] accept an A_GIMME, the arguments are then converted into a symbol, then that symbol is display as before.
I suppose x->x_sym could be a char* instead.
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2010-06-14 12:45
Message:
I had to change the code a bit for new Windows Vista/7 weirdness:
Following this discussion:
http://lists.puredata.info/pipermail/pd-list/2010-05/078312.html
I committed this to try to fix it:
http://pure-data.svn.sourceforge.net/viewvc/pure-data?view=rev&revision=135…
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2007-12-04 14:00
Message:
Logged In: YES
user_id=27104
Originator: NO
updated to fix minor bugs:
http://pure-data.cvs.sourceforge.net/pure-data/pd/src/x_interface.c?r1=1.3.…
File Added: made_print_post_all_arguments-0.40.3.patch
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2007-11-25 22:16
Message:
Logged In: YES
user_id=27104
Originator: NO
http://pure-data.cvs.sourceforge.net/pure-data/pd/src/x_interface.c?r1=1.3&…
File Added: made_print_post_all_arguments-0.40.3.patch
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2007-11-25 22:16
Message:
Logged In: YES
user_id=27104
Originator: NO
http://pure-data.cvs.sourceforge.net/pure-data/pd/src/x_interface.c?r1=1.3&…
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1838468&group_…
Its already being actively used in the GUI rewrite, which is currently
somewhat broken without it. I think the only client for that message
really would be pd-gui, and I'd happily do the work to change it if need
be. That's a much better option than omitting it.
.hc
Miller Puckette wrote:
> My usual strategy on stuff like this is wait until someone is actively
> using it - because one's guess as to how the feature should work is never
> how it's really needed when the time comes and then you're stuck maintaining
> two different versions of the same thing :)
>
> M
>
> On Sat, Jul 17, 2010 at 12:15:30AM -0400, Hans-Christoph Steiner wrote:
>> Yeah, I added the "pd set-startup", etc. stuff to m_glob.c and
>> s_path.c. Regardless of the startup procedure, I think pd-gui should
>> be able to request state from pd so it can stay in sync. A dialog
>> shouldn't be required to get this info. This is akin to IOhannes'
>> idea that pd-gui's state should be settable by pd at any point while
>> running.
>>
>> Then it makes sense to reuse the same procedure for getting the state
>> in the startup procedure, rather than having a custom method for only
>> the startup procedure.
>>
>> .hc
>>
>> On Jul 16, 2010, at 10:59 PM, Miller Puckette wrote:
>>
>>> Hi Hans --
>>>
>>> the "vwait" line didn't seem correct to me (could return right after
>>> Pd says to clear pd_path, before the other messages bubble up to
>>> append
>>> the various directories to it).
>>>
>>> I ended up fixing Pd to volunteer the path and startup stuff before
>>> sending
>>> pdtk_pd_startup so that none of the tcl side of things should be
>>> needed
>>> any longer. OTOH if you were planning to send "pd set-startup" etc at
>>> some later point in the tcl code, this would need to be supported in
>>> Pd...(?)
>>>
>>> cheers
>>> Miller
>>>
>>> On Fri, Jul 16, 2010 at 03:58:24PM -0400, Hans-Christoph Steiner
>>> wrote:
>>>> Hey Miller,
>>>>
>>>> I just noticed that you left out a key part of pd-gui-rewrite in
>>>> 0.43. There is the 'set-startup' and 'set-path' messages which
>>>> allows
>>>> pd-gui to get the state of those things without having to open the
>>>> respective preference dialog panels. The changes are in m_glob.c and
>>>> s_path.c, as well as these lines in pd-gui.tcl:
>>>>
>>>> pdsend "pd set-startup" ;# get ::startup_libraries
>>>> and ::startup_flags lists
>>>> pdsend "pd set-path" ;# get the ::pd_path list
>>>> vwait ::pd_path ;# wait for 'pd' to respond
>>>>
>>>> .hc
>>>>
>>>> ----------------------------------------------------------------------------
>>>>
>>>> "We have nothing to fear from love and commitment." - New York
>>>> Senator
>>>> Diane Savino, trying to convince the NY Senate to pass a gay marriage
>>>> bill
>>>>
>>>>
>>>> _______________________________________________
>>>> Pd-dev mailing list
>>>> Pd-dev(a)iem.at
>>>> http://lists.puredata.info/listinfo/pd-dev
>>
>>
>>
>>
>> ----------------------------------------------------------------------------
>>
>> Programs should be written for people to read, and only incidentally
>> for machines to execute.
>> - from Structure and Interpretation of Computer Programs
Patches item #1826507, was opened at 2007-11-05 18:16
Message generated for change (Settings changed) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1826507&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: bugfix
>Status: Closed
>Resolution: Out of Date
Priority: 5
Private: No
Submitted By: Hans-Christoph Steiner (eighthave)
Assigned to: Miller Puckette (millerpuckette)
Summary: fix data structures cursor on OSX and allow cursor configura
Initial Comment:
This patch moves the setting of the cursor names to u_main.tk so that they can be changed at runtime, allowing users to use all of the available Tk cursors from within patches.
It also fixes the problem with the CURSOR_RUNMODE_CLICKME not working on Mac OS X.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1826507&group_…
Patches item #1826144, was opened at 2007-11-05 07:41
Message generated for change (Settings changed) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1826144&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: feature
>Status: Closed
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: IOhannes m zmlnig (zmoelnig)
Assigned to: Miller Puckette (millerpuckette)
Summary: accept atoms as lists on righthand inlets
Initial Comment:
when an object has a list-method on its left inlet, it will happily accept floats and symbols as well (and handle them as 1-atom lists), if no specific method for these types is defined.
however, this is not true for right-hand inlets.
if such an inlet (with a "list" method, as declared with
receives a single float number (as output by the numberbox), it bails out with
"error: inlet: expected 'list' but got 'float'"
this is especially sad, as there is no way to add methods for several types of messages to a right-hand inlet (well, there are methods like using proxys; but we don't want to go into that)
a workaround for this is to run the message through [list] (or [list $1(), however i would rather not use this.
instead, the attached patch fixes this by modifying the inlet_* methods in m_obj.c to fallback to inlet_list() if all else fails.
btw, this is the code i use to add a list-method to the 2nd outlet:
new{
...
inlet_new(&x->x_obj, &x->x_obj.ob_pd, &s_list, gensym("list2"));
}
setup{
...
class_addmethod(myobj_class, (t_method)myobj_list2, gensym("list2"), A_GIMME, 0);
})
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2007-11-05 08:58
Message:
Logged In: YES
user_id=27104
Originator: NO
Sounds very useful, but I am sure there have got to be some unwanted
ramifications. Do you have any idea what they might be?
Is this ready to be included in 0.40.3-extended ?
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1826144&group_…
Patches item #1704511, was opened at 2007-04-20 10:45
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1704511&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: None
>Status: Closed
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: Hans-Christoph Steiner (eighthave)
Assigned to: Miller Puckette (millerpuckette)
Summary: Delete broken on Windows, key/keyup/keyname misses some keys
Initial Comment:
Pressing the Del key does nothing. Instead it should remove the character to the right. I tested this on Windows XP with Pd 0.38-4, 0.39-2, and 0.40-2. This works on Mac OS X. I did not test on GNU/Linux.
----------------------------------------------------------------------
>Comment By: Miller Puckette (millerpuckette)
Date: 2010-07-21 12:17
Message:
note -- only the C side was needed (gui side seems already to be updated as
of gui-rewrite)
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-05-09 12:53
Message:
Logged In: YES
user_id=27104
Originator: YES
Attached is a patch that fixed this, and also fixes BackSpace, Delete,
Tab, Space, Return, Escape for [key]/[keyup]/[keyname].
This patch also fixes bugs #1728414 and #1728405.
http://sourceforge.net/tracker/index.php?func=detail&aid=1728414&group_id=5…http://sourceforge.net/tracker/index.php?func=detail&aid=1728405&group_id=5…
File Added: fix_backspace_delete-0.41.4.patch
----------------------------------------------------------------------
Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2007-10-16 02:36
Message:
Logged In: YES
user_id=564396
Originator: NO
does it work once you have deleted a key with backspace? (sometimes
deleting is weird on linux)
apart from that: i just tested it with pd-0.40-1 on linux and both
backspace and delete work as expected)
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1704511&group_…
Patches item #1646604, was opened at 2007-01-28 13:33
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1646604&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: feature
>Status: Closed
>Resolution: Rejected
Priority: 5
Private: No
Submitted By: Martin Peach (mrpeach)
Assigned to: Nobody/Anonymous (nobody)
Summary: string type
Initial Comment:
This patch implements support for a string type in Pure Data. A t_string is basically a pointer to a byte array and a length.
The file str.c in externals/mrpeach/str implements an external that will work with this.
The t_string type should be useful for constructing and manipulating strings, very long integers, image rasters and other things. Arbitrary strings can be constructed to meet the requirements of external devices connected via serial or network interfaces.
----------------------------------------------------------------------
>Comment By: Miller Puckette (millerpuckette)
Date: 2010-07-21 11:03
Message:
should be done differently via arrays of numbers
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1646604&group_…
Patches item #1572290, was opened at 2006-10-06 10:48
Message generated for change (Settings changed) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1572290&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: None
>Status: Closed
>Resolution: Out of Date
Priority: 8
Private: No
Submitted By: Hans-Christoph Steiner (eighthave)
Assigned to: Miller Puckette (millerpuckette)
Summary: serious troubles with folders with non-ASCII chars on WinXP
Initial Comment:
On Windows XP, you cannot save a new file to any
directory that has non-ASCII characters in it. For
example, if someone has an accented E in their
username, then their home folder will be in
C:\Documents and Settings\USERNAME, where USERNAME is
the name with the accented E.
The work around is to save to the root of C:\, then
move the file to the directory with the non-ASCII
chars. After that, it is possible to save changes.
But the only way to open the file is to double-click it
and let it open via file associations. File->Open
crashes Pd-0.40-2 and nothing happens on
Pd-0.39.2-extended-test5.
This has been confirmed on Pd-0.39.2-extended-test5 and
Pd-0.40-0, on Windows XP US and Spanish editions.
----------------------------------------------------------------------
Comment By: IOhannes m zmlnig (zmoelnig)
Date: 2009-06-22 10:24
Message:
confirmed with PdX-0.41-4
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2008-05-11 12:48
Message:
Logged In: YES
user_id=27104
Originator: YES
the problem was that the Tcl internal character encoding was just being
sent straight to the pd process without being converted to the current
system encoding. I added Tcl_UtfToExternalDString() as an easy way to
convert from the Tcl encoding to the current system encoding and it seems
to work fine now.
I am raising the priority since it can prevent people from saving.
File Added: fix_encoding_bug_1572290-0.41.4.patch
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2006-10-06 10:52
Message:
Logged In: YES
user_id=27104
Here's an example error message when I tried to save a new file:
error: C:/Documents and
Settings/hans/Desktop/testbug-üé/Untitled-1.pd: No such file
or directory
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1572290&group_…
Patches item #1561099, was opened at 2006-09-18 16:55
Message generated for change (Settings changed) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1561099&group_…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata
Group: None
>Status: Closed
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Miller Puckette (millerpuckette)
Summary: Unused vars
Initial Comment:
Here is a patch with unused variables throughout PD.
It'll help readability.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1561099&group_…