Clean up _before XMLNode in AutomationList if no automation write occurred.

Automation watch sets position for new write pass when transport stopped.

Revert starting write pass only in write mode when transport located.

Grey out unavailable note context menu items rather than hiding them.

Clarify context menu for midi notes.

- right click on a note selects it if unselected or selection empty.

- note_context_menu is shown as described in #6348

Add comment to MRV snap workaround.

Fix snap modifiers not working on notes when grid is set to off.

Handle paste from non-toggled to boolean automation track.

- sort-of fixes #6431

suspend timers while flushing the gtk-event queue.

This fixes an issues where Ardour never completes to flush the GUI event
queue due to rapid updates for meters and similar events.

ARDOUR_UI::load_session() never returned and the complete session
ran inside Gtkmm2ext::UI::flush_pending(), this later causes a crash
at exit.

as side effect, this also speeds up session load.

RAII Timer Suspend wrapper

add API to suspend timers signal emission.

"End" -> "end" in end point trim command name.

Use automation line's frame of reference for y position.

- removes the need for 'pixel hunting' wrt NAME_HIGHLIGHT_SIZE.
- new control points generated by clicking on a line are placed
  where the verbose canvas cursor says they are.

Dont do duplocate calls to MidiRegionView::trim_front_ending () in TrimDrag.

- fixes #6658
- also leaves undo history alone if point trim does nothing.

Fix reversed ratio in last commit.

Start line drag using canvas y coords.

- fixes #5759

rework dbf28e3 (hidden parameters)

note to self: at least test compilation before git push.

don't display hidden automation-controls.

fix JACK/windows compilation

fix embedding AU/carbon UIs

the UI shows top/left (not centered, no resize) more detail work to come.

Revert "potential fix for some AU plugin UIs not displaying"

This reverts commit e8fe35d96ed13f3df0e2f03e2f347e908064a31d.

notify LV2 GUIs about port-changes when loading presets

use weak-jack's jack_port_rename wrapper

This avoids "[ERROR]: JACK: jack_port_set_name: deprecated".

if libjack does not have jack_port_rename(), weak_libjack.def provides
a fallback using jack_port_set_name().

potential fix for some AU plugin UIs not displaying

looks like not all AU plugin GUIs redraw themselves if
[win setAutodisplay:1] is unnot.

A user reported that some Waves plugins and Shattered Glass 32bit AUs
are blank in 4.4 on 32bit OSX.

see also 5eefdf7 and earlier AU/API changes.

another log-scale automation fix

LineDrag fixes wrt initial drag position and display of y value.

AutomationLine::drag_motion () returns any clamped fractions correctly.

Fix incorrectly placed control points (y-axis). fixes #6647.

- also use the return from AutomationLine::drag_motion () in
  anticipation of it correctly reporting its clamping to
  AutomationRangeDrag.

linux-package: add gcc5 suffix to the file-name

fun with valgrind

and another LV2-1.0.0 compat ifdef

amend 311adde (LV2-1.0.0 compat)

properly translatable plural forms

Changed MCP trim detection to match method used in GUI.

Markers use the selection change stack.

Amend last commit.

Set a larger minimum size for recent session scrollers.

back to mjpeg (revert 2cb511e and 3f157f9) for now

There is anecdotal evidence that using mpeg4 as codec leads to issues
(xjadeo indexes and gets stuck at 99%, likely in libavcodec).

The main motivation for using mpeg4 is/was windows/VFAT 2GB file limit
and improved video quality.  This will have to be revisited.

Delete SystemExec in the main process thread.

fix crash on SystemExec::Terminate

if Terminated() connects in the same thread and deletes the class itself
the closure in interposer_thread() can fail.

Relative snap for markers.

- also removes many uninitialised variable warnings in
  editor_drag.cc found by cppcheck.

add ifdefs for LV2-1.0.0 compat

ResizeNoteDrag selection click behaves as NoteDrag's does.

Clear point selection when uniquely selecting a note.

- Also fix selection undo when creating notes w/control
  in MouseContent mode.

Make mackie control  view mode save bank for all modes.
Also moved duplicated code to one function.

mime-type definition is version independent.

File glob "*.ardour" -> application/x-ardour is defined in ardour.xml

