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

NO-OP: whitespace (remove vi modelines)

Consolidate gdither noise code

Amazing PCM byte/array convert functions, unused.

The end of long and winding "road to compiling" cdc2d055f

Remove unused RDF in RIFF prototype

Remove even more cruft

http://lv2plug.in/ns/dev/host-info# doesn't exist, and nobody needs
a config.guess from 2005 :)

Move bug-tool into the Python Museum

Remove ancient, unmaintained xcode project files

Remove cruft, unused files

Clean up remnants from a half eaten apple

Add phase labels -180..180 deg for analysis display

NO-OP: whitespace

NO-OP: whitespace

Fix analysis plugin when using plugin-presets

Allow to special case plugins used for IR analysis

This is relevant for some VST specifics (e.g connected pins) or
similar audioMasterCallbacks that use either global or plugin-insert
specific data.

Copy state when replicating an AU plugin

This is only used for plugin-analysis, AU plugins are otherwise not
replicated, and variable-i/o is used instead

Properly copy LV2 plugin instances on replication

This fixes issues for stateful plugins and plugins with
previously loaded preset.

Take latency into account when analyzing live signal

NO-OP: whitespace

Re-work plugin-analysis (fix crashes)

Plugin-analysis uses a fixed number of samples, which may be larger
than the session's block-size. This caused problems for some VST
plugins that use audioMasterCallback to query the session's block-size.

Plugin FFT analysis now processes in chunks of the session's block-size
to avoid this issue. This also allows to increase accuracy for
all plugin standards (FFT size was increased to 8192).

More slaved_to() replacements, fix typo

Consolidate code, use dedicated API