-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
the recent commit 78b81aa3cb90 on the puredata/master branch breaks
ABI compatibility with externals compiled for Pd-0.43.
the problem is that the "sys_close()" symbol is removed for non-w32
platforms.
therefore all the externals on non-w32 that (already) use sys_close()
(at least i have written a couple of them) will fail to load with a
new version of Pd, unless they are recompiled.
this makes packaging externals for e.g. Debian a nightmare, as it
basically should trigger a .so-name change, but since we are linking
against the application instead of an ordinary library, all the tools
that would detect such an incompatibility will fail.
so please revert the "#define sys_close close" stanzas.
instead i would ask you to provide sys_open() (and friends)
implementations in s_path, even for platforms where they are mere
wrappers around the system functions.
it also makes the header-file much easier to read (i don't think
anything in a public header-file but function decorations should be
ifdef'ed)
fgamsdr
IOhannes
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
iEYEARECAAYFAlDO7BgACgkQkX2Xpv6ydvTMIQCfYM+ifUeF2H3Bgh/o5C4S2vuz
kBEAnjfhlPz5jlU1KEIoZbAumtYF++B7
=maMx
-----END PGP SIGNATURE-----
Patches item #3597233, was opened at 2012-12-18 08:43
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3597233&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: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Hans-Christoph Steiner (eighthave)
Assigned to: Nobody/Anonymous (nobody)
Summary: Win32: use binary open mode by default everywhere
Initial Comment:
This patch enables binary open mode on Win32 everywhere, thereby ignoring the special Win32 "text" translation mode. This makes the Win32 API more like the POSIX API, which is used by every other platform Pd supports.
But it seems to break somethings, like the loading of some soundfiles. Attached is an example patch for the failure.
----------------------------------------------------------------------
>Comment By: Miller Puckette (millerpuckette)
Date: 2012-12-18 09:41
Message:
Just tried this in Wine and then (swear words deleted here) cranked up my
XP SP2 machine and tried it
there too - I can't see anything wrong yet. Is it breaking on a more
recent version of Windows?
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3597233&group_…
Patches item #3597233, was opened at 2012-12-18 08:43
Message generated for change (Tracker Item Submitted) made by eighthave
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3597233&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: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Hans-Christoph Steiner (eighthave)
Assigned to: Nobody/Anonymous (nobody)
Summary: Win32: use binary open mode by default everywhere
Initial Comment:
This patch enables binary open mode on Win32 everywhere, thereby ignoring the special Win32 "text" translation mode. This makes the Win32 API more like the POSIX API, which is used by every other platform Pd supports.
But it seems to break somethings, like the loading of some soundfiles. Attached is an example patch for the failure.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3597233&group_…
Bugs item #3597187, was opened at 2012-12-18 05:24
Message generated for change (Tracker Item Submitted) made by ceplatel
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=3597187&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: pd-extended
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Charles Platel (ceplatel)
Assigned to: Hans-Christoph Steiner (eighthave)
Summary: window size changed when saved
Initial Comment:
Pd version 0.43.4-extended 20121215. Mac OSX Lion
Please use the attached canvas-test.pd:
1. open it: scroll bars are displayed
2. widen it a ittle to make the scrollbars disapear
3. reduce it so that the window limits are close to the grey canvas limits and before the scrollbars come back
4. save the patch, then close the window
5. open the patch again: scrollbars are back!
Note: with pd-extended 0.42, the window looks exacly like it was saved.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=3597187&group_…
Patches item #3587404, was opened at 2012-11-14 22:29
Message generated for change (Settings changed) made by zmoelnig
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3587404&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: Open
Resolution: None
Priority: 5
Private: No
Submitted By: IOhannes m zmölnig (zmoelnig)
>Assigned to: Nobody/Anonymous (nobody)
Summary: ability to start multiple pd-guis
Initial Comment:
starting with Pd-0.43, pd-gui checks whether it is already running and tries to reconnect to the existing one.
while this is ok in some situations, it is an issue in other situations:
- when starting multiple instances of Pd on different desktops (i'm using the xfce window manager), the new instance will automatically focus the desktop of the already-running instance, which makes it difficult to separate those instances again (assuming that each instance has a number of windows open on startup)
- when using pd-gui to connect to a remote pd-core (while another instance of Pd is running on the same machines) is practically prohibited by this behaviour
it would be great if one could turn of this feature (at least when starting from the cmdline; and esp. when a <remotehost>:<remotport> is given...currently there is onl a check for remotport>5400 (when starting without <remotehost>)
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2012-12-17 09:37
Message:
I don't think that's a good enough test. The one you removed is much
stricter: $argc == 1 When pd starts pd-gui, it only sends one arg: the
port number. If you want this to affect only remote connections, shouldn't
you make the test based on ::host?
----------------------------------------------------------------------
Comment By: IOhannes m zmölnig (zmoelnig)
Date: 2012-12-17 08:37
Message:
attached is a patch (against todays git/master) that doesn't try to connect
to a running instance if the user provided a remoteport to connect to.
it re-uses the logic already implemented in parse_args().
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2012-12-14 20:25
Message:
Since you are the master of remote pd-guis, IOhannes, I'm assigning this to
you. It makes sense to me to base it on the host:port arg.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3587404&group_…
Patches item #3413809, was opened at 2011-09-25 11:06
Message generated for change (Comment added) made by eighthave
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3413809&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: Hans-Christoph Steiner (eighthave)
Assigned to: Miller Puckette (millerpuckette)
Summary: use Ctrl-< and Ctrl-> to cycle thru open windows
Initial Comment:
This patch allows you to use Ctrl/Cmd < and > to cycle through all of the open windows in different directions.
----------------------------------------------------------------------
>Comment By: Hans-Christoph Steiner (eighthave)
Date: 2012-12-17 12:11
Message:
Oh, sorry, I think I misunderstood, and I just tried it on GNOME and it
behaved similarly. This was mostly meant for Mac OS X. I guess that [wm
stackorder .] does differrent things on different platforms. For anyone
who wants to improve this, the thing to do would be to test for platform
using $::windowingsystem in ::pd_menucommands::menu_raisepreviouswindow
----------------------------------------------------------------------
Comment By: Miller Puckette (millerpuckette)
Date: 2012-12-17 12:02
Message:
OK - but man, is it bizarre how ctrl-< actes in XFCE - I'm guessing this is
XFCE being wierd
and not our fault. Anyhow, I've applied this for 0.44
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2012-12-15 17:36
Message:
yes, that's what is intended. I think each WM will have different ideas of
how this should work, like for people who have focus changing based on
mouse presence not clicking. But I think this is the most common paradigm.
Others can be implemented in plugins
----------------------------------------------------------------------
Comment By: Miller Puckette (millerpuckette)
Date: 2012-12-15 11:02
Message:
I tried this and got funny behavior - ctrl < focussed the previous window
OK, but also sent it to the back of
the window stacking order.. Can this be what was intended???
----------------------------------------------------------------------
Comment By: max (maxn)
Date: 2011-09-26 05:01
Message:
That's the standard on german keyboard layouts for cycling through the
windows of the active application on OS X. I'd very much apprechiate if
this would work in Pd. That said, i think it might actually be a bug in
Tcl/Tk that it isn't working right now.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3413809&group_…
Patches item #3596868, was opened at 2012-12-17 03:11
Message generated for change (Comment added) made by eighthave
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3596868&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-dev
Group: None
>Status: Pending
>Resolution: Rejected
Priority: 5
Private: No
Submitted By: IOhannes m zmölnig (zmoelnig)
Assigned to: Miller Puckette (millerpuckette)
Summary: use HAVE_STDINT_H for including stdint.h
Initial Comment:
this is a follow-up for commit:5459876b16 that includes stdint.h.
the patch uses HAVE_STDINT_H to include the header, and only uses hand-crafted types as fallbacks (i envision the time when MSCV will be C99 compatible!)
----------------------------------------------------------------------
>Comment By: Hans-Christoph Steiner (eighthave)
Date: 2012-12-17 12:04
Message:
I don't have a strong opinion on this, but I agree with Miller.
----------------------------------------------------------------------
Comment By: Miller Puckette (millerpuckette)
Date: 2012-12-17 11:58
Message:
I think it's better kept 'MSC_VER" - I don't believe any other compile
chain anywhere
will fail to supply STDINT, and leaving it MSC_VER marks it clearly as
crufty BS that should
get cleaned out someday.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3596868&group_…
Patches item #3413809, was opened at 2011-09-25 11:06
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3413809&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: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Hans-Christoph Steiner (eighthave)
Assigned to: Miller Puckette (millerpuckette)
Summary: use Ctrl-< and Ctrl-> to cycle thru open windows
Initial Comment:
This patch allows you to use Ctrl/Cmd < and > to cycle through all of the open windows in different directions.
----------------------------------------------------------------------
>Comment By: Miller Puckette (millerpuckette)
Date: 2012-12-17 12:02
Message:
OK - but man, is it bizarre how ctrl-< actes in XFCE - I'm guessing this is
XFCE being wierd
and not our fault. Anyhow, I've applied this for 0.44
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2012-12-15 17:36
Message:
yes, that's what is intended. I think each WM will have different ideas of
how this should work, like for people who have focus changing based on
mouse presence not clicking. But I think this is the most common paradigm.
Others can be implemented in plugins
----------------------------------------------------------------------
Comment By: Miller Puckette (millerpuckette)
Date: 2012-12-15 11:02
Message:
I tried this and got funny behavior - ctrl < focussed the previous window
OK, but also sent it to the back of
the window stacking order.. Can this be what was intended???
----------------------------------------------------------------------
Comment By: max (maxn)
Date: 2011-09-26 05:01
Message:
That's the standard on german keyboard layouts for cycling through the
windows of the active application on OS X. I'd very much apprechiate if
this would work in Pd. That said, i think it might actually be a bug in
Tcl/Tk that it isn't working right now.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3413809&group_…
Patches item #3596868, was opened at 2012-12-17 03:11
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3596868&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-dev
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: IOhannes m zmölnig (zmoelnig)
Assigned to: Miller Puckette (millerpuckette)
Summary: use HAVE_STDINT_H for including stdint.h
Initial Comment:
this is a follow-up for commit:5459876b16 that includes stdint.h.
the patch uses HAVE_STDINT_H to include the header, and only uses hand-crafted types as fallbacks (i envision the time when MSCV will be C99 compatible!)
----------------------------------------------------------------------
>Comment By: Miller Puckette (millerpuckette)
Date: 2012-12-17 11:58
Message:
I think it's better kept 'MSC_VER" - I don't believe any other compile
chain anywhere
will fail to supply STDINT, and leaving it MSC_VER marks it clearly as
crufty BS that should
get cleaned out someday.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3596868&group_…