fix mistaken "do not roll" conclusion in TransportFSM::compute_should_roll()

remove fragile union design

Absurd to be memory-conscious about a couple of member variables in an object that represents user input

fix broken-ness caused by bad initialization of a StopTransport event in TransportFSM

Remember implicit InlineControl visibility

Fix ISO compliance

Add timestamp to log -- #7877

This adds the time when a log message is displayed.
ARDOUR_UI::display_message() parses the prefix,
so the timestamp cannot be prefixed in the beforehand.

Still, UI::process_error_message() is called directly in
the same thread, so this makes no significant difference.

rename DoTheRightThing to RollIfAppropriate

Session::request_locate() takes a tri-valued second argument for "roll-after-locate"

This allows callers to defer logic about auto-play/current rolling state and more to TransportFSM where it
can be cnentralized and is less ambiguous

Fix MIDI capture alignment

This fixes Audio/MIDI alignment when recording normally.
Loop and/or Punch-in recording is still not aligned properly
(both audio and MIDI).

_accumulated_capture_offset is initialized once, accumulated
offsets (rec_nframes) are kept, even when ::check_record_status()
is called multiple times.

Zero capture-offset regardless of data-type

_accumulated_capture_offset is only used for MIDI,
however this is helpful when debugging.
Previously audio kept accumulating the offset indefinitely.

Make it possible to use reasonable-synth in production

This allows to disable the xmass easter-egg for those who
don't celebrate x-mas.

Fix another stuck splash screen

Don't allow dialogs to re-create the splash only to hide it.

Fix test breakage from a855119bdd

Change default preferences as discussed on IRC today

Remove mostly broken "live band" session-template script

This script originated in Mixbus and has not been kept
up-to date to work properly with Ardour

Replace some Gtk:MessageDialog with Ardour's variant

This fixes some issues with stuck splash screen early on.

Update comments & icon of rubberband example Lua script

add styling specifics for TreeViews so that checkboxes are properly visible

fix 03c4335c1e20311 to use the correct test for zero length

That commit used the dangerous Beats::operator== (int) comparison, which only campares the beat portion

Fix thinkos in cubasish theme

Recover caineville theme

Recover clear_gray theme

Add some example Lua scripts

remove debug output

prevent everything except the name column from changing selection in EditorRoutes

store some/most of the TreeViewColumn pointers for EditorRoutes in the parent object

Add Lua bindings to inspect the Tempo Map

NO-OP: whitespace

Fix Ardour VAMP plugins (FFT)

Bug was introduced in 8ed33f1bc714c1 symbol visibility setting
in CFLAGS, CXXFLAGS was overridden. This resulted in publicly exposed
and bound kiss_fft symbols in libqm-dsp.

At runtime those symbols were resolved using previously bound
symbols in libcodec  (see below) that uses a mismatching implementation
(ardour/qm-dsp uses -Dkiss_fft_scalar=double)

#0  0x00007fffea793d40 in kiss_fftr () at /usr/lib/x86_64-linux-gnu/libcodec2.so.0.8.1
#1  0x00007fffcf4516ab in FFTReal::D::forward(double const*, double*, double*) (this=0x5555571d73a0, ri=0x55555a734810, ro=0x55555a7262b0, io=0x55555a728590)
    at ../libs/qm-dsp/dsp/transforms/FFT.cpp:121
#2  0x00007fffcf4510fd in FFTReal::forward(double const*, double*, double*) (this=0x555559868190, ri=0x55555a734810, ro=0x55555a7262b0, io=0x55555a728590)
    at ../libs/qm-dsp/dsp/transforms/FFT.cpp:186

Recover Unastudia theme.

Recover cubasish theme

Recover Blueberry Milk theme

Sort palette colors by name only

Add more clues to port themes to new paradigm

Fix more default theme colors

Tweak the tooltip for PDC button

Fix C++11ism

Drop references held by any GUI Lua script after execution

Fix re-packing of toolbar meter

This fixes a gtk-warning when loading a session from a
running instance:
IA__gtk_table_attach: assertion 'child->parent == NULL' failed"

Add rubberband Lua bindings to process ardour regions

Processor list should use the same bg color as other lists

Add missing theme elements, and fix some thinkos

Apply some missing widget names

Fix preference pane to access for external-sync

Fix external timecode GUI display

Only show mis/matching TC if FPS has been detected and
timecode master is locked.

Fix thinko in eee01188 (engine pulse spacing)

Add test synth to debug capture alignment

fix drawing of zero-length notes

Note that the result is too narrow to be manipulated, though it can be moved

use "extends to numeric_limits<Beats>::max()" rather than "zero length" for nascent (incoming) notes

removal incorrect/unnecessary forward decl

use Note::end_time() == numeric_limits<Beats>::max() as the indicator that a note is in-process w.r.t. recording

We used to use length() == 0, but this clashes with actual zero-length notes

Fix AFL position

