Tweak behavior of Play-With-Preroll action

Follow Edits => Follow Range
* "Follow Edits" button had several behaviors that confused users.
* "Follow Range" only has 2 behaviors:
** Click anywhere in Range mode (or Smart mode) to locate the playhead.
** When you select a Range, "Play" will play the selected range.

update mini-timeline to immediately display tempo-map changes

Show relevant preferences on sync-button right-click

cont'd work on mini-timeline:

* add context-menu to configure visible time-span
* prelight markers on mouse-over
* constrain active click area (mouse-up)
* change scroll-event granularity: multiple of configured span.

Session::_ac_thread_active should be used with atomics to create memory barriers, even though lock ordering is likely correct now

Session::auto_connect_thread_terminate() cannot use ::auto_connect_thread_wakeup() because that method is allowed to "fail".

Should fix hangs when closing a session with the ALSA backend (and perhaps others too).

update canvas test tool

Towards packing CairoWidgets on ArdourCanvas.

vertically center text in time rulers

NO-OP: whitespace

Update Marker display style: consistent height & color

Ruler labels should be normal weight.

Consolidate clock bg colors to use the same dark grey

Consolidate green clock colors to use the same green

Reinstate spacer between Punch In,Out.

Clean up Clock Tempo, TimeSignature labels

Have the computer count columns for us.

Shorten sync-button label/size

fix table layout typo

fix blank mini-timeline

consolidate mini-timeline code, latch expose

Simplify spacer code

Transport Layout: horizontal separator makes it look too busy. Commented out for now.

repack widgets, and add some spacer elements

Add clock-button tooltips

Tweak Tempo and Meter precision, and shorten the label to accomodate

Revert mini-timeline marker drawing to before 448f921d

consistent size of sync-button

show all parameters when adding a meter via the tempo dialog.

TempoMap - rename some methods.
gui_move_tempo   -> gui_set_tempo_position
gui_move_meter   -> gui_set_meter_position
gui_dilate_tempo -> gui_stretch_tempo

add frame parameter for Meter ctor and TempoMap::replace_meter(). allow for various failures.

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