ALSA backend: always register ports with systemic latency

and allow re-start to un/set reporting latency.

Don't directly call into engine to stop latency detection.

Fix flow for latency measurements that needs an engine stop.

Make AudioEngine::prepare_for_latency_measurement() idempotent. There are
multple calls (when switching to the measurement page and every time
a measurement is preformed).

Also the Stopped() signal is important for the Dialog state regardless
if the engine was stopped for latency or not.

Fix Editor/Mixer visibility (OSX, NSGLCanvas)

Only unpacking a widget retains the GdkWindow and and the widget is still
mapped. The NSView is still visible. See also 235631a6.

Annotate CC-to-Automation Lua-script and fix start/pos offset.

Consistent button visibility (mixer-strips + VCAs)

Fix nested VCA assigns and mixer-layout:

Show initial VCA assign button on all mixer-strips and VCAs if
there's no assignment.

fix a -Wreorder following d837914e2

AutomationLine time-unit conversion and paste API update

This fixes copy/paste of MIDI automation (time-unit: beat) from/to
Parameter automation (time-unit: samples).

It also fixes repeatedly pasting with tempo-ramps: pre-multiply length
before converting to samples.

Skip script-setup if there are no parameters (and name is unique)

Use a const iterator for sorted RouteList in Session::state

Sort Route xml node order by PBD::ID instead of by PresentationInfo

This prevents the node order from changing when the display order of the Routes
changes, which helps to reduce the amount of Session file change.

This is useful for testing and if keeping sessions under version control.

Resolves: #7327

Accommodate newly introduced source(s) in our MSVC project (gtk2_ardour)

Assume VST chunks are portable (Mac, Linux, Windows VST)

Fix conversion in 41b997a90 (monitor cut, invert)

Update LuaDialog scripts

Lua Dialog refinements:

* Add Heading Widget
* Swap OK/Cancel Buttons
* Update table layout (spacing + alignment)
* Some whitespace tweaks

Add missing Lua ClassKeys (windows build)

Fix Windows build: declare static class-keys (amend 201fd5574)

Update CC-to-Automation script: use Lua-Dialog for parameters

Add a Lua script to exercise the LuaDialog

Lua: Action Script Dialog & MessageBox

Some more Lua ARDOUR::Plugin bindings

Add an API to conveniently query plugin-parameter labels

Expose ArdourDropdown's menu for submenu packing

Remove old special case (ArdourDialog::on_response hides the dialog)

de-select WASAPI by default

Add example script to convert MIDI-CC to Plugin Automation

Simplify example script

Now that AutomationList is-a ControlList no explicit cast is needed.

Add ControlEvent Lua bindings and related methods to access MIDI-CC

Swap inheritance, simplifies Lua Bindings

Add regions at once rather than individually when restoring Selection state

This is a workaround for performance issues with the current implementation
when adding many regions to the selection one at a time.

If the Selection implementation was to change at some point and adding regions
to the selection only takes a small constant amount of time, then this
optimization may no longer be necessary.

Related to: #7274

Changed "ardour" to PROGRAM_NAME

Close all dialogs when closing a session (not only at exit).

Reset process-graph semaphores on engine-restart

Add mechanism to reset/lock semaphores

remove unused semaphore

NO-OP: whitespace

Privatize non-public methods

Set -b switch to only two columns, correct options text.

NO-OP: whitespace

Add a Readme pertaining to GH release tarballs

Remove stub file which was used for pre-commit hook testing

amend be62d335ee

fix endless loop when clicking on the canvas w/o backend

Add HTML output for -b so that output can replace actions page in manual.

OSC: correct spelling

Resolve potential variable name ambiguity for older compilers

Re-nice ALSA MIDI port names

Do not clear pretty-names set by the backend

Since 7dde6c3b8f pretty-names are no longer saved and hence initially
empty. The port-manager cleared existing pretty-port-names.

Fix thinko in 6ae047cdd and 8f488515 (prefer LV2 symbol over ID)

Use correct property names in PBD::Undo class

These have been "normalized" in XMLProperty ctor for many years, so this does
not change the Session format.

Use XMLNode::get_property in Session::restore_history

Avoid using std::stringstream due to potential future issues with C++ locale.

Also avoids potential NULL pointer dereferences.

Use hyphen for XML property names

These have been normalized/modified by the XMLProperty ctor for many years (~9)
so special treatment when reading properties should not be needed.

Use XMLNode::get_property in Session::memento_command_factory

Avoids potential issues with dereferencing a NULL XMLProperty pointer and
improves readability by using better locally scoped variable names.

Fix restoration of Plugin Controllable state ID's

Caused by thinko when merging/replacing two conditional clauses in the
string-convert changes which resulted in bindings not being restored and
regeneration of id's in XML.

Retain state of disabled ctrl-surfaces

Fix adeadlock (or rather NDEADLCK)

ControlProtocolManager::set_state() already takes the protocols_lock.
However effectively this is a NO-OP. During ::set_state no CPI should
be instantiated and  ControlProtocolManager::teardown() returns early.

Update preferences when Ctrl-surfaces change with session.

Notify GUI when tearing down ctrl-surfaces

squelch "curl failed: No Error"

NO-OP: whitespace

Show the splash-screen if the engine dialog's run() hid it.

Run gtk-main iteration after hiding dialogs

Fix operator precedence (amend e7e9e025)

Remove unused enum

Relax LocaleGuards

