mutex 'er up

Some overzealous locking to track down RequestObject related crashes.

bc0fa4d689a4 wrongly locked the current event loop's
request_invalidation_lock instead of the invalidation's list lock.

Also Abstract UI is able to delete requests concurrently with with
EventLoop invalidation.
e.g. PortManager::PortRegisteredOrUnregistered  and GlobalPortMatrixWindow
so the lock needs to be exposed.

If this solves various issues, mutexes should to be consolidated
(request_buffer_map_lock + request_invalidation_lock) and be chosen
such that there is as little contention as possible.

test diff for patch change performance when caching colours.

when dragging or copying multiple regions, respect position lock style.

- music-locked regions are moved by drag beat distance
  (will conform to tempo changes over the drag distance)
  audio-locked regions are moved by frame distance.

changes to adapt to modified version of GTK/Quartz which uses only MOD2 for Command, everywhere.

The default version of GTK2/Quartz uses MOD2+META for keys and MOD2 for scroll, which is
basically insane

Further fixes to non-cxx11 enabled builds

Hopefully all that is necessary this time

Fix for non-cxx11 enabled builds

Add option to Zoom to Selection on double click

Currently implemented for Region and Range selections.

The new option is false/off by default to maintain existing behaviour. I'm not
sure it should require another option, perhaps Zoom to Selection should be the
default and accessing the region properties dialog can be via Modifier+double
click, but further changes can be made on user feedback etc.

Related: #7112

Change Zoom to Selection action (Z key) to zoom on both axes

Add Zoom to Selection (Horizontal) action to access previous behavior.

Remove Editor::temporal_zoom_region as it was duplicate code and broken for
both_axes

Should Resolve: #7112

Fix layered-record undo, include changes to existing region(s)

Fix possible crash when removing ports with jack1

Add Slavable::AssignmentChanged signal to notify when a slavable is assigned/unassigned to/from a master

UI tweak for the spacer at the bottom of the master bus.

only update MidiRegion length_beats in partial copy ctor if there is an offset.

only set the exact musical position of the primary region for move/copy drag.

only recalculate MidiRegion start_beats if there is an offset in the 'part-of' ctor.

- when drag-copying MIDI regions, a partial region ctor is used
  which can lead to _start_beats being off by a sample.
  this in turn leads to 7168 happening, but only if magnetic snap
  is used and the regions are copied as a group.
  this should fix 7168, but the magnetic snap part is not yet addressed.

Towards making tape-tracks work again..

Fix a potential deadlock/crash (here tape-track peak-file)

read_peaks_with_fpp() already holds _lock, build_peaks_from_scratch()
takes the _lock again.

Depending on glib[mm] and the threading lib it may either result in a
deadlock, or with EDEADLK in undefined behavior when a
non-recursive lock is released twice.

Fix potential deadlock on session-load

disallow placement of audio-locked tempi within a frame of any other.

sort tempo curves initially as well.

fix loop/crash when dragging an audio-locked tempo over a music-locked one while snapped to grid.

no-op - rename TempoMap::set_active_tempos -> TempoMap::set_active_tempi

clean up some odd logic.

amend previous commit

rename MetricSection movable -> initial, but of course initial is !movable..

Add support for MacVST hidden/favorites

Add a small tool to experiment with libcanvas

there is no need to do a full set_colors() in MidiGhostRegion ctor.

tempo curve fiddling.

go back to always interpolating framed curve points.

make TempoMap::dump slightly more readable.

fix incorrect audio-locked tempo position when setting bbt via tempo dialog.

fix flickering in tempo curve when markers are reordered.

constify TempoCurve::tempo().

fix bar renumbering for audio-locked metrum.

fix typo in 914224f

Revert "Pixel hunt cairo-packer (status-bar, selection-clock border)"

This reverts commit 6015481377bc7c96d7e09b263a18b285fe2fc282.

Pixel hunt cairo-packer (status-bar, selection-clock border)

Check for reserved i/o that are not routes, fixes #7171

CairoPacker draws outside of its widget allocation

This fixes fixes TimeInfoBox packing when the InfoBox position changes.

Add editor actions to set tempo and meter

Tweak transport icons

Black selection & punch clock bg

Fix video-monitor "sync-source" changes

Add support for Pango Markup to ArdourButton

Consistent ArdourButton sizing for elements with LED

Speed up recent-session display with lots of snapshots.

Given that a session can easily have >100 snapshots and one cannot change
the sample-rate anymore, it's mostly useless to parse them all.

Add LIBPBD_API to a function declaration

The new specialisation for ConfigVariable<float>::set_from_string() needs to be exportable (it gets used somehow by ARDOUR::SessionConfiguration).

If adding LIBPBD_API causes a problem for gcc, we could change it to LIBPBD_TEMPLATE_MEMBER_API

Add some doc/comments to the timecode example script

remove dups.

For building with MSVC, std::strtof() didn't get introduced until VS2013

