fix unused but usable variable warning

fix const-cast warnings

remove const warning in SMF C code by changing function parameter argument qualifier

comment unused but potentially usable variables

comment unused but potentially usable variables

try using threaded waveviews in tracks also

Fixed incomplete merge for audioengine.cc
[To be reviewed by] Paul Davis

never use threaded waveview rendering for Tracks

comment out two verbose debugging outputs

clip level is adjusted only by region amplitude (which is audible), not by _amplitude_above_axis

attempt to correctly respond to changes in _amplitude_above_axis in ArdourCanvas::WaveView

increase debug bitset size to accomodate additional debug bits imported from Tracks

update waves' fader implementation, clean up whitespace, indentation, variable initialization and remove unused members

no-op: fix whitespace/indentation

Fixed use case when session is being replaced with new session with the same name.
We should check session state file (*.ardour) presence to determine if the session is new for TracksLive as TracksLive has a use case which gives user a possibility to replace existing session by clicking on it's state file when new session is being created.
[To be Reviewed by] Paul Davis

[Summary] This action is duplicated. To make TRX features work correctly this flag should be dropped later.
[To be Reviewed by] Paul Davis

Revert "remove unused variables". They are used on OS X.

This reverts commit 0fba19e9e6e6c519c89ce63149581b8f24b38c8c.

remove unused variables

fix whitespace/indentation and compiler warnings

remove compilation warning (-Wunused-but-set-variable)

new PBD::DEBUG API changes for Waves backend

Revert "replace two more instances of !! with implicit use of operator bool()"

This reverts commit e8eff9fb9c2fa6fc518404c2f9b731f8c28d3a23.

OS X compiler gets confused by this change.

replace two more instances of !! with implicit use of operator bool()

some more information output to log during save-as, for debugging

two required changes to default values for Tracks

Tracks doesn't know how to handle Engine (JACK) sync

fix API version SNAFU caused by re-copying EngineStateController out of Tracks

(re)implement Yevgeny's LTC branch changes in the context of ardour-merge-from-tracks (just the libs/* section)

fix indentation and style guide issues in new EngineStateController code

copy current EngineStateController from Tracks to import Yevgeny's LTC (re)connection related changes

fix previously cherry-picked commit to be Tracks-only, since Ardour/Mixbus are not affected

[Summary] Mac OS specific bug fix: Increased priority of the top menu bar events

[Details] g_idle_add () which was used for this purpose used G_PRIORITY_DEFAULT_IDLE>=200 which was too low for correct main menu requests processing, like showing Mixer or Big Clock Window.

The priority was increased to G_PRIORITY_HIGH_IDLE = 100.
According to GTK manual (https://developer.gnome.org/glib/stable/glib-The-Main-Event-Loop.html#G-PRIORITY-HIGH-IDLE:CAPS)
GTK+ uses G_PRIORITY_HIGH_IDLE + 10 = 110 for resizing operations, and G_PRIORITY_HIGH_IDLE + 20 = 120 for redrawing operations.

I assume that during record or playback on tracks there are too many redraw request which didn't give any chance for main menu request to be processed in time with old priority value.

This change must be considered as a hot fix so far.
The issue might be deeper. Will be investigated with Paul.

[Feature reviewed] AMishyn
[Reviewed by] PDavis, GZharun

Revert "experimentally remove some C++11 flags (likely to return)"

This reverts commit a27ad61695053fc18a9630dd9e7c2418416ba915.

This was not a fix for the problem it was an attempt to fix.

new file holding non-AVX stubs for AVX functions on linux

experimentally remove some C++11 flags (likely to return)

ensure that program version strings are not unicode

Fixed up types mismatch in AudioCallback and AudioCllbackData struct

fix typo in wavesbackend code

Revert "add operator bool() to Evoral::Beats"

This reverts commit f8b5263af5e4d5088f1d5551e999679515add432.

OS X compiler gets confused by this change.

Revert "use newly added operator bool() for Evoral::Beats rather than double-negation as boolean"

This reverts commit a19902b52f08bf33c54e60dfb49c44a1c154f4c9.

OS X compiler gets confused by this change.

Revert "replace two more instances of !! with implicit use of operator bool()"

This reverts commit e8eff9fb9c2fa6fc518404c2f9b731f8c28d3a23.

OS X compiler gets confused by this change.

replace two more instances of !! with implicit use of operator bool()

use newly added operator bool() for Evoral::Beats rather than double-negation as boolean

add operator bool() to Evoral::Beats

use consistent (large) size when calling to_ticks()

new Session::default_track_name_pattern (DataType) method, based on an idea in Tracks

add new Gtkmm2ext::pixel_size() to conveniently get width&height for a given font + string

Fixed up the comment

add additional check for playhead priority target when testing loop range, to match Tracks' logic

add Session::reconnection_in_progress(), used by Tracks

use the frame/sample based inter-scene gap, not the msecs one (now removed)

scene gap is measured in frames, not msecs.

Could have used "samples" here but that can wait till we do a big rename
throughout the code base

add some session-scope configuration variables needed by Tracks.

Also add some comments regarding the presence of certain variables in this file

add more directory names to set managed via various APIs (automation, analysis, plugins, externals)

Fixed a misprint in a comment

add API for controlling and determining active status of a SceneChange

Conflicts:
libs/ardour/ardour/scene_change.h

Waves Backend MERGED from Tracks
[To be reviewed by] Paul Davis

Backend MERGE:
Paul Davis' change 3d12a4da82100466f1e5e286260759f07c2e657a
[To be reviewed by] Paul Davis

Reverted Ben's default buffer size function implementation.
Related to  eeca7f7

Reverted Ben's m_DefaultBufferSize stuff as it looks redundant. Will be put back if we establish the opposite

remove more explicit compiler flags by placing them in the compiler flags dictionary

minor fix for wavesaudio backend after earlier changes + merge/meld madness

add PBD::DEBUG bits for WavesAudio and WavesMIDI

[Summary] AudioPort buffer does not need 64 byte alignment which cache_aligned_malloc provides.
Added new function which accepts argument to specify required alignment.

AudioPort buffer requires 32 byte alignment

[Review Required] YPosdnyakov

meld-driven unification of current ardour-ish WavesAudio backend and current tracks WavesAudio backend.

May be incomplete, and may not compile (testing to follow)

cosmetic fix (grygorii)

noop: fix indentation/whitespace

increase event pool for backend reset thread (somewhat unnecessarily, but lets keep Kyiv happy)

AudioEngine tweaks from Tracks, related to stopping audioengine and halting

merge in Tracks-derived session-end-is-N-seconds beyond last region end, but define N as zero for Ardour/Mixbus

fix indentation and remove whitespace

rename Session::add_session_range_location() to ::set_session_range_location()

enabling record (either tentative for punch or fully enabled) cancels play loop mode

implement missing methods for MIDISceneChange

mark session dirty after scene change is added/removed

add signal to indicate absence of removal of scene change from Location

remove artificial, accidental and utterly unintended limit of the numbering of scene changes

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.