Use lwrcase_dirname for desktop file names and substitutions.

set executable and icon name in desktop-file

Also opt for version-agnostic mime-type (file-format version
is independent of program-version and of file-extension)

Remove GhostRegion::CatchDeletion signal to reduce session close times

Currently when a GhostRegion is deleted by its "parent" RegionView it emits the
static GhostRegion::CatchDeletion signal which is connected to the
RegionView::remove_ghost method of every RegionView instance.

With a static GhostRegion::CatchDeletion signal a particular test session
causes 31 Million calls of RegionView::remove_ghost on Session deletion and the
session takes 70 seconds to close with a debug build.

The lifetime of a ghost region is tied to both the TimeAxisView(TAV) and
RegionView(RV) in that when a RegionView is deleted all GhostRegion instances
associated with the RegionView should be deleted or when a TimeAxisView is
deleted all ghost regions that are contained in the view should be deleted.

This means that there needs to be notification between GhostRegion and both
classes. Instead of using a signal for this as we know there are only two
listeners and GhostRegion already holds a reference to the TimeAxisView, also
take a reference to the parent RegionView in the GhostRegion constructor and
use it to notify the RegionView when GhostRegion destructor is called so it can
drop any references it holds.

Using a direct function call in the GhostRegion destructor to notify the
TimeAxisView and RegionView "parents" brings the unload/close time down for the
test session from 70 seconds to 4.5 seconds.

The GhostRegion also references canvas items that are added to the TimeAxisView
canvas group or at least a canvas group that it manages. So when the
TimeAxisView is destroyed and the canvas group that is the parent of those
items is destroyed, the GhostRegion's canvas items will also be
deleted/destroyed by the parent canvas item/group. This means the GhostRegions
must be destroyed when the TimeAxisView they are contained in is destroyed or
there will be dangling references to canvas items that have already been
deleted and trying to delete them again will be bad.

Enable i18n for info/error messages in MidiPatchManager

Only parse additional Session midnam files on Session load

When loading a Session add the Session patchfiles directory to the
MidiPatchManager search path and only process/parse the files for that
directory rather than refreshing/reparsing all the files. Similarly for unload,
just unload the devices that are from the Session specific midnam files instead
of removing the path and refreshing/reparsing all the files.

This will not remove the "system" midnam files as they are always added first
and duplicates from the session patchfiles directory are ignored.

Store the file path when constructing MidiNameDocument

This is so we can compare and see if we have already parsed the file

Decouple Session from MidiPatchManager and reduce parsing of midnam xml files

The MidiPatchManager only requires a reference to the session to get the path
to the Session midnam directory so change it so that the path is passed to
MidiPatchManager::add_search_path on Session construction and removed on
Session Destruction. This will also make it easier to test and reduce compile
times etc.

For the common case where the Session doesn't have a Session specific midnam
patch files directory(for instance a new session) it won't cause a refresh and
reparsing of all the midnam files. This saves about 2 seconds to load a Session
on my machine(fast machine with SSD), or about half the time spent in the
Session constructor for a new session.

There is still going to be that initial cost of parsing the midnam files when
the first session is created after starting Ardour. Options to remove that
would be to parse the files asynchronously and or use a faster xml
parser(eventually), neither of which seem worth doing at this stage.

This change will cause a performance regression for the uncommon case where a
Session with Session specific midnam files is unloaded and then another Session
with Session specific midnam files is loaded as it will cause the common midnam
files in midi_patch_path to be parsed twice(unload and load).

Add PBD::Searchpath::contains method to check if a Searchpath contains a path

Expose protected PBD::Searchpath API

I prefer to use these as they are more explicit than using the overloaded
operators.

Don't parse midnam patch files contained in Session twice

MidiPatchManager::refresh already adds the patch files contained in the session
folder

Refactor common code in MidiPatchManager class into new method

Don't call MidiPatchManager::refresh when creating MidiPatchManager instance

Currently when loading a session for the first time MidiPatchManager::instance
creates the MidiPatchManager singleton which calls MPM::refresh and all the
midnam files are parsed etc. MPM::set_session is then immediately called and
all the MPM state that has just been set when parsing all the midnam files is
cleared and the parsing of all the files is performed again but this time with
any session specific midnam patch files.