(use std::strtod() in earlier versions)

I can't find a source file called MIDIXML.cpp (??)

Add missing enum bindings

Add timecode example script

Clarify "frames" (video, timecode) vs "samples" (audio)

add Lua bindings for Timecode conversion

braces, please

Speed up recent session display (for many large sessions)

- don't parse XML into XMLTree
- only read the file, extract relevant elements
- don't read session-template contents, only test file

a-fluidsynth process events even when no sf2 is loaded.

Keep track of program-changes, re-apply Bank/PGM once the
soundfont is loaded. fluidsynth itself keeps track of CCs.

amend 581c7b6 -Wparentheses

Save/Restore MIDI Automation Controls (current CC, PGM)

Further tweaks to EngineDialog z-axis stacking

When there's no main window (initial setup, no transient parent),
preset a normal window listed in the task-bar.

The duality the Engine Dialog being used as Ardour-WM managed non-modal
Window (Menu > A/M Setup) and modal Dialog (AudioEngineSetupRequired)
complicates this a bit.

Speed up unloading large sessions.

Set MidiPgmChangeAutomation control value

Add Lua bindings to inspect MidiTrack::MidiControl

Fix another crash at exit.

fix copy/paste typo in prev. commit

Work around silence-trim config 0dBFS (7b1f97bf)

Revert "Fix export silence threshold by using a constant value for now"

This reverts commit 6784923a0587e9549f129faa7c75d784028d11a1.

fix parsing "-inf" in config variables

The default for export-silence-threshold is -INFINITY, written
as "-inf" (by cfgtool) into system_config. Yet parsing the config using
a std::stringstream results in "0" (due to bugs in various libc++).

Don't assume patch-banks are populated.

Allow sparse CC lists.

a-fluid synth: label Reverb & Chorus sends.

Also globally enable Reverb + Chorus FX (so that sends work by default)

create_drum_tracks lua script by Phillip Smith

add_filters lua script by Phillip Smith

Allow to send Immediate Patch Changes

a-fluid synth: list all bank/programs

Turn a-fluidsynth into C++

rough-in a-fluid synth midnam support

Fix #6753

Fix "Illegal instruction" due to recursive acquisition/multi release on a RWLock

See #6753 notes

Fix crash when track creation failed draging region

Add more check when creating a new track from a region drag into editor's empty area.

Fix #6496

Include last sample in reverse test for nonsilence in end trim during export

Don't move the Range Selection after performing Multi Duplicate

This is necessary for a single Duplicate so you can keep duplicating but it
doesn't make sense to move it for Multi Duplicate.

Fix Multi Duplicate for a Range Selections

As mentioned in the previous commit

Rename Duplicate Range action to Duplicate and assign existing Alt+d key binding

This will work the same as the existing binding with a Region Selection and
will also duplicate ranges in range selection mode.

Also add multi-duplicate editor action and use that for the default
multi-duplicate action, which will work the same in Object Selection/Grab Mode
but also supports Multi Duplicate of Range Selections....well it would if
that functionality worked correctly.

The duplicate and duplicate-multi region actions still have to exist to be used
in the Region menu as they should only be sensitive if there is a Region
selection which would not be the case if the editor actions were used.

Resolves: #7153

Fix issues in export with trim enabled

A complete reimplementation of AudioGrapher::SilentTrimmer::process to support
trimming the beginning and end in the same processing block

Fix export with trim end enabled to actually trim silent frames

Only add silence to beginning or end of export data if data was written

Should resolve: #6412

Rename AudioGrapher::SilentTrimmer method to reflect behaviour

Unlike many of the other uses of the term "frame" this is actually returning
the index to the first sample in the interleaved frame that contains
non-silence.

Always signal writing to file is complete at end of the export process

With end trim enabled, the only case that would successfully export was if
there was at least some samples above the silence threshold in the last export
processing block.

The issue was that the EndOfInput flag was not being passed to
AudioGrapher::SndFileWriter::process which would then call sf_write_sync and
emit the FileWritten signal to start post processing. Fix that by always
passing the EndOfInput flag in the last export process cycle.

Related: #6412

Fix export silence threshold by using a constant value for now

All float values defined in the CONFIG_VARIABLE macro seem like they are cast
to at some stage before writing (another issue that needs addressing). The
default value for export-silence-threshold (-INFINITY) is converted to a value
of 0 and as a result nothing is exported with trim enabled.

Use the same fixed silence threshold as Mixbus until proper bounds checking and
GUI is in place.

Related: #6412

Add script to run audiographer tests

add another useful PluginInsert binding

Prevent endless recursion on Evoral::Event<>

Fixup 08fffef (consistent class name)

Add Lua bindings to manage Groups

fader-to-trim lua script by Phillip Smith

set AtomObject body ID to zero (as per spec when unused)

Fix theme names (dash is used to separate program-name)

Remove boost_system test kludge

Update evoral test suite

Fix configuration