add Tracks specific handling of MMC Record Strobe, since Tracks has no concept of punch

properly track, update, maintain and set auto return state

Conflicts:
gtk2_ardour/ardour_ui_options.cc

make sure we allocate large enough buffers when doing a non-butler context disk buffer refill.

The size of the buffer now needs to reflect that we calculate read
refills in bytes, and if we are not using 32 bit float sample format
on disk, that can translate into > 1M samples.

remove Tracks runtime conditional, incorrectly added during manual merging

correctly track range selection trim and drag creation w.r.t playback priority

make editor_drag.h fully self-contained as a header file (it was missing more than a dozen necessary fwd decls etc)

Conflicts:
gtk2_ardour/editor_drag.h

start work on auto return target dropdown rather than button

Conflicts:
gtk2_ardour/ardour_ui.cc

change ArdourDropdown API to allow other types of menu elements

follow various events related to playhead priority (loop changes, parameter changes); remove debug output

remove debug output

fix copy-n-paste errors that used wrong buttons for playhead priorities

fix copy-n-paste errors that used incorrect enum values for playhead priorities

restore use of vertical margin on ardour knob popup tooltip/window

API to set tooltip y-margin
Conflicts:
libs/gtkmm2ext/gtkmm2ext/persistent_tooltip.h
libs/gtkmm2ext/persistent_tooltip.cc

give different Tracks + Ardour/Mixbus playhead priority functionality, and add missing set_track_loop() call for playhead priority

correct placement of buffering adjustments w.r.t Butler thread creation

[Summary] Moved session buffers adjustment during butler thread set up to the place we are ready to handle them.

[Summary] HOT FIX: Fixed crash which happens on an attempt to load a session with audio after preferences removal, when buffer preset is set to Medium.
[Details] It happened because ARDOUR::Butler::thread_work() was called before Buttler::map_parameters() method was called, which sets up the correct buffering parameters (set_buffering_parameters () call) according to chosen preset.
I've added this calls into Butler::start_thread () to make sure everything is set up and ready for use for Butler.

This is a hot fix, because it requires the review from Paul Davis, and probably more gentle solution. But it's possible this commit may become the main solution itself.

try to correctly set up build of AVX code for windows and linux, using compiler flags dictionary.

Adds use of 'pic' compiler flags dictionary item

store compiler flags in conf/bld context object so that we can use them selectively at build time

meter computation must use absolute value of all samples (ignoring sign bit)

add windows-only AVX functions

Made optimized function Prototypes more correct.

They shouldn't be dependant on ARDOUR types as long as they will work correctly with float
samples only and with 32 unsigned int as sample counter.

Corrected comments for x86_sse_avx_find_peaks() intrinsics built function

add new file with AVX functions.

Does not fully support Linux at this time

Added optimized AVX function for sample processing

Added AVX versions of existing 5 SSE functions. Added 6th AVX function to copy vectors which is 1.5 times faster then memcpy.
Data consistency and validness  is fully tested after processing with new AVX functions on aligned and non aligned buffers.

fix accidentally deleted return value

fix timing of Butler mapping config parameters to avoid crash.

Amazing that this didn't show up on Linux at all, or on the first run after build. Computers ... sigh.

Conflicts:
libs/ardour/butler.cc

two tweaks required to fix up previous commit

add preferences GUI elements to allow control of playhead priority items

initial version of playback priority design. No GUI control over options yet

Conflicts:
libs/ardour/ardour/session.h
libs/ardour/ardour/types.h
libs/ardour/enums.cc
libs/ardour/session_transport.cc
system_config

ensure that working buffers (for gain + mixing) are large enough.

Remove AudioDiskstream and AudioSource _working_buffers_size members, since they are no longer used

clean up cherry-pick of varifill tweaks from Waves Tracks

more tweaks for varifill model, and avoid filling playback buffers during session loading

Conflicts:
libs/ardour/diskstream.cc
libs/ardour/session.cc
libs/ardour/session_state.cc

Conflicts:
libs/ardour/session.cc

add initial support for vari-fill

When refilling playback buffer, try to fill it completely, or at least
using the next-lowest power-of-2 as the amount to read. When locating,
where we use do_refill_with_alloc(), only partially fill the buffer.

Work not yet finished, but possibly promising.

introduce the idea of buffering presets, along with 3 possible settings (plus custom).

Actual numbers for the parameters are still to be determined/verified, and probably subject to
some platform specificity

fix logic of previous commit for transport-stop during locate

Do not disable record (performing transport stop) during locate request when MTC source is activated

This is required for the case when we leave Tracks ready for record and then activate MTC source.

Add some libardour support for indicating MTC active status.

The general design here is wrong, because it should be more general and cover all possible sync sources.
But it does work, it is used in Tracks, and my attempt to do it correctly revealed the problem to be
an EXTREMELY difficult design issue (as in: two weeks of work on it did not really solve the
fundamental issues with slave design).

