Mixbus likes hyperthreading on Mac

Update Mixbus Mon-section #ifdef

Special case Lua copy-construction of trackable instances

This fixes an crashing issue with ArdourUI.SelectionList a bug
introduced in 6dc3bdf252c and 35dcd46d7d.

Since removal of the special cases in 35dcd46d7d, when using
a C-pointer in a std::list<>,
  std::list<class*>::push_back(TypeListValue)
TypeListValues<>'s Head was expanded to  "class*& const"
implied by void ::push_back(const T& value);

This resulted in lifetime issues with a classes that derive
from sigc::trackable (e.g. Ardour's Selection).
The reference leaves scope and isn't duplicated when it is pushed back
to the std::list<>.

The script scripts/select_every_2nd_region.lua crashed because entries
in the SelectionList were no longer valid.

Previously (before 6dc3bdf252c) TypeListValues explicitly
copy-constructed the value to work around the lifetime issue.
This new solution bypasses the issue by directly using the c-pointer
without dereferencing it.

Revert another sample/frame confusion

NO-OP: whitespace

Skip Subgroup menu-options when not available (no MIDI subgroups)

Add API to render a reflection

This is to be used sparingly because the pattern is dynamically
created every time. Mainly for the benefit of some Mixbus
widgets -- compared to ArdourButton::convex_pattern

Ardour/Mixbus codebase unification - ui config

Add option to leave vertical gap below audio regions

This is a backport from Mixbus, to leave are 4 "open" pixels
at the bottom of the track so one can see the grid-lines between tracks.

This is useful with low region-transparency. grid-lines behind
opaque regions are not visible.

Cleanup: Simplify conditionals

Remove support for building against LV2 older than 1.2.0

LV2 1.2.0 was released in 2012, over 6 years ago now.

Clarify default template description

LV2: check required feature and options

When discovering LV2 plugins, verify if a plugin's required features
and required options can be be satisfied.

This replaces a previous blacklist with a whitelist of supported
features/options.

Disable auto-input by default for new installs

This prevents audible feedback on laptops when adding new tracks
with default settings.

Backport Mixbus delivery patch

Unify build-system customization (Ardour/Mixbus)

Fix session-export util (timecode: frames, not samples)

Backport Mixbus toolbutton layout

Update Lua Script to use new MusicSample API

NO-OP: fix another comment

Remove cruft: unused typedef

NO-OP: whitespace

NO-OP: fix comment

Replace some more "samples" with "frames" (rectangles)

Another round of whitespace fixes

Fix typo in 37aaa2d1

Fix Timecode fps

Clarify API name (I/O-selector frame)

Revert more "frame/sample" replacements

- Coverage concerns rectangles (frames, not sample)
- frame-handles are rectangles
- layout container boxes are frames
- inline display boxes are display_frame

NO-OP: whitespace, alignment

Remove hardcoded project-name from Actions

Remove cruft (commented code)

NO-OP: whitespace, indent

Yet another "Rect frame" (not a sample)

Frame as in "picture frame", not sample

NO-OP: more libardour whitespace tweaks

Undo yet more frame/sample replacements

NO-OP: whitespace, comments

Cleaup meter-tap: reduce nesting and remove code-dup

Fix OMFtool; sample_rate vs frame_rate

Remove duplicate patchfiles

The previous commit also unified the naming schama to prefer underscores.

Import MIDNAM collection from Harrison Mixbus

Kudos to Nik and Ben at Harrisonconsoles. You Rock!

NO-OP name change: CD-frames (not samples)

NO-OP: whitespace

Fix space-alignment, mostly due to "frame" -> "sample" changes.

Undo incorrect sample/frame replacements in clearlooks

Undo more incorrect sample/frame replacements

NO-OP: whitespace

Indicate automation-latch as "touch" on the CC121

Without a dedicated "Latch" control, this does at least consistently
indicate "Touch", rather than show some arbitrary LED state.

NO-OP: whitespace

NO-OP: whitespace & revert samples -> [stack]frames

NO-OP: indent

Add missing semicolon (Mixbus)

Revert some incorrect frames -> samples replacements

fix copypaste bug

Enable midi-to-audio bounce when processing w/synth

Prepare midi-export for midi-to-audio bouncing.

* shift event time into process cycle before calling processors
* reset note-trackers when exporting, don't allow any pending
  events into the queue

Fix bounce-process: skip all processors before disk-reader

Bounce process always uses speed 1.0

NO-OP/Optimize Bounce: Lookup data-type only once

This is to prepare for midi-to-audio bounce: allow to change the
data-type to be used as output.

Verify Freeze/Export/Bounce polarity invert buffers

When using bounce w/process, the initial bufferset can [wrongly] be
  buffers.set_count (track.max_processor_streams())
Since the polarity-processor is first in the chain, iterations
over all buffer that don't have a matching _current_gain[]
or _control[] leads to memory-corruption or segfault.

Cleanup "Clean-up", hide dialog before starting cleanup

Backport a Mixbus special-case for plugin-tags

Fix OSC crash when there's no master bus

Fix MIDI Aftertouch (import, record and play) in the backend

Downgrade LADSPA scan errors to warnings

There's nothing that a casual user can about failure to scan
LADSPA plugins. So just inform the user.
It's not an error to worry about.

Indicate automation-latch as "touch" on the FP1

The FP1 does not have a dedicated "Latch" control, but this does
at least consistently indicate that Touch, rather than show some
arbitrary LED state.

Expose Latch automation mode to MCUs

Clarify documentation

Remove a hack of using flags to emit a signal-change from the GUI

The GUI should not be calling `manager.PluginListChanged();`
It's up to the plugin-manager to emit relevant signals.

This fixes an issue when adding/removing plugin-favorites using the
mixer's sidebar or via remote-control/surfaces or script.

NO-OP: Whitespace

AutomationRangeDrag tweaks for stacked layer view

Use y-pos and height of given TAV *layer* where the drag was initiated.

Remove unused variable

cont'd AutomationRangeDrag update

Properly adding guard-points for all lanes, not just the first found
for each range.

cont'd AutomationRangeDrag updates -- safety commit

This is work in progress towards fixing stacked region's region-gain
and multiple discontinuous regions spanning multiple tracks.

AutomationRangeDrag::setup() still does not collect all AutomationLine
points for certain overlap scenarios. There's more to come...

Fix Mixbus special case

Fix action-name

NO-OP: remove unused variable

Add a missing namespace qualifier

(is needed now, following commit #94271ee862)

Fix route-templates with external sends

Don't allow to cut/copy inserts via Drag/Drop

It was already verboten via ctrl-x/c/v since copying inserts using
the same I/O makes no sense, nor does moving them.

Re-layout ctrl surface enable button: align left

Since the Preferences window can be rather wide, and the table expands,
a button on the left is preferred.

Implement multi-region AutomationRange drag/drop

Cleanup: Remove unnecessary using namespace declarations

Cleanup: Fix inconsistent TransformDialog code style

NO-OP: add comment for cross-reference

Remove extra quotes from meta-data

Arguments are passed as argp[] array to execve() and don't need to be
enclosed by quotes.

Credit where credit is due.

Add a note to self

Fix mingw compile (declare int64_t and int32_t)

Do not create automation when shifting (insert/remove time)

This fixes a bug when shift() creates automation for parameters that
can not have any automation (hidden parameters, Mixbus PRE).

The GUI (RTAV) aborts() when it finds an automation lane for
a hidden parameter.

This also cleans up shift() operations in general. Empty automation
lanes should be left alone, no guard-point at zero should be added.

NO-OP: whitespac

NO-OP: whitespace

Fix generic-midi controllable race-condition

Continued work after e9b36f2bea. Prefer a shared_ptr<>.

MIDIControllable::write_feedback() runs in realtime context, directly
from the main process-thread. Synchronizing weak-pointers and deletion
across threads does not work reliably. Retaining a shared_ptr<> for
controllables that are in use can solve this.

Clean up MonitorProcessorControls

As opposed to regular AutomationControls these PBD:::Controllables
are not SessionObjects and don't emit a signal when the session
goes away.

Clean out session-global controllables

This isn't strictly speaking needed, there are only a handful of
users (most notably generic-midi ctrl surface, and Selection)

Remove static Destroyed(*), prefer StatefulDestructable API

This also add a debug-dump method to show remaining registered
Controllables.

Remove chicken/egg d'tor

Session::Controllables is a shared_ptr<> list. As long
as the session exists the Controllables will be around. Destroyed(*)
can only be called after the session is destroyed and releases the
shared_ptr<>

NB. this code had a nice hack to construct a "shared_from_this"
workaround. For future reference:

struct null_deleter { void operator()(void const *) const {} };
boost::shared_ptr<Controllable>(c, null_deleter())

Re-add global lookup for generic-midi ctrl state

Re-add Controllable registry

To facilitate a central registry with weak/shared pointer lookup,
  enable_shared_from_this<ARDOUR::AutomationControl>
was migrated to
  enable_shared_from_this<PBD::Controllable>

The main (and only) user is generic-midi surface's state interface :(

Update GenericMidiControlProtocol to use shared/weak Controllable pointers

This fixes a race-condition when a controllable is deleted
while sending feedback to the device.

Previously there was a race-condition MIDIControllable::write_feedback()
triggered from rt-thread, processed in Surface-thread and deleting
a route or processor.

This is a first step, currently state-restore is not fully functional
session->controllable_by_id() does not cover all Controllables.