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.

Use XMLNode::get_property in ARDOUR::ElementImporter

Use PBD::to_string/string_to in ARDOUR::ElementImporter class

Use PBD::to_string API from pbd/string_convert.h in AudioFileTagger

Use pbd/string_convert.h API instead of std::ostream for numeric formatting to
remain locale independent and not have to use a LocaleGuard. The formatting is
equivalent.

Use PBD::to_string API from pbd/string_convert.h in AudioTrackImporter

Use pbd/string_convert.h API instead of std::ostream for numeric formatting to
remain locale independent and not have to use a LocaleGuard. The formatting
is equivalent.

The code in this file has been disabled in the GUI for some time...and I doubt
it would now work correctly.

Use PBD::to_string() from pbd/string_convert.h in ardour ALSA utils

Use pbd/string_convert.h API instead of std::ostream for numeric formatting to
remain locale independent and not have to use a LocaleGuard. The formatting is
equivalent

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

Use pbd/string_convert.h API instead of std::ostream for numeric formatting to
remain locale independent and not have to use a LocaleGuard.

Use PBD::string_to/to_string when de/serializing in AutomationList class

This avoids requiring a LocaleGuard to get the correct numeric formatting and
saves/restores the automation data to the precision required for roundtrip
equality.

Remove unused header include in ARDOUR::Amp source file

Use PBD::string_to/to_string in IO::find_possible_bundle

Possible issue with numeric formatting using std::streams in locales with
grouping. Although I don't think it would be an issue in this case use locale
independant alternative for numeric formatting anyway.

Use PBD::to_string instead of std::ostream when setting up video server url

To avoid issues with locales that use grouping/thousands separators.

Use PBD::to_string to convert period count in EngineControl class

The functionality is the same so might as well use it.

Use PBD::to_string to convert bufsize to string in EngineControl class

string_compose uses std::ostream/stringstream which will insert thousands
separators in some locales. This was not a problem when LocaleGuard set the
global C++ locale to "C"

Use PBD::string_to functions in VideoUtils::video_query_info

LocaleGuard is in use so float <=> string conversions must be expected in C
locale format

Use snprintf instead of std::stringstream when converting color values to strings

std::ostream/stringstream will use the current locale to determine the numeric
formatting. If the locale uses grouping then thousands separators will be
inserted in the output which produces an invalid color string in
UIConfiguration::reset_gtk_theme() and when converting colors to strings in
UIConfiguration::store_color_theme()

This has not been a problem so far because it appears that LocaleGuard does not
reset the LC_NUMERIC value for the global C++ locale. So if a LocaleGuard is
created at any time before these functions are called(even if it goes out of
scope) the numeric formatting used by std::streams will use the "C" locale
formatting facets.

Use XMLNode::get/set_property API in Faderport surface

Remove now unused PBD::string_is_affirmative function

The equivalent function is now PBD::string_to<bool>

Use PBD::string_to<bool> in MidiTimeAxisView class

As the conversion was performed with PBD::to_string

Use PBD::string_to<bool> in RouteTimeAxisView class

As they were converted from bool to string using PBD::to_string

Remove conditional checks that are always true in RouteTimeAxisView class

Use PBD::string_to<bool>() in GenericMidiControlProtocol class

Equivalent to PBD::string_is_affirmative()

Use PBD::string_to<bool> in legacy ARDOUR::Route methods

Use PBD::string_to<bool> in ARDOUR::Processor class

Use PBD::string_to<bool> in AudioPlaylist class

Use AxisView::get/set_gui_property API in RouteTimeAxisView class

Use AxisView::get/set_gui_property API in MixerStrip class

Use AxisView::get_gui_property API in AutomationTimeAxis class

Use AxisView::get_gui_property in AudioRegionView class

Use AxisView::get_gui_property API in TimeAxisView class

Use AxisView::get_gui_property API in AxisView class

Add AxisView::get_gui_property method, use PBD::string_to<T> for string conversion

Use PBD::to_string to convert non-string types in AxisView::set_gui_property

Let AxisView::set_gui_property perform type conversion in TimeAxisView::set_height

Remove unused XMLNode::add_property methods

These are now unused and functionality is replaced by XMLNode::set_property

set_property is a better name as a node can only have properties with unique
names and the property will be set or reset(if it already exists). Changing the
name also makes it easier to transition and test the new API.

Use XMLNode::set_property API in xml tests

Use XMLNode::set_property API in copy-mixer Session utility

Use XMLNode::get_property in AudioClock class

Use XMLNode::get/set_property API in VideoTimeLine class

Use XMLNode::set_property API in VideoMonitor class

Use XMLNode::set_property API in UIConfiguration class

Use XMLNode::get/set_property API in SelectionMemento class

Use XMLNode::get/set_property API in Selection class

There were many possible value truncations occuring and some precision loss
with the double conversions.

Use XMLNode::get/set_property API in ProcessorBox and ProcessorWindowProxy classes

Use XMLNode::get/set_property API in ProcessorBox class

Use XMLNode::get/set_property API in MeterBridge class

Use XMLNode::set_property API in LuaInstance class

Didn't seem worth converting from XMLNode::property API to
XMLNode::get_property and there may be some ordering issues.

Use XMLNode::get/set_property API in LocationUI class

Use XMLNode::get/set_property API in Keyboard class

Use XMLNode::get_property API in AutomationTimeAxis class

Use XMLNode::set_property API in GUIObject class

Use XMLNode::get/set_property in ExportVideoDialog class

Use XMLNode::get/set_property API in EditorRulers class

Use XMLNode::get/set_property API in EditorRegions class

Use XMLNode::set_property API in ARDOUR_UI class

Use XMLNode::get/set_property API in ARDOUR_UI class

Add AudioClock::on() method as convenience method for serialization

Use XMLNode::get/set_property API in Mixer_UI class

Remove unused header include in video_timeline.cc

Use XMLNode::get/set_property API in Editor class

Use XMLNode::get/set_property API in EngineDialog class

Fixes many of the type conversions but I doubt any would have been an issue.

LocaleGuard is no longer necessary

Add PBD::to_string/string_to template specialisations for types in editing.h