Change get_routes to get_routelist where appropriate.

get_stripables and get_routelist are now presentationinfo aware

Consolidate VST Plugin Info

This unifies common PluginManager VST code to construct *VSTPluginInfo
and also provides a dedicated is_instrument() method for VST plugins.

Force VST category to be "Instrument" for instruments.

This way it is picked up by the default PluginInfo::is_instrument()
method.

Honor effFlagsIsSynth and always assume those VSTs are instruments

This will incorrectly list some vocders or reaktor and the likes of
plugins with audio + midi input as Instruments IFF they also announce
effFlagsIsSynth flag.

NB. this is a first step only. we still need to override PluginInfo

GUI options to select FLAC as media format

Add support for recording to .flac

NO-OP: whitespace

Close files at rec-stop (re-open as needed)

This is in preparation to allow recording .flac files which are either
read or write not cannot be in read/write mode

Skip updating options for hidden file-dialog

This may also fix an edge-case issue where changing track selection
combined with invalid file selection may trigger a bad_file_message.

DSP-Load Window: subscribe to newly added routes

This fixes an issue when adding plugins with the route (e.g MIDI)
or showing the window before adding a route in the first place.

Save Lua UI scripts separately

Previously EditorAction and ExitorHook scripts were saved with
instant.xml. The were saved with each session and in the config
dir (for new sessions). This allowed inconsistent UI setups, especially
when loading old sessions that had no or different scripts.

Now Editor scripts (actions and hooks) are saved in a dedicated file,
session-independently. This goes along with ui_config in general
e.g. action-table-columns

The scripts are not saved with ui_config file for two reasons:
ui_config settings related to built-in  ui_config_vars.h,
and in the future there may be further indirection like "ui-rc-file".

Note: previously loaded editor scripts are lost with this change.

NO-OP: whitespace

Remove C++11'ism

While gnu-gcc had `std::map:at const` as non-standard extension
it is n/a for older gcc on OSX.

Surprisingly this const& p() const; performs a tad better as well, likely
due to different exception handling.

Perhaps it is also worth investigating boost::flat_map<> as replacement
for std::map<>, here. Our common case is just a single entry, so using
a std::vector emulated mapping might help.

Fix AudioUnit, missing const in cbef72b8a13

Prefer const references for Pin/Channel maps

Another micro-optmization shaving off some ten microseconds for every
plugin. Also copying maps isn't RT-safe.

This may however cause issue if const map references can change
while a plugin is running.

Plugin pin-mapping micro-optimization

Prefer std::map::at() over std::map::operator[] to allow const maps.
For debug builds, there should probably some try{} catch{} along with this.

Optimize Plugin connect & run API, use const maps

Optimize Parameter Control Loop

For plugins with 10000 Control Inputs, dynamic_pointer_cast<> overhead
is significant, here ~2msec (~0.2usec per cast, optimized build, i7-5600U,
2.60GHz)

rename TransportMasterManager::init() to ::set_default_configuration() to make its purpose more clear

fix crash during first-run configuration of the application, caused by using an incomplete TransportMasterManager

Allow strips to add or remove personal sends
in processor box context menu

A listen route has no aux sends

Work around tooltips in treeviews selection issue #7678

When tooltips are disabled, but a tooltip column is set for a treeview,
the treeview selection behaves inconsistently. It requires a 3rd click
after expanding any child-rows to select a child.

