mini-tl: subscribe to location add/remove signals

there is no curve for identical note_types_per_minute values.

Transport Layout:
* Remove some redundant containers and frames, for a cleaner look.
* Adjust transport button spacing and sizes to more closely match the editor toolbar.

Transport Layout: Alert buttons do not need to be red when inactive

Transport Layout: Attempts to standardize button sizes and padding.

Mini-timeline: Any click not on a marker should trigger a locate

Mini-timeline: Use timeline colors from config

should not really bind a shared_ptr<Port> to a sigc slot, so don't do that

move termination of auto connect thread till after control protocols are destroyed.

Otherwise, control protocol ports don't actually get deleted

Prepare configurable mini-timeline time-span

remove channel filter status and button from MIDI track header, access via context menu

fix gcc4 compilation (local type used in template)

Refine transport tool bar layout

Prepare mini-timeline widget for toolbar

remove cruft

Add UI to set global track layered mode

Hide/remove per-track record-mode in favor of global setting.

Move selection/punch clocks to editor Sidebar

Commence transport tool bar re-layout

Rework shuttle control display

Add API to query editor's playhead cursor position

initial attempt at importing SMF tempo maps during MIDI import.

Still with debug output

add copy constructor, operator= and a clean ::dump() method for TempoMap

add arguments (unusued) to control whether or not SMF tempo map is used during import

add arguments (unusued) to control whether or not SMF tempo map is used during import

tweak presentation of SMF tempo option

show tempo information for MIDI files in preview pane; start to offer choice about using tempo map when importing

"toll free bridging" between smf_tempo_t and Evoral::SMF::Tempo

expose SMF tempo information in Evoral::SMF

add function to get number of tempos found in SMF tempo map

unused but slightly tricky code to add track name/instrument to an SMF file when saved

add comments about limitations for Evoral::SMF and tracks

MIDI tracer actually changes the event-loop

fix generic plugins for toggle with 2 scalepoints (enum)

Signed-off-by: Robin Gareus <robin@gareus.org>

some further invalidation details:

Prevent double unref during when the EventLoop terminates:
deleting the ringbuffer deletes all requests, some of which may
contain stale invalidation

remove the buffer_map_lock, now that signals ref-count the IR.

store InvalidationRecord in a Connection object and ref/unref it as appropriate

add ::use_count() method to InvalidationRecord

be a bit less flickery when redrawing tempo/meter markers.

fix unused Outside filled framed curve for constant points outside the draw area.

revert c3ab67dd now that this misbehaviour is understood.

fix issue with disappearing tempo curve at constant tempi.

tempo before the initial section is always constant.

amend incomplete change in previous commit.

more work on patch change redisplay.

- fixes just-introduced undo crash.

fix thinko in TempoMap::framepos_plus_qn.

- affects region relative time conversion when
  initial meter is non-zero.

correct return syntax

std::list::erase() needs a non-const iterator.;

at least for some versions of gcc.

speed up patch change redisplay when using scroomer / adjusting height.

- zoom is still a bit slow.

and now with erase

C++98 compatible iterator erase

rework request invalidation

This kills 2 birds with 1 stone: Removes the necessity of locks
and makes call_slot() realtime safe (req->invalidation->requests list
push_back). On object destruction, the invalidation-record (IR) itself is
invalidated.

Invalidated IRs are pushed onto a trash-pool and deleted in the event-loop
of the invalidated object (GUI thread) once all requests that reference it
have been processed.

One last detail remains: PBD::signal connect should reference the IR
and disconnect unreference it. This will guarantee that signal emission
will not reference the IR while the pool trash is dropped.

Add a trash pool for invalidation requests.

While EventLoop::invalidate_request() does invalidate request in the
request-list. It does *not* invalidate requests in the
per-thread-request-ringbuffer(s).
The invalidation record cannot be deleted in EventLoop::invalidate_request
see 6b5891a78f.

NO-OP: re-indent

Atomically to invalidate request

Yet another slightly overkill approach, but it /may/ explain crashes.

Add some more invalidation debug messages.

The threading anecdotes - Episode 7

When do_request() destroys the receiver object, the receiver will
free the invalidation record. So the IR needs to be removed from the list
before executing the request.

Invalid read of size 8
   at: AbstractUI<Gtkmm2ext::UIRequest>::handle_ui_requests() (abstract_ui.cc:242)
   by: BaseUI::request_handler(Glib::IOCondition) (base_ui.cc:141)
   by: sigc::bound_mem_functor1<bool, BaseUI, Glib::IOCondition>::operator()(Glib::IOCondition const&) const (mem_fun.h:2066)
   by: sigc::adaptor_functor<sigc::bound_mem_functor1<bool, BaseUI, Glib::IOCondition> >::deduce_result_type<Glib::IOCondition const&, void, void, void, void, void, void>::type sigc::adaptor_functor<sigc::bound_mem_functor1<bool, BaseUI, Glib::IOCondition> >::operator()<Glib::IOCondition const&>(Glib::IOCondition const&) const (adaptor_trait.h:89)
   by: sigc::internal::slot_call1<sigc::bound_mem_functor1<bool, BaseUI, Glib::IOCondition>, bool, Glib::IOCondition>::call_it(sigc::internal::slot_rep*, Glib::IOCondition const&) (slot.h:148)
   by: sigc::slot1<bool, Glib::IOCondition>::operator()(Glib::IOCondition const&) const (slot.h:643)
   by: cross_thread_channel_call_receive_slot(_GIOChannel*, GIOCondition, void*) (crossthread.cc:49)
   by: g_main_context_dispatch (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.5000.2)
   by: ??? (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.5000.2)
   by: g_main_loop_run (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.5000.2)
   by: gtk_main (in /usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0.2400.31)
   by: Gtkmm2ext::UI::run(Receiver&) (gtk_ui.cc:286)
   by main (main.cc:408)
 Addrd1b8 is 24 bytes inside a block of size 48 free'd
   at: operator delete(void*) (vg_replace_malloc.c:576)
   by: PBD::EventLoop::invalidate_request(void*) (event_loop.cc:98)
   by: sigc::internal::trackable_callback_list::~trackable_callback_list() (in /usr/lib/x86_64-linux-gnu/libsigc-2.0.so.0.0.0)
   by: sigc::trackable::notify_callbacks() (in /usr/lib/x86_64-linux-gnu/libsigc-2.0.so.0.0.0)
   by: ProcessorEntry::LuaPluginDisplay::~LuaPluginDisplay() (processor_box.cc:1757)
   by: ProcessorEntry::LuaPluginDisplay::~LuaPluginDisplay() (processor_box.cc:1760)
   by: ProcessorEntry::~ProcessorEntry() (processor_box.cc:251)

skip dead threads (if the event remains, it'll be cleaned up below)

fix potential invalid lock

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