* update AFL position when preference changes
* "after post-fader processors (before pan)" is before
  the main-out (not at the end).
* Fix "immediately post-fader":
  The amp, when added was the last element. ++after_end then
  made the iterator point to .end()

This likely worked in the past when the monitor send was added
immediately after adding the fader/amp before any other processors.

Scale the port matrix with the GUI/font-scaling setting.

Clarify insert-time option

Stop engine for new session creation

This restores Ardour5 behavior and works around a missing "OK"
button in the engine-dialog.

Special case JACK, sample-rate cannot be changed for new sessions

Allow to configure sample-rate of new sessions

When creating a session from the Editor (after Session > Close,
or directly via Session > New) the engine-dialog needs to be displayed
to allow configuring the sample-rate.

This also consolidates scripted session setup: meta_session_setup()
is now called from build_session(), instead of all callers.

NOP: Trim whitespace and convert tabs to spaces

Update reset_mixer script to use new well-known controls

Remove unused variable

Latency debug script: include MIDI ports

Fix typo in Lua binding for compressor enable control

expand comment to include (more of?) the full story about uncombining and copies

expand the test used to decide if we need to make copies when uncombining a compound region

never remove entries from the CompoundAssociation map

add concept of an owner region (by ID) to PlaylistSource

fix bugs (thinkos?) that confused PlaylistSource's id() with its original()

NOOP: reindent

increase explicit refcnt for playlists when used by a PlaylistSource

Fix MIDI loop recording - Revert "Fix MIDI disk-writer flush"

This reverts commit 254f22e372ff1155e76bea2c9a6f9c6c85a94868.

Towards fixing MIDI capture alignment

When there is no overlap (Evoral::OverlapNone) of local transport
position and the record-range, MIDI data does not need to be
offset.

This matches audio recording: Only write to the capture ringbuffer
when there is an overlap.

(There is still some unknown, unresolved discrepancy remaining
to be tracked down)

Fix CaptureAlignment debug messsage

prevent MIDI note starts being drawn earlier than their region start

Revert to use an image surface for CairoWidgets

This partially reverts 2edbda252619 and is a follow up to 0b266a54f,
to fix performance issues with MS Windows graphics performance.

Mini-timeline, use explicit cairo-group to consolidate rendering

fix required offset when reading MIDI data near loop end/start

improved/new DEBUG_TRACE output

plugins should NOT resolve MIDI notes at loopend

Their data will come from (1) disk, in which case the DiskReader will do the resolve (2) live input
in which case the player/user will do the resolve

imrpove debug msg

fix serious but rare error in RTMidiBuffer's storage of MIDI events with size > 3

sizeof (T) is in units of bytes, not bits. Use C++ standard CHAR_BITS instead.

Fix samples_as_time_string formatting

Fix MIDI Port-latencies

Only audio ports have additional latency due to I/O re-sampling
for vari-speed playback/capture.. MIDI ports are not affected.

Remove debug output

Fix MIDI disk-writer flush

to_write must not exceed `total = _samples_pending_write`.
If the write succeeds (events spanning `to_write` samples are written)
to_write is atomically subtracted from `_samples_pending_write`.

Amend 8a5cbb8: fix variable-names and comments

Reenable the correct sort column and type when redisplaying regions

Dummy: Engine-Pulse audio/midi generators

This produces synchronous events on Audio and MIDI ports.
One rvent per second, exactly at every second since engine-start.

MIDI: C-4 Note-on/off (1 sample long)
Audio: +1/-1 transition:
       +1 in sync with Note-on,
       -1 in sync with Note-off

Proper forward declaration for Lua bindings (amend 2afef6e4bde)

Fix windows fallback for LV2 state:freePath feature

Fix Windows builds (Lua binding undefined reference)

remove the (unused) concept of a MIDI patchfile folder per session

move MidiPatchManager initialization into ARDOUR "global" scope, rather than per session

use glibmm threads for the midnam load thread, instead of raw pthreads (for portability)

remove unnecessary call to xmlKeepBlanksDefault()

move call to xmlKeepBlanksDefault(0) to before the creation of an XML parser context

the value is used by the parser context; the old code called it only after the *first* parser context
was created. therefore the first XMLTree::read() call has its behavior determined by libxml2's default
for this value, rather than by our explicit choice (do not treat whitespace as a note). All subequent
read() calls will use our value.

This variable inside libxml2 is actually per-thread, which just increases the stakes for calling
xmlKeepBlanksDefault() at the right time

Add Lua bindings for UIConfiguration variables

repeat baf0cdcbef1 but for BaseUI, thus covering all control protocols

fix handling of flags in the OSC input handler

Some builds of glib on macOS end up delivering IO_PRI when IO_IN is also set. This differs from our own build stack
version, but it isn't really an error, so we should handle it.

Allow public access to the editor-mixer-strip

This is currently used by Mixbus to sync visibility of switches on
the editor-mixer-strip with the main mixer.

Add a missing semicolon