Arodur itself on longer depends on C/C++ locale for saving/loading sessions.
However, the Localeguard is kept for 3rd party plugins: Reset the C locale
to "C" to enforce consisten numerics and portable sessions as well
as verify that no plugin changes the C++ locale.

Do not set C++ locale.

Ardour translations and GUI string formatting only depends on the C locale.
Modifying the C++ locale may produce erratic results on various systems
(in particular OSX and macOS) and cause incompatibilies with plugins.

Fix crash when clicking on the canvas while un/loading a session.

GTK events are still handled while un/loading a session (e.g display
"Loading" message or showing the splash triggers an event-loop run).
Global canvas elements (e.g. Rulers) may respond to session-specific
actions.

Keep Dialogs which report process around.

Since 5.8-245-g3e43585fa, a response hides the dialog Window
in ArdourDialog::on_response (to prevent dialogs windows staying
around unresponsively while Ardour does background work).
This logic does not apply to Dialog Windows which implement
ProgressReporter or support dialog responses other than OK, Close.

NO-OP: re-order code, add comments

FP8: some code comments and cleanup

Changes needed for building Tim's new 'string_convert' stuff with MSVC (gtk2_ardour)

Changes needed for building Tim's new 'string_convert' stuff with MSVC (libardour)

Changes needed for building Tim's new 'string_convert' stuff with MSVC (libpbd)

Don't explicitly store Auditioner -> Monitor connections.

The monitor section connection is implicit and done by default.
This fixes an issue with the auditioner connections being lost when
switching between sessions that use/don't use the monitor-section.

Previously:
1) load session with monitor section, save session
  -> global config explicitly saved
  "auditioner-output-left" -> "ardour:Monitor/audio_in 1"

2) Load a session w/o monitor-section. The config is parsed,
  "ardour:Monitor/audio_in" port does not exist, connection was lost.

Prefer boolean over literal

Remove cruft. ChanCount::INFINITE is not used

Change header include order to try and avoid INFINITE define on windows

This doesn't seem to be an issue with a newer version of the Mingw-w64 headers
that I use (5.0.2rc2).

Use PBD::string_to_uint32 in Playlist class instead of boost::lexical_cast

Use fixed size type for Playlist sort id as it is serialized

Use PBD::to_string in AutomationTimeAxis instead of boost::lexical_cast and string_compose

Add additional PBD::string_to/to_string specializations for PBD::ID

Use PBD::to_string to set state-dir property in LV2Plugin

To prevent issues with numeric grouping with formatting used by string_compose
template/stringstream when global C++ locale != "C"

Use PBD::to_string() in LV2Plugin instead of string_compose for state_dir

To prevent issues with numeric grouping with formatting used by string_compose
template/stringstream when global C++ locale != "C"

Use std::string::operator+ instead of string_compose in VSTPlugin

Use std::string::operator+() instead of string_compose template in Route class

Using string_compose in this instance doesn't seem necessary, is worse in terms
of readability and is no doubt slower.

Remove PBD::to_string() function from pbd/convert.h

All uses of this function have now been replaced by PBD::to_string() from
pbd/string_convert.h

Remove this function so that it isn't mistakenly used to perform numeric to
string conversion when the result is being used for serialization as that only
works if the global C++ locale is set with LC_NUMERIC=C, which is the case
currently but may not be in the future.

Use PBD::to_string() from pbd/string_convert.h in VCATimeAxisView

Use PBD::to_string from pbd/string_convert.h in VCAMasterStrip class

Use PBD::to_string from pbd/string_convert.h in SoundFileBox class

Numeric formatting is equivalent.

Use PBD::to_string from pbd/string_convert.h in RouteTimeAxis

Numeric formatting is equivalent as LC_NUMERIC=C in global C++ locale

Use PBD::to_string from pbd/string_convert.h in MixerStrip class

The numeric formatting is equivalent. Even though this string is being used in
the UI as a label I think this is another case where we don't want a localized
numeric string, which would only be relevant with a track count >=1000.

Use PBD::to_string from pbd/string_convert.h in MeterStrip class

The numeric formatting is equivalent. The string is being used in a label so it
could be argued that we want localized numeric formatting in this case, but as
it is only relevant if we have >=1000 meter strip keep it the same for now.

Use PBD::to_string from pbd/string_convert.h in ExportTimespanSelector

The C++ global locale is currently set to LC_NUMERIC=C by the first instance of
LocaleGuard so this change means numeric formatting is equivalent.

As the string is being used to construct a label, perhaps we do want to
generate localized numeric formatting in this case. Anyway keep it the same for
now.

Use PBD::to_string() from pbd/string_convert.h in ControlSlaveUI

The numeric formatting is equivalent.

Use PBD::to_string from pbd/string_convert.h in WaveView debug code

Use PBD::to_string from pbd/string_convert.h in ARDOUR::VCAManager

The numeric formatting is equivalent to iostreams when global C++ locale is set
to "C" without the need for a LocaleGuard.

I'm not sure this would have been an issue as the number of VCA's may never get
high enough for digit grouping of the locale to affect numeric output, but now
it is not a possibility.

Use PBD::string_to/to_string in ARDOUR::user_config_directory_name()

The numeric formatting is equivalent without requiring a LocaleGuard or the
global C++ locale is "C".

Use PBD::to_string from pbd/string_convert.h in ARDOUR::ExportFilename

The numeric formatting is equivalent to using iostreams in the C locale without
having to use a LocaleGuard or set global C++ locale.