MPM::instance and MPM::set_session consume about 55% of the time spent in the
Session ctor according to kcachegrind and removing the double call to refresh
brings Session construction time for a particular test session down from 7.5s
to 5.5s

Fix a small buffer issue which could sometimes prevent sessions from being able to load

In the function 'LV2Plugin::add_state()' the snprintf() call can easily print 19 or even 20 bytes - so a 16-byte buffer wasn't large enough.

Build freedesktop files if requested.

Amend last commit.

Center some windows known to be presented by ardour.

fix edge-case: (move playhead w/o session)

remove invalid assertion - #6634

There is a highly unlikely case where the render thread can have zero
requests in the queue, but it is not supposed to be terminated.

 1) WaveView::queue_get_image();
wake up thread, *but* the thread does not start yet
 2) WaveView::cancel_my_render_request();
and now the thread starts.

1,2 are initiated by user actions from the GUI thread and are normally
orders of magnitude slower than scheduler-thread wakeup.

Add some newly introduced sources to our MSVC project (libardour)

Merge branch 'chaot4-fix_overwrite_file'

Also use overwrite_file_dialog at all the other places.

Confirmation on overwrite for track and session templates. -fixes #6587

Add overwrite option to save_template. (default = false)

Before it could not overwrite.

fix automation update at transport stop when in automation Play mode

fix mute automation in Play automation state

Add AutomationControl::set_value_unchecked() and AutomationControl::writable() and use them.

Classes derived from AutomationControl now check ::writable() in their ::set_value() methods to ensure that they
do not attempt to overwrite data sent to them while automation playback is underway.

add commentary and clean up blank lines

More desktop file renaming.

Remove old ardour3 desktop files.

Add ardour4 desktop and mime type files.
- looks like the install target for these has been removed, but
  the existence of these may be of help to packagers.

User interaction preferences tooltips clarify button names.

variable name change: make purpose of formerly named "_values" in LV2 plugin UI code more clear

And if this is wrong, then rename it again with the right one

variable name change: percent is 0..100, 0..1.0 is a fraction

ensure that mutator function for CONFIG_VARIABLE_SPECIAL is used when assigning initial value during construction

Make sure Mackie Control view modes start with route 1 except Mixer mode goes to last position.

Added "Selected" view mode to Mackie Control.

Fix crash in new CoreaudioSource code (on invalid file)

confirm LV2 GUI changes

If a parameter change is initiated by the UI, the host sends a
notifications to confirm (echo) or invalidates (replaces) the value.
(automation: touch, playback,...).

Stateless LV2 GUIs without internal data-model depend on this.

hack to keep UI responsive with heavy AU views.

OSX copy/constraint mod tweak (amend 0675fe1)

do not update LV2 output port displays unless the value has changed

add commentary

also update LV2 output port displays as part of periodic updates

fix AudioUnit parameter listening to work with new ParameterChangedExternally semantics

Clean up generic and LV2 plugin GUI to deal with ParameterChanged(Externally) alteration

rename Plugin::ParameterChanged to ParameterChangedExternally to reflect its intent, and clean up the libardour side of result.

The signal exists to notify listeners that something outside of the host's control (e.g. a plugin's own GUI for AU or VST)
has modified a plugin parameter. Previous code had strange feedback loops and ambiguous semantics.

Revert "rename ParameterChanged signal in Plugin to ParameterChangedExternally to reflect its intent, and clean up the result."

This reverts commit 336b2eb9a4a8634bae84a15e952d20335aa28c12.

possible fix for invalid cleanup space calc on win32.

NOOP, re-indent

remove cruft

rename ParameterChanged signal in Plugin to ParameterChangedExternally to reflect its intent, and clean up the result.

The signal exists to notify listeners that something outside of the host's control (e.g. a plugin's own GUI for AU or VST)
has modified a plugin parameter. Previous code had strange feedback loops and ambiguous semantics.

Significant modification of LV2 GUI updating was required.

Still to be tested for feedback loop issues: AudioUnits

remove unused "copy modifier hint"

suggest primary-mod as default for copy

fixes inconsistency on OSX.