[PD-dev] string2any doesn't create in mrpeach/tcpsocketserver-help.pd

[Roman Haefeli]

On Wed, 2012-03-21 at 17:25 -0400, Martin Peach wrote:
> On 2012-03-21 17:20, Roman Haefeli wrote:
> > On Wed, 2012-03-21 at 22:13 +0100, Roman Haefeli wrote:
> >> On Wed, 2012-03-21 at 15:26 -0400, Hans-Christoph Steiner wrote:
> >>> Hey Martin,
> >>
> >> Hey Hans
> >>
> >> Sorry to chime in, but I wrote that particular abstraction.
> >>
> >>> I just ran the load_every_help.py test script on Pd-extended 0.43 and saw this:
> >>>
> >>> loading: /Applications/Pd-0.43.1-extended-20120310.app/Contents/Resources/extra/mrpeach/tcpsocketserver-help.pd
> >>> declare: declare: unknown declaration
> >>> declare: declare: unknown declaration
> >>> tcpserver listening on port 2468
> >>> verbose(3): tcpclient: (GPL) 20111103 Martin Peach, compiled for pd-0.43 on Mar 10 2012 04:07:08
> >>>   any2string 0 -1
> >>> error: ... couldn't create
> >>>   string2any 0 59
> >>> error: ... couldn't create
> >>> tcpserver listening on port 11001
> >>>   string2any 0 59
> >>> error: ... couldn't create
> >>> tcpserver listening on port 11002
> >>> slipdec: maximum packet length is 65536
> >>> slipdec: maximum packet length is 65536
> >>> slipenc: maximum packet length is 65536
> >>> slipenc: maximum packet length is 65536
> >>>
> >>
> >> Should be fixed with r16087.
> >> Thanks for checking!
> >
> > It seems Martin preferred [import] over [declare] and fixed it as well.
> >
> 
> Yes, [declare] didn't work for me.

[declare -stdpath extra/moocow] from my commit just a few minutes ago
did work. The problem was not [declare], but a wrong declaration. Back
when the help patch was written, moocows stuff was automatically loaded
in Pd-extended and I only added the  [declare -stdlib extra/pdstring] to
make the help-patch work on Pd-vanilla installations. Now, since
pdstring stuff is not automatically loaded anymore, the first mentioned
[declare] was necessary.

In Pd-extended, one  can also use [import moocow] which does exactly the
same, but might break in Pd-vanilla installations where [import] is not
installed.

Roman