Massaged the routine a bit to bring it more in line with what vanilla
currently has, and backported some recent changes in rev. 1f383be3 and
rev. ff9e52c6 from vanilla which are supposed to fix some bugs in
sys_domicrosleep.

Albert Gräf authored Sep 01, 2020 and Albert Gräf committed Sep 09, 2020

This is useful, in particular, to print out pointer values with the
%zx specifier, since it prevents the annoying format warnings when
using %zx with t_int.

Albert Gräf authored Sep 04, 2020 and Albert Gräf committed Sep 09, 2020

This can be seen, e.g., when running regression_tests.pd gui-less in
Windows, as our CI script does. The easy fix is to just bypass the gui
finalization code in glob_quit() when running gui-less (sys_nogui).

We're not yet saving the hex colors as that will break in older patches.
But we do print out warning messages in this interim period and will
save the hex colors at a later date. Once that change is made, all the
interim versions will be able to read the new format

Added a "floatsize" method to [pdinfo]
Cleaned up the format specifier for [print] and gui_vmess
Got rid of unnecessary "l" in "%lg"

This makes it easier to point the gui dir at a local copy of the repo for
easier development. You can use this to make it easier to test out changes
to the GUI in purr-data/pd/nw.

Setting GUIDIR should even work if you haven't installed Purr Data to the
system. Just make sure the nw.js binary has been downloaded and expanded to
purr-data/pd/nw/nw. (This happens automatically when you build.)

This patch makes it possible to have multiple running instances of Purr Data,
and to leverage that interface from Pd-l2ork to get [pd~] working on all
platforms

Some difficulties required revisions to [pd~] in Pd Vanilla's code
* use quoted path args for Windows when starting the [pd~] subprocess
* replace sys_havegui() from Pd Vanilla's recent "threadening" with sys_nogui

The "-unique" flag requires the OSX app bundle to be able to start by calling
the binary in Pd-l2ork.app/Contents/Resources/app.nw/bin/pd-l2ork. This should
also make it possible to run Purr Data on OSX both by using an app bundle
(where the GUI starts the Pd process) and by installing and running from the
command line (where the Pd process starts the GUI)

The "--user-data-dir" flag sets the directory for various nw.js state used
by a running instance. To prevent races nw.js requires that each running
instance have its own directory that is not shared with any other instanc...

first draft for opening files from command line or graphical file manager in the current Purr Data instance

These bugs were discovered via static code-analysis
https://scan.coverity.com/projects/pure-data-pure-data
(The CID numbers are the "Coverity IDs" used to identify the various bugs)

- Don't free binbuf within glist_readfrombinbuf (CID:74871)
- it must be deleted by the caller
- Initialize sockaddr_in (CID:74861,74862,74863,74866)
- Initialize indelay/outdelay (CID:74864,74865)
- Initialize device-arrays (CID:74853,74854,74855,74856)
- Closing unneeded sockets (CID:74827)
- Initialize sockaddr_in (CID:74860}
- Close file-descriptor if the soundfile is unparsable (CID:74822,74823)
- Close file-handle in cleanup (CID:74816)
- Call va_end() when handling variadic args (CID:74872)
- Added missing break (CID:74800)
- Prevent NULL-dereferences (CID:74792,74793)
- Size checks when copying into string buffers (CID:74844,74845,74846,74847,74849)
- Freeing no-more used storage (CID:74832)
- Fixed code block (CID:74806)
- Initialize sockaddr_in (CID:74860)
- Initialize array-values to 0 (CID:74867)
- Free allocated memory (CID:74813,74830)
- Removed allocation of unused memory
- Use pd_free() to free templates (CID:74814)
- template_free() does not free the entire structure (it misses the pd_new() part)
- initialize variable (CID:77865)
- Prevent NULL-pointer dererencing (CID:77864)
- Free heap-allocate binbuf after use (CID:74833)

quick fix for backslashes in legacy tcl commands breaking the GUI parser-- just send "dummy" command arg

escape "\r" coming from Pd before evaluating

fix regression where Pd would eat the first character of a symbol if the symbol contained a double quote.

(I forgot to actually test the branch that handles strings with quotes in them)

allow this to work:
gui_start_vmess("foo", "");
gui_start_array();
gui_s("hello");
gui_end_array();
gui_end_vmess();

In other words, a gui message with a single array argument. There should rarely be a need for this (if at all), but it should still send a well-formed message to the GUI (which it wasn't before this bug fix)