Fix region-gain offset when separating ranges

Squelch error message when toggling master-bus visibility.

OSC: Add send/plugin page values to GUI

OSC: save and restore send/plugin page size.

Fix range-separate for musically-locked MIDI regions

Cloning a region retains the region-lock style.
Playlist partitioning set the region start-property only (audio-time).
If a MIDI region is locked to musical-time, Properties::start is ignored
and overwritten by Properties::start_beats.

Special-case VCA at 0 (-inf dB), force gain slaves to -inf dB

Merge event control lists when disconnecting a master-ctrl

Add infrastructure to merge ControlLists

OSC: Allow set_surface to set send and plugin page sizes.

Apply master-value to automation on disconnect.

Write inverse master automation.

 * The UI and ctrl-surface controls use and display the combined value,
including control-masters.

 * The Automation lane of a control is the raw value of the control
without masters.

When touching (or writing) automation, the control-master needs to be
factored out (or subtracted). e.g press+hold a control -> write inverse
master automation.

amend 11ba1854 -- locked curve-evaluation is not public

FP8: use proper API to detect automation-playback

The custom code didn't take touch + touching into account when
another surface was touching automation.

Fix another thinko in 9581cb26 + 02b087c5 (VCA gain automation)

This went unnoticed because: VCA gain automation was always applied
(regardless of automation state) but when it was not playing
master_ratio() factored it out again (per block).

Fix nasty duplicate XML node

Remember subgroup-bus

Allow group gain sharing + VCA again

OSC: select channel plugin support with paging.

Fix Monitor Group override

Fix relative grouped + VCA slaved gain

Consistent "cancel/close" button position on the left.

"Add and Close" is an odd one out and this button should really be
removed for consistency. Still it allows for a 2-click or
2-keyboard-shortcut very common action.

Fix a compiler warning (optimized builds)

Restore the state of the autoplay button in the import dialog

Make Import the default action in the Import Dialog

Double clicking on a file or pressing the Return key will now import it.

This means with autoplay enabled you can navigate a directory of audio files
using the up and down arrow keys and import the wanted files (with Return)
without having to touch the mouse.

Being able to use double click to import a file means you don't have to move
the mouse back and forth between the file browser and the import button.

You can still manually audition the file with space as before.

Remove unnecessary temporary variables in MasterRecord::set_state

XMLNode::get_property only modifies the argument if the property is found and
conversion is successful.

VCA/SlavableAutomationCtrl re-design:

* remember master-ctrl value on assignment & save with session
* Control/AutomationCtrl only stores ctrl's own value (w/o master)
* virtual AutomationControl::get_value () -> use SlavableAC method
* MasterRecord uses weak-ptr (fixes recursive ~Controllable() deadlock)

a-EQ: Tweak knob layout to be less confusing

Fix thinko in 9581cb26 - scratch-buffer can't be used recursively.

OSC: add option to send /reply instead of #reply (OSC1.0 compat)

Emit Session::Located only if _engine.transport_frame() is up to date

If _engine.transport_frame() is not up to date, we emit Located in
Session::backend_sync_callback() because that's when audible_frame() is up to
date. We don't want to emit it twice, because then, the playhead jumps back and
forth.

* mouse click in the ruler -> jump to requested location
* mouse release -> jump to old location (because audible_frame has to catch up)
* backend_sync_callback() called -> jump to new location

Make Session::backend_sync_callback() emit Session::Located

If we sync to jackd AudioEngine::transport_frame() is not yet updated when
Session emits Located. Then the playhead ends up in an obsolete
position. Therefore we emit Session::Located() also from within
Session::backend_sync_callback() as that is called when AudioEngine is done
with the relocation.

semantically more correct ordering of previous commit

linker-agnostic version of recent commit to get ControlProtocol selection state set correctly

Implement slaved boolean automation and update mute special-case

Restore VCA Automation state

amend a1b4f9b8ab - handle disconnecting from all masters

set first selected stripable for control protocols before they are instantiated

NO-OP: whitespace

Fix deletion of VCA with slaved controls.

