Add a time+layer equivalence check

This is more useful than exact_equivalent() since _start offset
may not match in some cases (compounds, import, record with different
capture latencies).

However shared group editing (range, copy/paste) does result in
consistent layers and matching position+length.

Add/remove source(s) in our MSVC project (gtk2_ardour)

Update french translation

Fix some mistakes (mostly mixes between nouns and verbs) and tweak some
translations. Also reuse existing translations in some places because
they were better.

Update from POT, no content change

Remove redundant call, fix VST bypass indicator

PlugUIBase c'tor already sets the correct value, using
PluginInsert::enabled. That also correctly shows bypass state of VSTs
that support effSetBypass

Use VST's print_parameter() when applicable.

Remove generic parameter-printer

This has been superseded by value_as_string() along with meta-data
from parameter-descriptor, which is supported by all standards, except VST.

Remove cruft, unused variable

Remove unused value parameter printers

Fixes 2 mistakes in the German translation file

French and German fixes

Work-around crash when printing bindings

(ardour -b or Keyboard-Shortcuts > Print Bindings)
see also https://github.com/Ardour/ardour/pull/447

do not save MIDI port info unnecessarily

on engine (re)start, just mark MIDI port info dirty

add runtime warning message

avoid crazy levels of work when MIDI Port Info preferences panel is not visible

drop silly canonical name stuff from MIDI port info, and just store backend name (GUI version)

