rework locking (fa07233a, 112fba182)

For now: use a single lock, which should fix all related crashes.
optimize (with less contended partial locks) if this works.

GUI changes to allow user to specify how/if to use SMF track/instrument names during import

change ordering of options for multichannel import

temporary default value for ImportStatus.midi_track_name_source

use SMF track or instrument names in imported MIDI track names, if requested

add new member to ImportStatus object to allow specifying how to name new MIDI tracks

add a new enum to identify how SMF data should be used when naming imported MIDI tracks

add API to Evoral::SMF to retrieve all track/instrument names for use when importing

extract track and instrument names from SMF while loading

add braces and warning for unlikely error

fix incorrect test for textual meta events in libsmf

braces please

braces, please

free track name/instrument string memory when necessary

add track and instrument name members to smf_track_t

improving logic/flow for SMF type 1 files part 1:

check num tracks for type1 and offer multichannel option if appropriate

release locks before deleting record

amend prev commit

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