Patches item #1285113, was opened at 2005-09-08 09:56
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1285113&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: documentation
Status: Open
Resolution: None
Priority: 5
Submitted By: Piotr Majdak (petibub)
Assigned to: Miller Puckette (millerpuckette)
Summary: [print] cuts output, include this in help patch
Initial Comment:
[print] cuts the output if the message is a list with
two or more symbols. The second and following symbols
are printed up to the length of 78 characters and an
asterisk is added.
This behaviour should be mentioned in the help patch
print-help.pd, see the attachment as an example.
----------------------------------------------------------------------
>Comment By: Miller Puckette (millerpuckette)
Date: 2005-12-31 13:02
Message:
Logged In: YES
user_id=313747
took for 0.40.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1285113&group_…
Patches item #1353230, was opened at 2005-11-10 07:40
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1353230&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: wishlist
Status: Open
Resolution: None
Priority: 5
Submitted By: Thomas Grill (xovo)
Assigned to: Nobody/Anonymous (nobody)
Summary: class loader hook
Initial Comment:
Hi all,
i'd like to propose a loader hook functionality, so
that external logic can be used if the loading of a PD
external binary object has failed.
This opens up two possibilites:
1) it allows to use external objects written in
languages other than C (above all scripting languages)
which cannot provide an exposed class_setup function to
be used in the same way as traditional C-coded externals.
2) it allows to create external object functionality
more dynamically, since the exact name need not be
known (and fixed into binary) beforehand.
The attached patch is for s_loader.c. The sys_loader
function would still have to be exposed in m_pd.h
gr~~~
----------------------------------------------------------------------
>Comment By: Miller Puckette (millerpuckette)
Date: 2005-12-31 12:54
Message:
Logged In: YES
user_id=313747
Had to fool with this and the "altname" patch to get them to
work together. Should be working now (but I have no non-C
externs to test this on...)
----------------------------------------------------------------------
Comment By: Thomas Grill (xovo)
Date: 2005-11-23 03:20
Message:
Logged In: YES
user_id=350252
small usage example:
(a "cil" external has to be loaded in PD to invoke the
cil_setup function)
static int loader(char *dirname,char *classname)
{
// TODO: check the classname, if it's in our scope
// return 0 if it isn't
// here we could load some binaries, like in
sys_load_lib_internal
// or we could just create new pd classes, like in
typical xxx_setup functions
// ok, class has been loaded
return 1;
}
void cil_setup()
{
// install our class loader
::sys_loader(loader);
}
----------------------------------------------------------------------
Comment By: Thomas Grill (xovo)
Date: 2005-11-23 03:12
Message:
Logged In: YES
user_id=350252
here's a "diff -uw" patch from the pd root directory
involving both m_pd.h and s_loader.c
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1353230&group_…
Patches item #1358937, was opened at 2005-11-17 04:52
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1358937&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: wishlist
Status: Open
Resolution: None
Priority: 5
Submitted By: IOhannes m zmölnig (zmoelnig)
Assigned to: Miller Puckette (millerpuckette)
Summary: class loader for non alpha-numeric classnames
Initial Comment:
object-names containing non-alphanumerics like [||~]
can (sometimes) be not represented on filesystems (e.g.
"|" is a forbidden character) and more often they
cannot be used to construct a valid setup-function
("||~_setup" or "||_tilde_setup" are really bad).
the way the "~" is handled, is non-generic and thus
sub-optimal.
as a solution me and hcs proposed an encoding into
alphanumeric-values, using a hexadecimal representation
of all characters but [0-9A-Za-z_] (e.g. "+" is ascii
43 and is thus represented by "0x2b" (hex-value all
lowercase and prepended with "0x")
e.g. if we have a new class "mtx_||", pd first attempts
to find a file called "mtx_||.dll". if it succeeds, it
will try to call the "mtx_||_setup()" function.
if that fails we suggest to try and call a function
"setup_mtx_0x7c0x7c()" (the keyword setup is now at the
beginning of the function-name, in order to prevent the
names starting with numbers and in order to distinguish
between the normal setup-methods).
if no "mtx_||.dll" can be found, pd should then search
for a file "mtx_0x7c0x7c.dll" (guaranteed to be
representable on any filesystem); search this file for
the 2 setup-functions.
if all fails try to find "mtx_||.pd" and then
"mtx_0x7c0x7c.pd"
----------------------------------------------------------------------
>Comment By: Miller Puckette (millerpuckette)
Date: 2005-12-31 12:24
Message:
Logged In: YES
user_id=313747
It's ugly, but I think it's really needed anyway. It would
also be good someday to fix abstractions to do this too.
Anyway, I took it for upcoming version 0.40
----------------------------------------------------------------------
Comment By: IOhannes m zmölnig (zmoelnig)
Date: 2005-11-17 04:54
Message:
Logged In: YES
user_id=564396
attached is a patch against s_loader.c, s_class.c and
m_stuff.h that achieves this functionality
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1358937&group_…
Patches item #1312885, was opened at 2005-10-04 06:30
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1312885&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: 8
Submitted By: IOhannes m zmölnig (zmoelnig)
Assigned to: Miller Puckette (millerpuckette)
Summary: 2GB limit for soundfiles
Initial Comment:
using [readsf~] and [writesf~] you cannot read/write
files that are larger than 2GB;
while this limit seems big for soundfiles, it is easily
reached when doing multi-channel recordings (24
channels @ 44.1kHz take ~15 minutes to reach this limit)
to enable support for large files (>2GB), one has to
use 64bit variants of open and friends (open64(),...)
attached is a patch that enables the use of these
variants in d_soundfile.c when "_LARGEFILE64_SOURCE" is
defined.
the patch also changes the configure.in in order to
check whether defining "_LARGEFILE64_SOURCE" is a good
idea;
you can turn off largefile support via the
"--disable-largefile" option.
(btw, max/msp cannot write soundfiles > 2GB too)
----------------------------------------------------------------------
>Comment By: Miller Puckette (millerpuckette)
Date: 2005-12-31 12:04
Message:
Logged In: YES
user_id=313747
Took for 0.40... the whole "configure.in" file needs a rewrite
(which might require changing this bit) but I've never been
able to get the autoconf system to work for me.
----------------------------------------------------------------------
Comment By: IOhannes m zmölnig (zmoelnig)
Date: 2005-10-04 09:48
Message:
Logged In: YES
user_id=564396
probably a bit more explanation:
"_LARGEFILE64_SOURCE" is a special define that enables large
files support when including fcntl.h, unistd.h,...
the conditional defines found in d_soundfiler.h are just
there to use the new functions (like "open64") instead of
the normal ones (like "open")
independently from my solution i found the following in
http://www.cwp.mines.edu/pipermail/seisunix/2005/001082.html
<snip>
#ifdef _LARGEFILE64_SOURCE /* if the largefile source flag
is set */
# define open open64
# define creat creat64
# define lseek lseek64
# undef off_t
# ifndef _GNU_SOURCE
# define off_t long long
# else
# define off_t off64_t
# endif
# define fstat fstat64
# define stat stat64
# undef fseeko
# define fseeko fseeko64
# undef ftello
# define ftello ftello64
# define fopen fopen64
# define freopen freopen64
#endif
</snip>
another possibility would be to use the "__REDIRECT" define,
so fcntl.h would do the defines itself, but i am a bit
afraid of this.
and another question is, whether it would be a good idea, to
have these defines in "m_pd.h", so all files using that
would automatically have large-file support....
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1312885&group_…
Patches item #1368484, was opened at 2005-11-28 10:35
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1368484&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
Submitted By: Mathieu Bouchard (matju)
Assigned to: Nobody/Anonymous (nobody)
Summary: [poly] memory leak
Initial Comment:
fixed [poly] memory leak
removed unused vars
----------------------------------------------------------------------
>Comment By: Miller Puckette (millerpuckette)
Date: 2005-12-31 11:50
Message:
Logged In: YES
user_id=313747
Took for 0.40
----------------------------------------------------------------------
Comment By: Thomas Grill (xovo)
Date: 2005-11-30 05:46
Message:
Logged In: YES
user_id=350252
here's the missing patch:
Index: x_midi.c
===================================================================
RCS file: /cvsroot/pure-data/pd/src/x_midi.c,v
retrieving revision 1.2.8.3
diff -u -r1.2.8.3 x_midi.c
--- x_midi.c 30 Nov 2005 13:36:58 -0000 1.2.8.3
+++ x_midi.c 30 Nov 2005 13:44:24 -0000
@@ -1187,7 +1187,7 @@
static void poly_setup(void)
{
poly_class = class_new(gensym("poly"),
- (t_newmethod)poly_new, (t_method)poly_clear,
+ (t_newmethod)poly_new, (t_method)poly_free,
sizeof(t_poly), 0, A_DEFFLOAT, A_DEFFLOAT, 0);
class_addfloat(poly_class, poly_float);
class_addmethod(poly_class, (t_method)poly_stop,
gensym("stop"), 0);
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1368484&group_…
Patches item #1070226, was opened at 2004-11-20 17:13
Message generated for change (Comment added) made by eighthave
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1070226&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: Rejected
Priority: 5
Submitted By: carmen rocco (ix9)
Assigned to: Nobody/Anonymous (nobody)
Summary: win32/GCC fixes
Initial Comment:
* various #defines intended to reduce warnings in MSVC
create warnings in GCC
* catch stuff in s_entry and dllexport stuff in m_pd is
unnecessary/broken with GCC
* couldnt find TCL/wish.exe if in system PATH
* .pdrc wasnt being read on windows, where it is called
pd.ini and can be located in pd/ or ~/ (portions from
timblechmann & matju, includes previous "comments in rc
file" patch
----------------------------------------------------------------------
>Comment By: Hans-Christoph Steiner (eighthave)
Date: 2005-12-31 03:47
Message:
Logged In: YES
user_id=27104
this is now old, and some of the changes have been
incorporated in other patchs.
----------------------------------------------------------------------
Comment By: Miller Puckette (millerpuckette)
Date: 2005-05-18 00:41
Message:
Logged In: YES
user_id=313747
I'm not sure what to do with "pd.ini" since I'm trying to phase
out "pdrc" already. I could take the other changes, but
have taken
so many other patches today I'm afraid of dropping a patch
in halfway.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1070226&group_…
Patches item #1217414, was opened at 2005-06-09 02:27
Message generated for change (Comment added) made by eighthave
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1217414&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: 3
Submitted By: Hans-Christoph Steiner (eighthave)
Assigned to: Miller Puckette (millerpuckette)
Summary: change defines MACOSX to __APPLE__
Initial Comment:
gcc defines __APPLE__ automatically on Mac OS X so
there is no need for the -DMACOSX, #ifdef MACOSX, etc
This patch just changes every relevant instance of
MACOSX to __APPLE__ and removes -DMACOSX in configure.in.
This does not change the functionality at all, but
cleans up the code, using standard constructs, and
makes things consistent with __linux__ and __FreeBSD__.
----------------------------------------------------------------------
>Comment By: Hans-Christoph Steiner (eighthave)
Date: 2005-12-31 03:42
Message:
Logged In: YES
user_id=27104
FYI: I've been using this sed script as part of the
Pd-extended builds and so far no problems.
Also, the Win32 platform does have a common macro, its
_WIN32 (not WIN32 like I previously mentioned, tho WIN32
seems to work most of the time).
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2005-12-02 13:23
Message:
Logged In: YES
user_id=27104
Rather than a patch, its much easier to just run this sed
command:
sed -i.bak 's/MACOSX/__APPLE__/g' *.[ch]
This will only replace #ifdefs and comments, it won't affect
any code. These leaves -DMACOSX in the makefile for legacy
support.
On Windows, MSVC, Cygwin, and MinGW use WIN32 and the
automatic platform define. But the sed replacement isn't as
straightforward.
----------------------------------------------------------------------
Comment By: IOhannes m zmölnig (zmoelnig)
Date: 2005-09-30 09:00
Message:
Logged In: YES
user_id=564396
assigned to miller.
its oh so bad, that w32 doesn't really have such an
automatic define.
(mingw does define __WIN32__; probably it would be good to
add that one to the NT/MSW magic in m_pd.h)
----------------------------------------------------------------------
Comment By: Hans-Christoph Steiner (eighthave)
Date: 2005-06-09 14:10
Message:
Logged In: YES
user_id=27104
For Apple documentation on this topic, check out the
"Predefined Macros" section of this webpage:
http://developer.apple.com/technotes/tn2002/tn2071.html
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1217414&group_…
Patches item #1371497, was opened at 2005-12-02 01:59
Message generated for change (Settings changed) made by eighthave
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1371497&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: 7
Submitted By: Hans-Christoph Steiner (eighthave)
Assigned to: Miller Puckette (millerpuckette)
Summary: tiny fix on optional MacOS X/Jack
Initial Comment:
This patch has no effect unless Pd is compiled with
--enable-jack on Mac OS X. It will not affect any
other kind of build. This patch makes the Jack
dependency optional. Apple calls it "weak libraries",
aka "soft imports".
Here's the very simple patch:
--- configure.in 16 Aug 2005 04:06:28 -0000
1.15
+++ configure.in 2 Dec 2005 06:56:30 -0000
@@ -316,11 +316,11 @@
EXTERNTARGET=pd_darwin
if test x$jack == "xyes";
then
- LDFLAGS=$LDFLAGS" -framework Jack"
+ LDFLAGS=$LDFLAGS" -weak_framework Jack"
fi
if test x$jack == "xrun";
then
- LDFLAGS=$LDFLAGS" -framework Jack"
+ LDFLAGS=$LDFLAGS" -weak_framework Jack"
fi
fi
----------------------------------------------------------------------
Comment By: Miller Puckette (millerpuckette)
Date: 2005-12-31 00:36
Message:
Logged In: YES
user_id=313747
OK, will drop this in. I'm curious how Pd will avoid
crashing when asked to use jack when it isn't there - will
put it on my dolist to try to test this someday.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1371497&group_…
Patches item #1371467, was opened at 2005-12-01 21:26
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1371467&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: 7
Submitted By: Hans-Christoph Steiner (eighthave)
Assigned to: Miller Puckette (millerpuckette)
Summary: backwards compatible makefile.in cleanup
Initial Comment:
I cleaned up makefile.in so that it follows the GNU
standards for configure variables. Plus I added a
couple variables which allows the install target to
work properly across platforms.
This has the added benefit of making it easier to
compile Pd from the Pd-extended build system.
Tested on Pd-0.38.3 and 0.38.4 on all platforms and
applies cleanly to the HEAD of pd in CVS.
----------------------------------------------------------------------
>Comment By: Miller Puckette (millerpuckette)
Date: 2005-12-30 22:09
Message:
Logged In: YES
user_id=313747
Took for 0.40
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1371467&group_…
Patches item #1371478, was opened at 2005-12-01 21:55
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1371478&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
Submitted By: Hans-Christoph Steiner (eighthave)
Assigned to: Miller Puckette (millerpuckette)
Summary: Minor define fix for MinGW
Initial Comment:
MinGW doesn't use the #pragmas that MSVC does, and
throws a warning. Changing #ifdef NT to #ifdef
_MSC_VER will make sure that the #pragmas are only used
with compiling with MSVC and not MinGW or Cygwin.
This patch is part of the Pd-0.38.4-extended builds and
applies cleanly to .38.x and CVS HEAD in combination
with all of the patches that I am currently using (in
CVS at packages/patches).
----------------------------------------------------------------------
>Comment By: Miller Puckette (millerpuckette)
Date: 2005-12-30 21:41
Message:
Logged In: YES
user_id=313747
took for 0.40
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1371478&group_…