Disable nwworkingdir workaround in gui_canvas_saveas for nw.js 0.46+, fixes the first bug reported in #669.

Additional optimizations to avoid unnecessary runtime string comparisons for the purpose of OS detection.

Fixed inconsistent window sizes across Linux and OSX (this was a bug), and Linux/OSX and Windows (this is an nw.js inconsistency linked to innerHeight and innerWidth misreporting their size for some reason on Win10).

Fixed the window resizing bug when saving a patch with optimal zoom, closing it, and then reopening it (see http://disis.music.vt.edu/pipermail/l2ork-dev/2020-August/002901.html). Please note that this will need to be reverted once we migrate to nw.js 0.46+ (as per provided comment inside the canvas_check_geometry. Also improved the patch scrollbars to be flush against the window edges.

Instead, we just test for object creation.

This currently misses cases where abstractions can't create their dependencies
from other abstraction libs, plus some other cases. But as long as we aren't
testing for that, we don't need the noise of the error messages.

In my test, this reduced log size from 4.2MB to 3.2MB.

Example of the new output (taken from Gem compilation):

      CXX      libplugins_la-record.lo
    record.cpp:58:3: warning: #warning better handling of duplicate codecs [-Wcpp]
     # warning better handling of duplicate codecs
       ^~~~~~~
    record.cpp:87:3: warning: #warning isThreadable [-Wcpp]
     # warning isThreadable
       ^~~~~~~
    record.cpp:257:3: warning: #warning enumerateProperties [-Wcpp]
     # warning enumerateProperties
       ^~~~~~~
      CXX      libplugins_la-video.lo
      CXX      libplugins_la-videoBase.lo
      CXXLD    libplugins.la

Not all components honour the V=0 flag -- perhaps they don't use
Automake, or use a very old version. However, many do.

See also: https://www.gnu.org/software/automake/manual/html_node/Automake-Silent-Rules.html

See also: jwilkes/purr-data#666

Jean-Michaël Celerier authored 4 years ago and Albert Gräf committed 4 years ago

Else this does not seem to build with MSVC 2019

The GitLab CI commands are now separated from the job definitions, using
YAML anchors.

The build and test commands should now show as separate, collapsible
sections in the job output (as documented at
https://docs.gitlab.com/ee/ci/pipelines/#custom-collapsible-sections).

The Debian image preparation is moved to before_script to separate it
from the build and test commands. Note that this adds an implicit 5 minute
time limit on this part of the script (see
https://gitlab.com/gitlab-org/gitlab-runner/-/issues/2716).

The commands are also now written once and copied using YAML anchors
(as documented at
https://docs.gitlab.com/ee/ci/yaml/README.html#yaml-anchors-for-before_script-and-after_script).

Addressed the issue where optimal zoom did not update the scrollable area and reenabled the zoom of the scrollbars when zooming (this scrollbar zoom size may require further reflection to decide the best path forward).

comment the line that defines an extra inlet for the 'inlet~' object

show error message in the case its a inlet(~) object but argument 'fwd' is not specified. Third approach

fix for case fwd message is not properly built, pd could crash

add zcheckgetfn method. given a selector and a list of param types, checks if the object has that method and if the params coincide

switch to mess3 method for calling fwd callback and add proper argument verification,  pack conditions to call inlet_fwd inside it, selector 'fwd' can be sent again through inlet objects

create wrappers of symbol and blob for inlet_fwd

create wrappers of pointer for inlet_fwd