Re: [PD-dev] adding a built-in [change] object to the built-in GUI objects
On Dec 1, 2012, at 3:28 AM, Jonathan Wilkes wrote:
- Original Message -
From: Hans-Christoph Steiner h...@at.or.at
To: pd-dev@iem.at List pd-dev@iem.at
Cc:
Sent: Friday, November 30, 2012 11:20 PM
Subject: [PD-dev] adding a built-in [change] object to the built-in GUI
objects
Lots of patches use the built-in GUI objects for displays, and often a fast
stream of events is hooked straight up to the GUI object, causing the GUI
object
to send many pointless updates, like draw commands when the number hasn't
changed, or multiple draw commands per screen refresh cycle.
The multiple draw commands per screen refresh cycle seems like the more common
source of needless draw commands.
That should be addressed too, but that's a lot more complicated. Honestly, I
think it would be better to rewrite the basic GUI objects from scratch rather
than put more into the current ones.
I propose to only send the GUI update commands when the relevant value has
changed. I think this should apply to both the main element, like the
slider
knob, and the label for that GUI object, since that's often used as a
display. The code change is pretty simple, but I was wondering if people
thought there could be any problems caused by this
At the end of hslider_set, why not just check if x-x_value==(int)(100.0*g +
0.4)
before assigning it? If its already equal just return.
Good idea, that's what I ended up doing:
http://pure-data.svn.sourceforge.net/viewvc/pure-data?view=revisionrevision=16643
Then I did something similar for the [label ( draws:
http://pure-data.git.sourceforge.net/git/gitweb.cgi?p=pure-data/pd-extended.git;a=commitdiff;h=0271c92082c6db85eccba0f0b1226b9dbff09ceb
.hc
Here is the needed change, for example, for the hslider knob:
index b352fb9..88681fc 100644
--- a/src/g_all_guis.h
+++ b/src/g_all_guis.h
@@ -185,6 +185,7 @@ typedef struct _hslider
t_iemgui x_gui;
int x_pos;
int x_val;
+int x_prev_val;
int x_center;
int x_thick;
int x_lin0_log1;
index 470771a..e1a3c83 100644
--- a/src/g_hslider.c
+++ b/src/g_hslider.c
@@ -33,7 +33,7 @@ static t_class *hslider_class;
static void hslider_draw_update(t_gobj *client, t_glist *glist)
{
t_hslider *x = (t_hslider *)client;
-if (glist_isvisible(glist))
+if (glist_isvisible(glist) x-x_val != x-x_prev_val)
{
int r = text_xpix(x-x_gui.x_obj, glist) + (x-x_val +
50)/100;
int ypos=text_ypix(x-x_gui.x_obj, glist);
@@ -57,6 +57,7 @@ static void hslider_draw_update(t_gobj *client, t_glist
*glist)
x-x_thick = 0;
}
}
+x-x_prev_val = x-x_val;
}
}
___
Pd-dev mailing list
Pd-dev@iem.at
http://lists.puredata.info/listinfo/pd-dev
___
Pd-dev mailing list
Pd-dev@iem.at
http://lists.puredata.info/listinfo/pd-dev
Re: [PD-dev] adding a built-in [change] object to the built-in GUI objects
- Original Message -
From: Hans-Christoph Steiner h...@at.or.at
To: Jonathan Wilkes jancs...@yahoo.com
Cc: pd-dev@iem.at List pd-dev@iem.at
Sent: Monday, December 3, 2012 11:02 AM
Subject: Re: [PD-dev] adding a built-in [change] object to the built-in GUI
objects
On Dec 1, 2012, at 3:28 AM, Jonathan Wilkes wrote:
- Original Message -
From: Hans-Christoph Steiner h...@at.or.at
To: pd-dev@iem.at List pd-dev@iem.at
Cc:
Sent: Friday, November 30, 2012 11:20 PM
Subject: [PD-dev] adding a built-in [change] object to the built-in GUI
objects
Lots of patches use the built-in GUI objects for displays, and often a
fast
stream of events is hooked straight up to the GUI object, causing the
GUI object
to send many pointless updates, like draw commands when the number
hasn't
changed, or multiple draw commands per screen refresh cycle.
The multiple draw commands per screen refresh cycle seems like the more
common
source of needless draw commands.
That should be addressed too, but that's a lot more complicated. Honestly,
I think it would be better to rewrite the basic GUI objects from scratch
rather
than put more into the current ones.
It'd be even better to use a modern GUI toolkit that has simple tools to
implement
bleeding edge UX technology from the past 15 years. Stuff like hyperlinks. :)
-Jonathan
I propose to only send the GUI update commands when the relevant value
has
changed. I think this should apply to both the main element, like the
slider
knob, and the label for that GUI object, since that's often used as
a
display. The code change is pretty simple, but I was wondering if
people
thought there could be any problems caused by this
At the end of hslider_set, why not just check if
x-x_value==(int)(100.0*g + 0.4)
before assigning it? If its already equal just return.
Good idea, that's what I ended up doing:
http://pure-data.svn.sourceforge.net/viewvc/pure-data?view=revisionrevision=16643
Then I did something similar for the [label ( draws:
http://pure-data.git.sourceforge.net/git/gitweb.cgi?p=pure-data/pd-extended.git;a=commitdiff;h=0271c92082c6db85eccba0f0b1226b9dbff09ceb
.hc
Here is the needed change, for example, for the hslider knob:
index b352fb9..88681fc 100644
--- a/src/g_all_guis.h
+++ b/src/g_all_guis.h
@@ -185,6 +185,7 @@ typedef struct _hslider
t_iemgui x_gui;
int x_pos;
int x_val;
+ int x_prev_val;
int x_center;
int x_thick;
int x_lin0_log1;
index 470771a..e1a3c83 100644
--- a/src/g_hslider.c
+++ b/src/g_hslider.c
@@ -33,7 +33,7 @@ static t_class *hslider_class;
static void hslider_draw_update(t_gobj *client, t_glist *glist)
{
t_hslider *x = (t_hslider *)client;
- if (glist_isvisible(glist))
+ if (glist_isvisible(glist) x-x_val !=
x-x_prev_val)
{
int r = text_xpix(x-x_gui.x_obj, glist) +
(x-x_val +
50)/100;
int ypos=text_ypix(x-x_gui.x_obj, glist);
@@ -57,6 +57,7 @@ static void hslider_draw_update(t_gobj *client,
t_glist
*glist)
x-x_thick = 0;
}
}
+ x-x_prev_val = x-x_val;
}
}
___
Pd-dev mailing list
Pd-dev@iem.at
http://lists.puredata.info/listinfo/pd-dev
___
Pd-dev mailing list
Pd-dev@iem.at
http://lists.puredata.info/listinfo/pd-dev
Re: [PD-dev] adding a built-in [change] object to the built-in GUI objects
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 2012-12-03 18:10, Jonathan Wilkes wrote: It'd be even better to use a modern GUI toolkit that has simple tools to implement bleeding edge UX technology from the past 15 years. Stuff like hyperlinks. :) if hyperlinks is the criterion, then i don't see any problems with tcl/tk. fgasdmr IOhannes -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.12 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAlC83a4ACgkQkX2Xpv6ydvRrYwCg4GEd95Nns/+NYfxLTBwLL02y VckAoNH7DA9MPFwGcpfGYAXNohrI7Q8k =0weM -END PGP SIGNATURE- ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
[PD-dev] [ pure-data-Patches-3591846 ] portaudio fixes
Patches item #3591846, was opened at 2012-12-02 08:56 Message generated for change (Comment added) made by millerpuckette You can respond by visiting: https://sourceforge.net/tracker/?func=detailatid=478072aid=3591846group_id=55736 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: Open Resolution: None Priority: 9 Private: No Submitted By: IOhannes m zmölnig (zmoelnig) Assigned to: Miller Puckette (millerpuckette) Summary: portaudio fixes Initial Comment: this patches fixes the current portaudio implementation. since it is too big to be added to the patch-tracker (i imported portaudio-20121129), you can download it from http://iem.at/~zmoelnig/0001-updated-to-portaudio-20121129-snapshot.patch.bz2 -- Comment By: Miller Puckette (millerpuckette) Date: 2012-12-03 10:05 Message: I tried this and got: [msp@fuzz pd]$ git apply ~/bis/var/download/0001-updated-to-portaudio-20121129-snapshot.patch /home/msp/bis/var/download/0001-updated-to-portaudio-20121129-snapshot.patch:287: trailing whitespace. /home/msp/bis/var/download/0001-updated-to-portaudio-20121129-snapshot.patch:300: trailing whitespace. /home/msp/bis/var/download/0001-updated-to-portaudio-20121129-snapshot.patch:336: trailing whitespace. * The text above constitutes the entire PortAudio license; however, /home/msp/bis/var/download/0001-updated-to-portaudio-20121129-snapshot.patch:341: trailing whitespace. * they can be incorporated into the canonical version. It is also /home/msp/bis/var/download/0001-updated-to-portaudio-20121129-snapshot.patch:342: trailing whitespace. * requested that these non-binding requests be included along with the warning: squelched 32389 whitespace errors warning: 32394 lines add whitespace errors. ... is there any way you can supply a patch that doens't add trailing white space to 1/2 ot the Pd source? thanks M -- Comment By: IOhannes m zmölnig (zmoelnig) Date: 2012-12-02 09:07 Message: raising priority, since the current state of affairs breaks all builds from current git. what i did: - after the import of the portaudio-20121101 snapshot, some files where missing to smoothly integrate with Pd's autotools system. - so I re-imported portaudio with the missing files (and used the 20121126 snapshot, since that was the one i found on the portaudio website) - I tried to keep the import to a minimum (but imported all files an ordinary make would want to run a compilation) - i added the new portaudio build-system to pd/configure.ac (so pa's configure will get called automatically by Pd's configure) - i also adjusted the link-path of libportaudio in pd/src/Makefile.am to it's new location -- You can respond by visiting: https://sourceforge.net/tracker/?func=detailatid=478072aid=3591846group_id=55736 ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
Re: [PD-dev] [ pure-data-Patches-3591846 ] portaudio fixes
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 2012-12-03 19:05, SourceForge.net wrote: Comment By: Miller Puckette (millerpuckette) Date: 2012-12-03 10:05 Message: I tried this and got: [...] ... is there any way you can supply a patch that doens't add trailing white space to 1/2 ot the Pd source? thanks M hi miller. for what it is worth, my usual workflow now includes a whitespace checker, so i don't incidentally submit patches with whitespaces any more (i simply enabled the pre-commit hook in my .git/hooks/). this has been active for some time now (on all machines i currently develop on) otoh, the trailing whitespace in _this_ patch was intended, in order to keep the portaudio sources as untouched as possible. the portaudio source base doesn't seem to care at all, whether they have lines ending with whitespace or not. afaict, your original portaudio import also contains trailing whitespace (at least that's how i discovered that in order to keep the patch-set minimal, i should preserve the trailing whitespaces). if you would prefer, i could prepare another import of portaudio without trailing whitespaces. fg,asdr IOhannes -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.12 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAlC87K4ACgkQkX2Xpv6ydvT5lQCeIVbZtcj0X2r49wxIiSDW4LR3 dFAAn1T2CUMbw1SLCRIA0nVT7/Qid98V =3CjY -END PGP SIGNATURE- ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
[PD-dev] [ pure-data-Patches-3591846 ] portaudio fixes
Patches item #3591846, was opened at 2012-12-02 08:56 Message generated for change (Comment added) made by millerpuckette You can respond by visiting: https://sourceforge.net/tracker/?func=detailatid=478072aid=3591846group_id=55736 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: Open Resolution: None Priority: 9 Private: No Submitted By: IOhannes m zmölnig (zmoelnig) Assigned to: Miller Puckette (millerpuckette) Summary: portaudio fixes Initial Comment: this patches fixes the current portaudio implementation. since it is too big to be added to the patch-tracker (i imported portaudio-20121129), you can download it from http://iem.at/~zmoelnig/0001-updated-to-portaudio-20121129-snapshot.patch.bz2 -- Comment By: Miller Puckette (millerpuckette) Date: 2012-12-03 10:46 Message: ... in case this makes it easier, here's the snapshot I last used: http://crca.ucsd.edu/~msp/tmp/pa_snapshot_20121031.tgz if you try diff -r --brief between this and Pd's copy in src/portaudoi you can see how I pruned the portaudio tree to include only stuff relevant to Pd. Oddly, I had to supply one extra file though, pd/portaudio/src/os/mac_osx/pa_mac_hostapis.c -- Comment By: Miller Puckette (millerpuckette) Date: 2012-12-03 10:05 Message: I tried this and got: [msp@fuzz pd]$ git apply ~/bis/var/download/0001-updated-to-portaudio-20121129-snapshot.patch /home/msp/bis/var/download/0001-updated-to-portaudio-20121129-snapshot.patch:287: trailing whitespace. /home/msp/bis/var/download/0001-updated-to-portaudio-20121129-snapshot.patch:300: trailing whitespace. /home/msp/bis/var/download/0001-updated-to-portaudio-20121129-snapshot.patch:336: trailing whitespace. * The text above constitutes the entire PortAudio license; however, /home/msp/bis/var/download/0001-updated-to-portaudio-20121129-snapshot.patch:341: trailing whitespace. * they can be incorporated into the canonical version. It is also /home/msp/bis/var/download/0001-updated-to-portaudio-20121129-snapshot.patch:342: trailing whitespace. * requested that these non-binding requests be included along with the warning: squelched 32389 whitespace errors warning: 32394 lines add whitespace errors. ... is there any way you can supply a patch that doens't add trailing white space to 1/2 ot the Pd source? thanks M -- Comment By: IOhannes m zmölnig (zmoelnig) Date: 2012-12-02 09:07 Message: raising priority, since the current state of affairs breaks all builds from current git. what i did: - after the import of the portaudio-20121101 snapshot, some files where missing to smoothly integrate with Pd's autotools system. - so I re-imported portaudio with the missing files (and used the 20121126 snapshot, since that was the one i found on the portaudio website) - I tried to keep the import to a minimum (but imported all files an ordinary make would want to run a compilation) - i added the new portaudio build-system to pd/configure.ac (so pa's configure will get called automatically by Pd's configure) - i also adjusted the link-path of libportaudio in pd/src/Makefile.am to it's new location -- You can respond by visiting: https://sourceforge.net/tracker/?func=detailatid=478072aid=3591846group_id=55736 ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
Re: [PD-dev] [ pure-data-Patches-3591846 ] portaudio fixes
Aha... so all the whitespace trouble was their fault, not yours :) Anyhow, it's on the tracker, but you can get the one I was using: http://crca.ucsd.edu/~msp/tmp/pa_snapshot_20121031.tgz but I'd sort of rather not haul all the examples, tests, qa, and wierd APIs into the Pd tree if there's any smooth way to avoid that - can it be done by making minor adjustments to makefiles in Pa insteadby any chance? thanks M On Mon, Dec 03, 2012 at 07:17:21PM +0100, IOhannes m zmoelnig wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 2012-12-03 19:05, SourceForge.net wrote: Comment By: Miller Puckette (millerpuckette) Date: 2012-12-03 10:05 Message: I tried this and got: [...] ... is there any way you can supply a patch that doens't add trailing white space to 1/2 ot the Pd source? thanks M hi miller. for what it is worth, my usual workflow now includes a whitespace checker, so i don't incidentally submit patches with whitespaces any more (i simply enabled the pre-commit hook in my .git/hooks/). this has been active for some time now (on all machines i currently develop on) otoh, the trailing whitespace in _this_ patch was intended, in order to keep the portaudio sources as untouched as possible. the portaudio source base doesn't seem to care at all, whether they have lines ending with whitespace or not. afaict, your original portaudio import also contains trailing whitespace (at least that's how i discovered that in order to keep the patch-set minimal, i should preserve the trailing whitespaces). if you would prefer, i could prepare another import of portaudio without trailing whitespaces. fg,asdr IOhannes -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.12 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAlC87K4ACgkQkX2Xpv6ydvT5lQCeIVbZtcj0X2r49wxIiSDW4LR3 dFAAn1T2CUMbw1SLCRIA0nVT7/Qid98V =3CjY -END PGP SIGNATURE- ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
Re: [PD-dev] [ pure-data-Patches-3591846 ] portaudio fixes
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 2012-12-03 19:49, Miller Puckette wrote: Aha... so all the whitespace trouble was their fault, not yours :) Anyhow, it's on the tracker, but you can get the one I was using: http://crca.ucsd.edu/~msp/tmp/pa_snapshot_20121031.tgz but I'd sort of rather not haul all the examples, tests, qa, and wierd APIs into the Pd tree if there's any smooth way to avoid that - can it be done by making minor adjustments to makefiles in Pa insteadby any chance? yes, i think this should be possible as well. do you have a particular reason to use the 2012-10-31 snapshot rather than the one from tonight? i could try to make your import work as well (i cannot fully remember what was the reason your import broke the build; i think it was mainly due to some forgotten build-files, which are usually gitignored because they are generated; in the case of portaudio i think we have to make some exceptions to the never add generated files to the VCS rule) fgmasdr IOhannes -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.12 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAlC89ZYACgkQkX2Xpv6ydvRJCACfa9T8GtvlNdUG+dX2JBAv0fSC KSUAoM5Xz7dJbPiq58rt4F500uxOkRCp =EbYz -END PGP SIGNATURE- ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
[PD-dev] [ pure-data-Patches-3582746 ] fix setting of [bng]'s hold time in properties panel
Patches item #3582746, was opened at 2012-11-02 14:39
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detailatid=478072aid=3582746group_id=55736
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: Pending
Resolution: Accepted
Priority: 5
Private: No
Submitted By: Hans-Christoph Steiner (eighthave)
Assigned to: Miller Puckette (millerpuckette)
Summary: fix setting of [bng]'s hold time in properties panel
Initial Comment:
one of the variables didn't get renamed in the GUI rewrite:
--- a/tcl/dialog_iemgui.tcl
+++ b/tcl/dialog_iemgui.tcl
@@ -73,7 +73,7 @@ proc ::dialog_iemgui::sched_rng {mytoplevel} {
$mytoplevel.rng.max_ent configure -textvariable $var_iemgui_max_rng
$mytoplevel.rng.min_ent configure -textvariable
$var_iemgui_min_rng }
if {[eval concat $$var_iemgui_max_rng] $define_min_flashhold} {
-set $var_iemgui_max_rng $iemgui_define_min_flashhold
+set $var_iemgui_max_rng $define_min_flashhold
$mytoplevel.rng.max_ent configure -textvariable $var_iemgui_max_rng
}
if {[eval concat $$var_iemgui_min_rng] $define_min_flashbreak} {
--
Comment By: Miller Puckette (millerpuckette)
Date: 2012-12-03 11:19
Message:
applied for 0.44
--
You can respond by visiting:
https://sourceforge.net/tracker/?func=detailatid=478072aid=3582746group_id=55736
___
Pd-dev mailing list
Pd-dev@iem.at
http://lists.puredata.info/listinfo/pd-dev
Re: [PD-dev] [ pure-data-Patches-3591846 ] portaudio fixes
No good reason to use one snapshot over another except that it helps limit the number of changes git sees in the source tree. So if the newer snapshot makes material improvements that might be the better way after all. cheers M On Mon, Dec 03, 2012 at 07:55:20PM +0100, IOhannes m zmoelnig wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 2012-12-03 19:49, Miller Puckette wrote: Aha... so all the whitespace trouble was their fault, not yours :) Anyhow, it's on the tracker, but you can get the one I was using: http://crca.ucsd.edu/~msp/tmp/pa_snapshot_20121031.tgz but I'd sort of rather not haul all the examples, tests, qa, and wierd APIs into the Pd tree if there's any smooth way to avoid that - can it be done by making minor adjustments to makefiles in Pa insteadby any chance? yes, i think this should be possible as well. do you have a particular reason to use the 2012-10-31 snapshot rather than the one from tonight? i could try to make your import work as well (i cannot fully remember what was the reason your import broke the build; i think it was mainly due to some forgotten build-files, which are usually gitignored because they are generated; in the case of portaudio i think we have to make some exceptions to the never add generated files to the VCS rule) fgmasdr IOhannes -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.12 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAlC89ZYACgkQkX2Xpv6ydvRJCACfa9T8GtvlNdUG+dX2JBAv0fSC KSUAoM5Xz7dJbPiq58rt4F500uxOkRCp =EbYz -END PGP SIGNATURE- ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
Re: [PD-dev] adding a built-in [change] object to the built-in GUI objects
- Original Message - From: IOhannes m zmoelnig zmoel...@iem.at To: pd-dev@iem.at Cc: Sent: Monday, December 3, 2012 12:13 PM Subject: Re: [PD-dev] adding a built-in [change] object to the built-in GUI objects -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 2012-12-03 18:10, Jonathan Wilkes wrote: It'd be even better to use a modern GUI toolkit that has simple tools to implement bleeding edge UX technology from the past 15 years. Stuff like hyperlinks. :) if hyperlinks is the criterion, then i don't see any problems with tcl/tk. It's not a binary matter. It's a question of whether I can describe a hyperlink with the tools that tcl/tk provides and not have to spend the same amount of time as I would in c to implement them. Specifically-- tags in a tk canvas are per item while its the opposite in tk text widget. This isn't a difference between defaults for widgets, but incompatibility between them. It means in a canvas you can't easily set Enter/Leave events for a region made up of multiple polygons (which would require unmaintained tkzinc library), and alternately you can't easily define hyperlink Enter/Leave bindings in a text widget without ending up with accidental contiguous tag regions (two hyperlinks in a row, or one hyperlink above another on the next line). Wrt to the Browser2.0 plugin, the straightforward workaround of a proc that defines a new tag name for each hyperlink would probably end up having performance issues since that could potentially be 1000s of unique tags per search. My specific workaround isn't pretty and I can improve it some, but it's still going to suffer from this problem. -Jonathan fgasdmr IOhannes -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.12 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAlC83a4ACgkQkX2Xpv6ydvRrYwCg4GEd95Nns/+NYfxLTBwLL02y VckAoNH7DA9MPFwGcpfGYAXNohrI7Q8k =0weM -END PGP SIGNATURE- ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
[PD-dev] building for Windows on GNU/Linux
Hey Miller, I see that you are using MSVC via Wine on GNU/Linux. I don't know how well supported that arrangement is, for a widely supported version of that arrangement, you should try running the MinGW builds on GNU/Linux. The makefile.mingw should work fine for that just set compiler (i.e. make CC=/path/to/mingw-gcc). Here's lots of docs for doing this on Fedora: https://fedoraproject.org/wiki/MinGW Most distros include a mingw-gcc cross-compiler, so they just need to 'yum install', 'apt-get install', etc. .hc ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
Re: [PD-dev] building for Windows on GNU/Linux
yeah, one of the reasons I'm trying wine out is so I can have both MSVC and mingw running (and be sure they don't step on each others' toes, which I can do by setting up separate disks for them to live on). Amazingly, I can even run Pd in wine, although I haven't tried to do anything with it. this is part of a larger project to set up a few virtual environments (wine but also xen for running various linuxes and Windowses) so that I can be more confident that changes I make won't be breaking things for everyone else. Of course (see Zmoelnig's new thread) I seem so far to have mostly made negative progress. cheers M On Mon, Dec 03, 2012 at 04:55:31PM -0500, Hans-Christoph Steiner wrote: Hey Miller, I see that you are using MSVC via Wine on GNU/Linux. I don't know how well supported that arrangement is, for a widely supported version of that arrangement, you should try running the MinGW builds on GNU/Linux. The makefile.mingw should work fine for that just set compiler (i.e. make CC=/path/to/mingw-gcc). Here's lots of docs for doing this on Fedora: https://fedoraproject.org/wiki/MinGW Most distros include a mingw-gcc cross-compiler, so they just need to 'yum install', 'apt-get install', etc. .hc ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
[PD-dev] [ pure-data-Bugs-3592292 ] Feedback on beta version: issue when editing
Bugs item #3592292, was opened at 2012-12-03 15:20 Message generated for change (Tracker Item Submitted) made by ceplatel You can respond by visiting: https://sourceforge.net/tracker/?func=detailatid=478070aid=3592292group_id=55736 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: pd-extended Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Charles Platel (ceplatel) Assigned to: Hans-Christoph Steiner (eighthave) Summary: Feedback on beta version: issue when editing Initial Comment: Pd version 0.43.4-extended 20121101. Mac OSX Lion with 2 screens. Sometimes when several window patches are open for editing, they can loose the pd menu. It can be retrieved by clicking in the TCL console windows. It never happen withe the former Pd version 0.42.5-extended. Not easy to reproduce to study this issue. -- You can respond by visiting: https://sourceforge.net/tracker/?func=detailatid=478070aid=3592292group_id=55736 ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
[PD-dev] [ pure-data-Patches-3585457 ] support for OSSv4
Patches item #3585457, was opened at 2012-11-08 12:17 Message generated for change (Comment added) made by millerpuckette You can respond by visiting: https://sourceforge.net/tracker/?func=detailatid=478072aid=3585457group_id=55736 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: Pending Resolution: Accepted Priority: 7 Private: No Submitted By: IOhannes m zmölnig (zmoelnig) Assigned to: Miller Puckette (millerpuckette) Summary: support for OSSv4 Initial Comment: one of Pd's older audio-backends is OSS. while OSS is rather old and not a fast-moving target, some development does happen... e.g. a while ago (probably years) all SOUND_PCM_-defines have been renamed to SNDCTL_DSP_. luckily, the linux headers provided fallback defines, so old code (like Pd) would still compile. now OSS is not dead and is currently developed as OSSv4. unfortunately, the new OSSv4 headers have dropped the old fallback defines, and instead pro-actively create errors when the old defines are still used. the attached patch changes these defines (and provides fallbacks in case the *new* defines are not defined), in order to be able to still use the OSS backend. -- Comment By: Miller Puckette (millerpuckette) Date: 2012-12-03 18:01 Message: applied to 0.44 -- Comment By: IOhannes m zmölnig (zmoelnig) Date: 2012-11-08 12:19 Message: raising priority since the old s_audio_oss.c breaks compilation. -- You can respond by visiting: https://sourceforge.net/tracker/?func=detailatid=478072aid=3585457group_id=55736 ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
[PD-dev] [ pure-data-Patches-2947822 ] space character in gui labels causes disaster
Patches item #2947822, was opened at 2010-02-08 05:12
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detailatid=478072aid=2947822group_id=55736
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: Open
Resolution: Fixed
Priority: 5
Private: No
Submitted By: Matteo Sisti Sette (sistisette)
Assigned to: Miller Puckette (millerpuckette)
Summary: space character in gui labels causes disaster
Initial Comment:
Steps to reproduce:
1 - Open the attached patch
Note the size of the radio and toggle. Note also they have send symbols set
to xxx and yyy respectively
2 - Click on the big bang at the top
This composes a symbol containing space characters and sends it as a label to
both the radio and the toggle.
This seems to work: the label is actually set with spaces, and the radio and
toggle still work.
However, if you right-click on any of them and try to select properties, the
properties dialog won't open
3 - Now save the patch and close it
4 - Open the patch again. The radio and toggle properties have been completely
reset: their size have been reset to default, and also the number property
of the radio. They have lost their send symbol too: you can see the outlet
and if you use them, the r objects won't receive anything.
However, if you open the properties dialog of the radio or toggle, you'll see
that the send property is still there (not the same can be said for the size
and number); if you now do Apply or Ok, the send symbol will be restored.
So, spaces in the label of a gui object cause weirdnesses and potential
disaster. Note that data is lost (the properties of the gui objects) if the
patch is saved after assigning such a label, and i is lost silently with no
error message at any moment.
There are ather forbidden characters ithat don't behave properly n labels,
such as the open square bracket [ (as reported in another bug report).
However the space character is more disatrous as it causes data loss.
Object should either accept ALL characters for a label, or properly detect and
refuse forbidden characters, which should be documented, avoiding unexpected
consequences.
--
Comment By: Miller Puckette (millerpuckette)
Date: 2012-12-03 19:27
Message:
Note the much simpler way the number box finesses this problem. I'm
scared of adding
code to m_binbuf.c to escape spaces (what will happen when we later allow
backslashes
into the mix?). But without the binbuf patch the fix to the IEM guis
doesn't work - so I'm
tempted for now just to do as in the number box - it's not incompatible to
do it right at
some ater date when we can thonk the whole thing through (it's all over the
Pd code and all
needs to be figured out together).
--
Comment By: IOhannes m zmölnig (zmoelnig)
Date: 2012-11-13 16:45
Message:
added another patch (0002_*) that implements escaping spaces and tabs when
saving the files to disk.
i guess both patches should be applied for full fun.
--
Comment By: IOhannes m zmölnig (zmoelnig)
Date: 2012-11-13 15:36
Message:
the bug.pd file does not correctly show the labels with blanks after
reloading but it doesn't throw any warnings)
however, the bugfix-patch doesn't work for me.
nevertheless the problem is known and is not in the pd-gui communication,
but rather in the way the label is stored on disk (and then interpreted on
reload):
afaict, blanks (and other special chars) really must be escaped with
backslash (e.g. A\ A\ A) when saved to disk
--
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2012-11-13 12:52
Message:
the values were not quoted, so just quoting them with {} fixes it. The
patch is attached.
--
You can respond by visiting:
https://sourceforge.net/tracker/?func=detailatid=478072aid=2947822group_id=55736
___
Pd-dev mailing list
Pd-dev@iem.at
http://lists.puredata.info/listinfo/pd-dev
[PD-dev] [ pure-data-Patches-3587383 ] before loading a GUI plugin, check its not already loaded
Patches item #3587383, was opened at 2012-11-14 21:05 Message generated for change (Comment added) made by millerpuckette You can respond by visiting: https://sourceforge.net/tracker/?func=detailatid=478072aid=3587383group_id=55736 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: Pending Resolution: Accepted Priority: 7 Private: No Submitted By: Hans-Christoph Steiner (eighthave) Assigned to: Miller Puckette (millerpuckette) Summary: before loading a GUI plugin, check its not already loaded Initial Comment: only load GUI plugins with the same name once, to prevent mayhem The actual file name of the plugin is stored in a list. Whenever a plugin is being loaded, first pd-gui checks whether that file name is listed as loaded. If its loaded already, pd-gui prints a message then ignores it -- Comment By: Miller Puckette (millerpuckette) Date: 2012-12-03 20:24 Message: Mayhem you say? we needs more of that, what? OK for 0.44 -- You can respond by visiting: https://sourceforge.net/tracker/?func=detailatid=478072aid=3587383group_id=55736 ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
[PD-dev] [ pure-data-Patches-3587384 ] Replacing deprecated functions
Patches item #3587384, was opened at 2012-11-14 21:06 Message generated for change (Comment added) made by millerpuckette You can respond by visiting: https://sourceforge.net/tracker/?func=detailatid=478072aid=3587384group_id=55736 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: Pending Resolution: Accepted Priority: 5 Private: No Submitted By: Nobody/Anonymous (nobody) Assigned to: Miller Puckette (millerpuckette) Summary: Replacing deprecated functions Initial Comment: vexp_fun.c currently uses the old BSD-style functions drem and finite. These functions have been removed from XCode 4.4 and later, so that the expr~ family of externals fails to compile now. This patch replaces drem with remainder and finite with isfinite, which should work everywhere. -- Comment By: Miller Puckette (millerpuckette) Date: 2012-12-03 20:32 Message: accepted for 0.44 -- Comment By: Hans-Christoph Steiner (eighthave) Date: 2012-11-20 12:34 Message: Makes sense to me. remainder() is the C99 replacement for drem(), and isfinite() is the C99 replacement for finite(). C99 marks drem() and finite() as deprecated. -- You can respond by visiting: https://sourceforge.net/tracker/?func=detailatid=478072aid=3587384group_id=55736 ___ Pd-dev mailing list Pd-dev@iem.at http://lists.puredata.info/listinfo/pd-dev
