Remove dead/annoying/unsafe code

Note the old Note::operator= was unsafe, since it made shallow copies of the on
and off events, which results in a double delete of events when the notes are
destructed.

Skip testing example lua scripts which are not bundled

Fix Sequence test

Remove dead code

Fix event type and parameter type confusion

I'm not sure if this is really the best way to do event types (should it
just be a completely static enum in evoral, or completely dynamic and
provided by the type map, or a mix like currently?), but previously the
event type was frequently set to either total garbage, or parameter
types, which are a different thing.

This fixes all those cases, and makes Evoral::EventType an enum so the
compiler will warn about implicit conversions from int.

Fix warnings

Remove Evoral::MIDIEvent

It is slightly questionable whether type specific methods like
velocity() belong on Event at all, these may be better off as free
functions.  However the code currently uses them as methods in many
places, and it seems like a step in the right direction, since, for
example, we might some day have events that have a velocity but aren't
stored as MIDI messages (e.g. if Ardour uses an internal musical model
that is more expressive).

In any case, the former inheritance and plethora of sloppy casts is
definitely not the right thing.

Factor out and extend MIDIXML implementation

Fix Sequence/Event const-correctness issues

Making one string translatable

It should allow this dialog window to be translatable : http://pix.toile-libre.org/upload/original/1480706219.png

I hope this is the correct way to do so. Please double-check what I've done before merging.

Use correct action in map for Korg nanoKontrol2

According to the comment in line 79, this button should toggle between mixer and
editor windows - but it didn't (for me). Apparently that action named has changed
in Ardour 5.x - anyway, using this command, it works for me.

update summary & connection-matrix colors when route-color changes

Fix un/setting group colors

and a more intrusive KDE workaround

remove scope from translated phrases.

pay some attention to the special guest of the night: KDE window stacking

One fix.

One fix

French translation update

French translation update

more translation + fixes

Ensure the window-close button is visible for the Engine-Dialog.

French translation update

A few more improvements/fixes and new strings.

fix compilation with newer version of *mm and gcc-6.2 (RefPtr<> cast)

update system_config

Remove close/OK buttons from engine dialog

prefer bool operator

allow to remove a session from the recent list

NO-OP backport changes from Mixbus branch.

Fix route color selection dialog

French translation update - fixes

a few fixes

French translation update

- a bunch of "fuzzy" stuffs removed (after checking/correcting those)
- a bunch of min/maj fixed
- >50 new translated strings
- >40 updated/fixed strings
- consistency across "… -> ...", "xrun -> décrochage", "lock -> verrouiller" and "mixer -> console de mixage"

French translation update

Two small fixes.

moving POSIX_VISIBLE to the waf script part

separate openbsd cases

Making buildable under OpenBSD.

Protect a-comp's display/state against NaN/Inf.

protect a-delay again Inf, NaN, HUGE and stuff.

Speed up track deletion when Editor-Mixer is visible

properly handle FS pitch-bend - closes #7140

Consistent sub-menu separators for Preferecens in fr translation

OSC: Solo toggle should maintain state when locked, rec and rec_safe should show true state.

Fix Duplicate Track with Copy Playlist option selected

Previously two new empty playlists were created.

Related to and mentioned in issue #7141

OSC: Squelch select feedback when no feedback is configured.

Fix reversed behaviour of Playlist -> New and New Copy

Resolves: #7146

Thanks again to elgoun for the patch

Fix duplicate track with new playlist so only one playlist is created

Instead of 2 empty playlists, thanks to elgoun for the patch

Resolves: #7141

update default config (see also 5ec21347a)

NO-OP; Backport changes from Mixbus branch

Don't include program-name with <Option name="color-file" />

This allows for e.g. mixbus and mixbus32c to share the config, but
use different color-files at runtime.

Fix loading, recording & saving MIDI with PolyKeyPressure events.

GUI still does not handle this (neither automation lane, nor List Editor),
but the file loads, plays and exports correctly.

revert 335debfa for the PA backend

The same device may have different names for input + output

OSC: no op, white space cleanup

OSC: Make plugin and plugin parameter numbering 1 based to match everything else.

Add [hidden] option to list "Dummy" backend with optmized bundles