Create sidechain ports when created from template (#0007680)

When a route with a sidechain is created from a template or by route
duplication the number of ports of the sidechain are set according to the
XMLNode defining the sidechain. Then the names are set according to the name of
the newly created route.

Thus all the pin connections defined in the template are replicated in the
newly created route.

Allow ListenBus to be added by GUI

Prefer key-event dispatch over emulated events for VST plugins.

Add API to dispatch keyboard events to VST Plugins

Correct new track/bus/VCA dialog text

Consider size of port matrix column labels for size of row labels

The column labels are drawn in an angle towards the edge where the row labels
are. Therefore, if the column labels are much longer than the row labels it can
happen that they go beyond the left edge. In that case we have to add an extra
width to the row labels.

Update names of sidechains.

When a PluginInsert is created it does not have an owner right away. That's why
a we need to set the sidechains name once the owner is known, in order to
include owner's name into the name.

Furthermore we need to follow renames of the owner.

Put the sidechain ports into a dedicated tab in PortMatrix

Skip VST version check to detect MIDI inputs

Some plugins return 0 from effGetVstVersion and are hence not detected
as instruments. e.g. VeeSeeVSTRack (VCV). This may also explain why MIDI
port of some other VSTs are not exposed.

Most VST2.4 return 2400, so checking  [effGetVstVersion] >=2 (and not 2000)
was incorrect to begin with.

Fix key-forwarding to linuxVST UIs

Tested with VeeSeeVST

Allow to limit strip-inline controls (GUI optimization)

Preference Dropdown to limit Automatable Parameters

Add option to limit automatable control parmaters

VCVRack VST currently exposes 9999 automatable-control parmaters.
This slows down various GUI dropdown lists and dialogs.
(even worse: those parameters are not mapped to anything by default).

This change allows to limit automatable parameters to a reasonable number,
without loosing state of already automated parameters in existing sessions.

Remove duplicate call

Automatable::add_control() already does insert a given parameter
to the _can_automate_list list if it's automatable.

NO-OP: whitespace

OSC: Allow non-cue surfaces to create personal send and ListenBus

Add ListenBus to stripable types

New Route dialog: add fallback default selection

This fixes an issue with Mixbus. If the default "Create Audio Tracks.."
template/script is not available. Nothing is selected.

Selecting a type/template is required to update various other dialog
fields. eg. "Name" field.

Previously: Create VCA. OK+Close. Re-open dialog. Now the
implicit default was "Audio Track" but the Name-field still showed "VCA".

Safe-guard non-rt-safe sidechain operations

This is required by 7d8b93ad for showing error when actively recording.
It also prevents other UIs (OSC, Lua,..) to change re-configuring
sidechain ports/plugins while recording.

Consolidate PinDialog Session/recording checks

- _session cannot be NULL while the dialog is visible.
  This removes extra `if (_session)` tests.
- Operations now no longer fail silently, but show error messages.

Set session of PluginPinWidget created after ::set_session()

PluginPinWidgets initially receive their Session pointer by
PluginPinDialog::set_session(). When a RouteProcessorChange occurs all
PluginPinWidgets are recreated. Those newly created instances need a pointer to
the session.

Processor-box control context menu update

Don't show hidden controls inline in the mixer-strip

Only allow LV2 touch events for control inputs

This fixes a crash when a plugin sends a touch events for non-control ports.

Fix incorrect VST callback fall-though (effective NOOP)

Fix another compiler warning

Wimplicit-fallthrough fixes for tools/utils

Wimplicit-fallthrough fixes for gtk-ardour

Fix some Wimplicit-fallthrough

A "fall through" comment is most portable way to indicate
"no break, fallthru" cases.

 * __attribute__ ((fallthrough))  // is not portable
 * [[fallthrough]]; // is C++17

Fix some compiler warnings (exception catches)

Glib throws a const FileError exception

Fix Lua bindings (music-time, mapping)

Plugin analysis numerics

Show vertical marker-line (freq) and precise numeric data of the
transfer function (dB, deg).

Fix race-condition/deadlock, plugin-copy while rolling

lili93's session (#ardour) triggered this w/jackd 512fpp:
Drag/Drop copy a latent plugin from one track to another while rolling.
The GUI-thread as well as the auto-connect thread concurrently call
jack_recompute_total_latencies(). The auto-connect thread holds
a process lock while doing so. The GUI does not use any mutexes.
This randomly deadlocks in libjack.

backtrace: https://pastebin.com/6m3KGhWS

Fix typo in binding, amend 9bbbc7fbe

Add some missing link libs to MSVC zita-convolver

Fix an edge-case in Dummy backend's random generator

INT_MAX == 2^31 is not a valid random seed for this LCG.

Sort VST factory presets

This is more of a workaround than a fix. Simply by forcing the
URI to be sortable (VST-user presets are sorted at the end).

This eventually needs a deeper cleanup and API consolidation

PluginInfo::get_presets(bool) returns an ordered std::vector.
However this API is only used in a few cases where the plugin is
not instantiated. PluginInfo::get_presets(bool).

The problem is Plugin::get_presets(). The method returns
information that was previously cached by Plugin::find_presets() in
std::map<URI, ...>

Remaining changes needed to build zita-convolver with MSVC

Add MSVC support for building the 'zita-convolver' library (which is now needed by libardour)

Accommodate newly introduced source(s) in our MSVC project (fluidsynth)

Potential fix for lost VST MIDI events

Ardour's VST MIDI buffer API does not yet implement offsets and limits.
When a cycle is split, the same midi-buffer is used for all sub-divisions
leading to duplicate, offset, events.

MSVC compatibility for zita-convolver

Avoid use of 'std::to_string

Signed-off-by: Christopher Arndt <chris@chrisarndt.de>

Add plugin version and parameter count to VST user presets

Signed-off-by: Christopher Arndt <chris@chrisarndt.de>

Personal monitor now saves and restores

Example convolution plugin (Lua script, hardcoded IR)

Lua bindings for convolution DSP

Implement basic libardour convolution DSP

zita-convolver windows/mingw build fix

- semaphore is available,
- pthread_t is not a pointer type (but here it's not used anyway)

NO-OP: clang-format whitespace

zita-convolver: add namespace & visibility, remove global externs

Add upstream libzita-convolver 4.0.3

Allow Lua DSP processors to report latency

Add personal monitor sends

Fix copy/edit in e047b01aa256

Fix scalar initialization of shared_ptr<>.

Fluidsynth 2.0 uses anonymous unions (added in c11, or gnu99)

Follow fluidsynth's API changes introduced with 2.0.0

Updated patch from Andreas Müller <schnitzeltony@gmail.com>

Update Fluidsynth to 2.0.1

Noop: fix white space

Option to hide send if aux is hidden

Prevent excessive meter redraws for inactive meters at zero II

Same as be826f363522dbca96c8263d404c57c4fe9ff0e2

Signed-off-by: Andreas Müller <schnitzeltony@gmail.com>

correct comment

use IsPhysical port flag to decide what to show in the "system" tab of a patchbay/matrix

add new API for retrieving port flags from backend

Fix MMC range 0..127 (libardour uses and 0x7f)

when calling Session::engine_halted() after a user-driven engine stop, make sure Engine::running() will be false

LCXL: complete overhaul and Mixbus support

LCXL: Assign one callback function per Knob/Fader

OSC: personal monitor can hide the aux bus

OSC: personal monitor send hidden follows aux hidden

OSC: personal monitor should only send signal present on change

indentation fix

#include tidy-up

change region->row map of EditorRegions to use iterator, not RowReference

RowReference was responsible for 75% of the cost of building the region list, and incurs cost
at every later insert/delete/reorder. Since both ListStore and TreeStore have persistent iterators,
use them instead