drop silly canonical name stuff from MIDI port info, and just store backend name (libs version

rework name/canonical-name stuff for MIDI port info (GUI edition)

rework name/canonical-name stuff for MIDI port info

French translation update

Reduce #includes - avoid ardour_ui.h

Fix some more Gtk::Menu memory leaks

Pass stderr mode properly to vfork wrapper

Fix mp3 export

For reasons yet to be investigated, closing stderr when using a
pipe into ffmpeg results in ffmpeg's stderr being written
verbatim in the output file.

Request for comment: Bug or feature?

Fix some Gtk::Menu memory leaks

A Gtk::manage()d widget will be deleted when its parent container
is destroyed. Top-level context menus are not inside a container and
hence need to be manually deallocated.

The solution here is to use a shared Gtk::Menu pointer that is
centrally de/re-allocated.

This works because the GUI is single-threaded and at most one
context menu is visible at a time.

Cleanup Editor/Marker context-menu

This could have caused crashes previously:
::build_range_marker_menu() may have created range_marker_menu
while the GUI wanted transport_marker_menu.

Prefer using RAII to suspend signals

Fix some Gtk::Menu related memory leaks

A Gtk::manage()d widget will be deleted when its parent container
is destroyed. Top-level context menus are not inside a container and
hence need to be manually deallocated.

This adds explicit delete calls for menus where there is a
member variable reference to the Menu.

Remove cruft: unused members

Remove unused region_edit_menu & edit-point action

save/restore MIDI port pretty names, but prefer backend pretty name if available

selection must be enabled to allow editable columns :((

redesign naming and reload of MIDI port information (GUI edition)

redesign naming and reload of MIDI port information (library edition)

a more nuanced version of the "do-not-automatically-disconnect-selection-follows-input

Fix thinko:  if the entered_regionview is already part of the selection, do not treat it as a special case when splitting.

Remove unused fluidsynth setting (since fluidsynth 2.0)

Use unique midi-port names for Ardour's Dummy backend

Ardour/ALSA allow multiple MIDI devices with the same name

Retain generated plugin menu

Fixes occasional crash when Menu is re-used.
PluginSelector::plugin_menu() is called every time the processor box
shows the context-menu. The provided menu needs to remain valid
regardless of usage.

Add missing newline on error message

Fix typo

LCXL:Disable global mute/solo/recarm in device mode

Tweak session load

Listen positions are already correctly set when the route state
is restored. The Route directly uses the Config.

This skips duplicate calls to re-configure all processors on all Routes
during RCConfiguration::map_parameters() on session-load, twice
(pfl-position, listen-position)

Allow to call set_state() on existing routes

Delaylines are not saved in the XML and internal-return is an
invisible processor not explicitly re-added when the state is restored.

They are [re]inserted during Route::setup_invisible_processors().
So this method need to be called after restoring processor state
(indirectly via configure_processors_unlocked as needed).

PS. During route creation this call happens explicitly and on session load
hookup_io() -> Route::output_change_handler() implicitly sets this up.

Properly map MonitorSection state on session load

Tweak, optimize route setup

Speed up route creation: add_processor() takes a process-lock
and calls configure_processors() but this is done explicitly at the
end of init() already.

Close stderr of child processes on MacOS

This fixes a bug on some modern mac systems. Related to
setup_logging() changing stderr to use ASL and write to com.apple.console.

When a forked application writes to stderr while ASL is used,
the child is terminated for some reason.

Use enum for exec stderr parameter (2/2)

Use enum for exec stderr parameter (1/2)

Enable de-click in disk-reader

This breaks auto-return and other post-transport locates until
such transport-states wait for de-click before seek().

do not disconnect MidiPortSelection-flagged ports  from everything when (MIDI track) selection changes

If the user manually connects such a port to something, then it is
up the user to disconnect it too

Fix recursive WriteLock crash due to signal emission

Route::processors_changed() signal connects to a same thread session-slot
Session::route_processors_changed() which can end up iterating over
Routes and Processors to re-calculate latency.
This takes a ReadLock after WriteLock, which eventually leads a duplicate
unlock and segfault.

Make it compile again

ISO C++ forbids applying 'sizeof' to an expression of function type

Add/remove source(s) in our MSVC project (audiographer)

remove double assignments and one debug trace output; tweak second debug trace output

debug trace output tweaks

remove debug output

remove conditional surrounding wait-for-master-to-catch-us logic

remove debug output

do not initialize LTC timecode rate from session FPS

fix decision about sending MTC to be based not just on whether the transport master is locked, but whether it is external

audioengine skip should trigger full LTC_Slave::reset(), not just feed silence to the decoder

LTC_Slave: count frames since reset

debug trace output tweaks (including avoiding stderr/stdout issues)

LTC_Slave::reset() should change _current.speed back to zero

NO-OP: brace position

NO-OP: brace position

expand scope of LTC_Slave::reset() to include more object state

fix semantics of LTC_Slave::locked() - we must have detected FPS to be considered locked

fix initialization order

rename LTC_Slave::prev_sample to prev_frame, correcting s/frame/sample/ changes

removed _locked property from TransportMaster (it's a method, not a property)

remove ill-conceived use-session-timecode-format option

Add GUI option to include session-meta-data when exporting mp3

FFmpeg/MP3 export supports tagging

Remove another unused include

Show VCA pane when a new VCA is created.

Add a signal to indicate VCA creation

As opposed to VCAAdded this is not emitted when re-loading a session
and only emitted when a user creates a new VCA.

Revert "Show VCA pane whan adding a VCA"

This reverts commit 9a0260148d7c582037bdc13aab6b412379638ab3.

Show VCA pane whan adding a VCA

Prefix an asterisk to indicate plugin replication

This saves space compared to "(NxN) ", especially with narrow
mixer-strips

Replicated plugin instances may or may not be mono

GUI updates to handle monitor-section changes

This fixes various MS related action sensitivity issues, as well
as action-states.

Also manually adding a monitor-section to a session now always
immediately shows it, regardless of prior ToggleMonitorSection
visibility.

Update scripts, monitor-section API change

Revert Monitor-Section to be per session

This partially reverts 639dff3a7c7. When loading a session,
the monitor-bus that was saved with the session is used.

This changes semantics of the monitor-section/config.

Config::set_use_monitor_bus(bool) is used to initiate a change!
Notification about the change is sent asynchronously by
Session::MonitorBusAddedOrRemoved

It is no longer possible to directly call add/remove_monitor_section()
and leave the session + config in an inconsistent state.

NO-OP: whitespace

Don't use c99 (amend 959947e7f84a)

Add/remove source(s) in our MSVC project (gtk2_ardour)

Add/remove source(s) in our MSVC project (libardour)

simplifications to the logic and additional comments for code that handles auto-return at transport stop

auto-return is still available if using JACK transport

Remove unused RulerType

NO-OP: whitespace

Abort when Action cast failed

Previously the action-manager only called ::abort() when
an action could no be found. This add a further restriction
that the expected class must match.

Consolidate ToggleAction cast, use new API

Furthermore we can assume that all Actions are registered and
Unregistered actions will ::abort() execution.

Initialize per session instant state

Per session (instant.xml) GUI options do not have a sigc::slot<> backed
configuration interface to initialize the Actions.

This fixes a first-time start issue (no instant.xml) where GUI panes
may be visible (default) while the actions are "off" (default).

This is a tentative solution, instant.xml backed ToggleAction
initialization needs to be consolidated somehow.

x-fade uses cairo-generated icons, remove XPM