The crashed previously because:
 A VCA is-a Automatable is-a Evoral::ControlSet

 After VCA's d'tor completes ~Automatable runs and emits signal to
 DropReferences of all master-controls.

 This in turn calls SlavableAutomationControl::master_going_away()
 for slaved parameters for the given master-control

 In ::master_going_away() the weak-pointer reference to the master's
 AutomationControl (owned by the destroyed VCA) is still valid.

 Execution is in the d'tor of Automatable which is-a ControlSet and
 the ContolSet keeps a reference to the Control and hence also to the
 AutomationControl which is-a Evoral::Control.

 So master_going_away() locks a boost::shared_ptr<ARDOUR::AutomationControl>
 which is actually the MuteControl owned by the VCA.
 It calls SlavableAutomationControl::remove_master() which
 in turn calls MuteControl::pre_remove_master() which uses the
 MuteMaster API to retrieve the value. The MuteMaster however is the
 VCA that has just been destroyed.

The solution is twofold:
 1) emit "drop_references" from the VCA d'tor itself,
    before the VCA is destroyed.

 2) disconnect a slaved control from the master's drop_references signal
    when un-assigning a master-control.

NoteCreateDrag - Fix incorrect note length due to use of region-relative position

Remove Automatable::value_as_string API from libardour

Keep Pannable::value_as_string() for now. That is another inconsistency
which needs cleaning up. GUI StereoPanner and MonoPanner print
the value as they see fit, the panner-plugin provided formatting
is not used.

Update GUI, drop Automatable::value_as_string() API

Prepare to eventually drop Automatable::value_as_string()

Probably due to historical reasons, there are two APIs to format a
control's value. In all both variants end up calling either
ARDOUR::value_as_string() or the Controllable's formatting function
(except for panners).

Prepare AutomationTimeAxisView for non-route (VCA) automation

a-EQ: Revert one of the previous changes

One of the previous changes was not a typo, revert it.

Fix incorrect positioning of tempo line subdivisions if first meter is non-zero

Commit cebefe6 assumed that frame 0 was the music origin.
Silly me.

Add a method for obtaining the frame position of beat/whatever zero

Fix crash when showing plugin-UI in safe-mode

TempoLines deletes its bfc on destruction

Tempo lines display subdivisions correctly over a tempo change

TempoMap::get_grid() supplies a list of beat positions, leaving the lines
to work out any subdivision positions.
This is fine, unless a tempo section falls in between beats.
Use a BeatsFramesConverter along with a quarter note position
(in the BBTPointsList) to make this easier.

Fix reversed logic in legacy tempo secton detection

Delete tempo lines when session goes away

Do not duplicate note id in copy constructor

This fixes selection undo after copy-dragging notes, but there
are probably other cases where duplicate note ids may cause problems.

Remove debugging from TempoTest::qnDistanceTestRamp ()

Fix uninitialised tempo section variable

Should fix 7390.

Fix rec-box w/count-in

Since 478f26b2ad, transport_rolling() is only true when actually rolling.
Count-in is a no-roll process (don't move playhead, no playhead UI
position interpolation 55b8b448).

But transport isn't exactly stopped either (preparing to roll), so
during count-in transport_stopped() == transport_rolling() == false.

Update LuaDialog example script to include File/Folder selection

Add a file-chooser LuaDialog Widget

cut buffer must not mess around with libardour selection

Fixes range mode selection/cut/copy that would previously clear track selection

a-EQ: DSP bugfix

Previously there were large spurious spikes in the signal when
the bandwidth parameter was adjusted on a pure sine tone
for the peaking circuits. This has been *greatly* reduced if not
eliminated by fixing a typo in two of the equations.

OSC: add optional paging to sends in select.

Fix a-EQ when parameter changes are very slow

If the parameters change too slowly the filter may never get updated. Indeed,
in spite of v_f0, v_g or v_bw being updated, set_params may never be called,
thus v_filter never updated.

add route dialog now has "Add" and "Add and Close" buttons.  "Add" keeps dialog open so user can add multiple templates.

Apply VCA master gain automation to Amp (Fader, Trim)

Add infrastructure for evaluating VCA automation curves

amend 67f9f6fd: no recursive ReaderLock

add const-ness: Evaluating a curve does not change it.

Note that the ControlList's lock and cache are already mutable.

Add a scratch buffer for automation.

Useful as temporary buffer: This allows a controllable to
get a master's automation-curve and combine it with its own
(gain, trim, send) automation buffer.

NO-OP: whitespace

Import Dialog is Window. Dialog Esc behavior needs to be emulated.

Remove "Close" button from import-dialog (prefer WM close button)

if actively recording, prevent interaction between mouse and audio clock widget(s)