Restrict Device selection to valid choices.

Add API to enforce valid device selection.

OSC: Add patch from 7137 and make it work for manual port selection.

speed up track/bus creation a bit

RAII to postpone processor changes/graph recalculation

OSC: Patch from 7136 added cleaned and tested.

cont'd work to prevent duplicate playlist names

when duplicating tracks and copying playlists, use a new name for the copied playlist

Fix forwarding of output-buffers.

fbc8504f9eb74 swapped the iterators: Instead of iterating over output
buffers like BufferSet::read_from(), the new code iterates over
the current buffers. In case of 1 in -> 2 out, only the 1st channel
was be copied.  Also unlike BufferSet::read_from() the ChanCount was
not updated.

Processors after a Delivery e.g. out-meter, AFL processor did not see
all channels.

fix default output map display for no plugin processors

(eg. mono->stereo deliveries)

make the session do the right thing when clearing solo state during non-loading conditions

remove redundant (identical) 2nd conditional, and improve comment

when clearing route solo state, do the required update

Do not try to restore Route solo state after clearing all solo state

fix implementation of SoloControl::clear_all_solo_state()

This no longer emits Changed() unless a change was made, and more
importantly, calls AutomationControl::actually_set_value() if
the self-soloed value needs to be changed

provisional changes to speed up solo changes to large numbers of routes.

Moves global update of solo state and emission of Session::SoloChanged to
a single point after 1 to N solo controls are changed.

Also avoid unnecessarily emitted Activated() signal for listen controls,
though Process::{activate,deactive}() should probably be redesigned to
avoid this in a "deeper" way

Speed up AFL/PFL changes for large sessions

update_latency_compensation() and resort_routes() are expensive and
were called for every individual change.

The DisplaySuspender is useless with async Session RTEvents

Don't allow duplicate playlist names

_maps_from_state is legal after unfreeze (restore state)

a-High/LowPass allow 8K samples inclusive

midi ghost region fill colour is per event. don't store it. ensure _outline is set.

amend last commit.

midi scroomer updates streamview during drags.

French translation update

Mainly improvements to the Preference tabs.

small correction to de.po

Fix mysterious crashes such as #7049

Fixes an issue with corrupted std::lists<> due to concurrent writes
to the invalidation list which eventually resulted in
EventLoop::invalidate_request() not invalidating requests.
Concurrency sucks rocks hard.

another hack to accommodate different project names

Mixbus32C-$MAJOR  + Mixbus32C (dash before major version)
vs Ardour$MAJOR + Ardour
vs Mixbus$MAJOR + Mixbus

catch up with .de translation

clean up pending region selection if a region goes away.

MidiRegionView::clear_events() does _selection.clear(). remove the extra one.

fix a crash when deleting  a MIDI region with CC automation (see recipe below)

- object mode: select a MIDI region that has CC automation.
  Move the mouse over a CC automation event,
  then press backspace to delete the selected region

- the recipe is mentioned in #7049, but seems unrelated.

clicking on an automation line selects adjacent points correctly.

decouple midi region note range and height from midi streamview.

- this allows a midi region drag to update the visible notes
  correctly while crossing MIDI streamviews with a differing
  note range.
  as a side effect, fixes a bug where changing
  note range on a track did not draw some notes
  (apply_note_range redisplays the model).

son't try to select non-existent notes after editing (and thus crash).

- should fix 7135

constify some TempoMap functions.

remove set but not used variable in tempo.cc

MIDI LFO generator plugin

amend 66488e117 - BBT clock scrolling handles multiple audio-locked meters.

don't display notes that start at region end.

- its not obvious that this is happening in sustained mode,
  but Percussive reveals all.

fix thinko in 8e8c4a73f1e.

filter events in HitCreateDrag, remove some code.

prevent Percussive mode from displaying a ghost hit at region end.

NoteCreateDrag - remove some code

TempoMap::bbt_duration_at() handles an audio-locked meter.

- fixes some odd results when scrolling down/up over the BBT clock display.

Make Cut mode respect snap modifer

Show cursor in Cut mode on button press at cutting position

Previously it would be shown at the mouse cursor position even though the
Region is cut/split at the snap/quantize point if no motion occurred.