So, here it is for now, c/o Grygorii

Revert "[Summary] Adding missing initialization of the date &time formats. “Init trashes” caused a bit ruined logic of the export dialog."

This reverts commit c9a190b7470d29bc6528195323a907a8d619c1d4. Change was
already in place (and used correct initialization order as well).

[Summary] Adding missing initialization of the date &time formats. “Init trashes” caused a bit ruined logic of the export dialog.

[Summary] Do not restore routes connection using XML IO state for TracksLive, because this is redundant for TracksLive.
Moreover if the device is changed XML state of IO contains port names of  the old device which causes attempts to connect to non existent ports.
[Reviewed by] Paul Davis

Fixed f65026f0a1cbc64862a82109c28f5d493fc45f2e merged commit related to waves backend
[Reviewed] PDavis

[Summary] Just adding comment for better code-maintaining.

superior fix for limits on track name length

[Summary] Fixed bug: Dragging Region with copying it provoked a crash. The reason is: too short room for name generated for new region.

[Summary] Fixed race condition: "should_do_transport_work" must not be dropped to 0 outside, it's illegal. It may cause "should_do_transport_work" go below 0 and make butler to round for a very-very long time (hang and possible lock of other threads) before stop() call returns.

_butler->stop () is synchronous, so its safe to wait in this call for butler to finish it's work.

[Reviewed by] PDavis

when reversing regions, use the original source's SR, not the current session one

when creating new sources for a non-realtime Filter process, add option to use the same SR as the initial sources

after changing session SR, reset all write sources to use new rate

Tracks does not (want to) support destructive tracks

Tracks specific fix for SR reverting to previous value while creating new session

add 176.4kHz to available sample rates in export GUI

add 176.4kHz to export sample possibilities

more reliable check for session files, to cover *.ardour.bak cases, and exclude them

Tracks puts the master bus remote ID into a different "namespace" than Ardour

Tracks does not use a K20 meter for the master

[Summary] In Location::set_state - adding emitted FlagsChanged (if any flag changed). This is not a hot fix - it's for UNDO changed skipping state for SKIP markers. Paul has approved this change.

remove unnecessary and unused changes from grygorii

remove unused method

fix whitespace/indentation and remove references to ScopedConnectionLists that were not used even in Tracks

[Summary] Eliminated redundant marker update notification which lead to creation of huge amount of redundant session events

noop: indentation

add required header

make use of loop range as default export range being Tracks specific

[Summary] Per Igor's request: if auto play loop is set on, the main timespan should be the range loop location.

when loop range is set, if loop-is-mode and seamless looping, set loop for all tracks

fix inadvertent typo

Tracks uses -inf gain on newly created tracks

bring in more connection-related changes from Tracks

add engine_state_controller.cc to libardour sources when building tracks

add track auto-naming options from Tracks

fix crash caused by missing return; add explanatory comments

fix typo in Tracks build

include Tracks engine state controller header for Tracks build

add required profile header

change names for scene I/O port connection methods

avoid redundant IO port reconnections in Tracks runtime case

noop: more style guide fixes

noop: fix indentation to match style guide

add EngineStateController code from Tracks

Fixed issue with crash when invalid get_connections request is made on invalid port handle. (Grygorii)

add in more Tracks-related auto-(re)connect changes

add in more Tracks-related auto-(re)connect changes, all conditionalized by Profile

initial incorporation of Tracks' automatic reconnect features, based on current Tracks code

merge several of the trickiest Tracks- and platform-specific RC configuration variables

no-op: indentation and whitespace fixes

[Summary] Added export cleanup
[Reviewed by] Andriy Mishyn

indentation and formatting cleanup

[Summary] Fixed issue with export built on Mavericks. There was not check on empty sets in ExportFormatSpecification class.
C standard, section 6.5.6.8 says:
"...if the result points one past the last element of the array object, it shall not be used as the operand of a unary * operator that is evaluated..."
So GCC compiler and CLANG compiler (Mavericks) process this operation different way. GCC returns 0 on an attempt to dereference end iterator when CLANG returns a non 0 value.

[Summary] Fixed windows build

add channel_count() to audio source API

another signal related to record state added from Tracks.

This doesn't appear to have obviously distinct semantics from RecordStateChanged
but Tracks is using it and I can't verify instantly that it could be replaced
by RecordStateChanged.

distribute use of RouteAddedOrRemoved to appropriate places.

It might be wise to use this rather than GUIIdle but that needs further investigation

include emission of RouteAddedOrRemoved in Session::add_audio_track()

integrate Session::remove_routes() from Tracks and replace Session::remove_route() internal implementation.

The singular case replicated (and was the model for) the plural case.

re-do previous commit with less variables

Tracks calls tracks "Tracks"; Ardour uses type-names

Session::reconnect_midi_scene_ports() is a Tracks thing.

It requires the EngineStateController which is not merged at this time

add missing profile header