OSC: Some surfaces may use float for ssid. Accept this too.

OSC: check controllable exists before using

Add Lua bindings for VCAs and VCAManager

Prepare for automation control master

Basic infrastructure to allow VCA automation and Trim automation:
look up events during automation playback.

Credit where credit is due

NO-OP: whitespace

duplicate routes start off unsoloed to avoid issues related to upstream / downstream buses

when creating a new session after working on one, the "Cancel" button should be a quit button.

This avoids ending up with a null Session pointer in the GUI (#7369)

Allow export presets for 176.4KHz -- "Ardour6: music for bats"

Update Russian translation

Use string concatenation and PBD::to_string in EventTypeMap::to_symbol()

Avoid using PBD::string_compose for serialization as correct behaviour depends
on the setting of the global C++ locale.

Remove normalization of property names from libpbd xml code

It may have been OK to add this code temporarily at this low level, but as it
was introduced in 2008, this change is made with the assumption that any
Sessions that were affected have been re-saved since then with corrected
property names and that "normalization" is no longer necessary.

Timing results before changes:

XMLTest::testPerfMediumXMLDocumentTiming
   Create : Count: 10 Min: 30375 Max: 48253 Total: 431727 Avg: 43172 (43 msecs)
   Write : Count: 10 Min: 42553 Max: 49163 Total: 453353 Avg: 45335 (45 msecs)
   Read : Count: 10 Min: 70307 Max: 75987 Total: 734923 Avg: 73492 (73 msecs)
XMLTest::testPerfLargeXMLDocumentTiming
   Create : Count: 10 Min: 154486 Max: 307856 Total: 2678989 Avg: 267898 (267 msecs)
   Write : Count: 10 Min: 304273 Max: 343274 Total: 3169158 Avg: 316915 (316 msecs)
   Read : Count: 10 Min: 496920 Max: 541394 Total: 5260410 Avg: 526041 (526 msecs)

Timing results after changes:

XMLTest::testPerfMediumXMLDocumentTiming
   Create : Count: 10 Min: 21437 Max: 39749 Total: 348622 Avg: 34862 (34 msecs)
   Write : Count: 10 Min: 42320 Max: 49989 Total: 446135 Avg: 44613 (44 msecs)
   Read : Count: 10 Min: 63252 Max: 68389 Total: 660841 Avg: 66084 (66 msecs)
XMLTest::testPerfLargeXMLDocumentTiming
   Create : Count: 10 Min: 113458 Max: 263225 Total: 2215595 Avg: 221559 (221 msecs)
   Write : Count: 10 Min: 305439 Max: 339986 Total: 3150697 Avg: 315069 (315 msecs)
   Read : Count: 10 Min: 447560 Max: 488121 Total: 4698903 Avg: 469890 (469 msecs)

Use correct property name in generic MIDI surface

Fixes restoring the feedback interval property, but as the mutator/setter
method doesn't seem to be used this change should have no affect.

Use correct names when setting MidiTrack properties

These were changed in the XMLProperty ctor and they now match the names used in
MidiTrack::set_state()

Remove attempt to retrieve invalid Meterbridge properties

The property names with underscores are "normalized" to replace the underscores
with hyphens in the XMLProperty constructor from within Meterbridge::get_state
and so have always been written with hyphens.

Use correct property names in Meterbridge::get_state()

These property names have been "normalized"/changed to use a hyphen in the
XMLProperty ctor since 2008. That is before the Meterbridge existed so there
should be no files existing that contain these properties with underscores.

Use correct property names for Editor transport controllable state

These are normalized in the PBD::XMLProperty ctor, which is why
Editor::get_transport_controllable_state() uses the hyphenated names to
retrieve the properties and will not result in any state format change.

Always create a deep copy of MIDI regions when copying playlists.

Pretty print control-outputs in generic-UI

Implement basic AU parameter-printing

(don't leave buf uninitialized)

Include ParameterDescriptor in read-only control output

OSC: fix return value for successful touch

OSC: Added lpf and hpf controls for freq,enable and slope

OSC add feedback for extra comp and eq controls

OSC - make /strip/list accept a parameter like everything else

Accommodate recently removed source(s) in our MSVC project (gtk2_ardour)

Remove old jack_utils.h header that was moved to the JACK backend

Also remove a related test that has not been part of the unit tests since the
move (years).

Sort tempo map after legacy sections have been assigned a pulse