fix up some suboptimal and crash-prone aspects of 7c263f3bc4f3bddd8094c9baecf584503012acc8 from Jeremy Carter

bcf2000: make play button toggle roll

Fix ChannelLeft command in BCF2000 mackie profile

Merge remote-tracking branch 'upstream/master'

add abort() to non-reached code

This cleans up a lot of false-positives in static analysis
and also helps compilers to optimize code paths in general.
(tagging the fatal stingstream operator as ‘noreturn’ is
far less trivial)

Fix various sticky/broken cursor issues.

This still isn't quite right, but it's a lot less broken than before, at least.

the return of the PPC nostalgia

Fix wonky note length when create-dragging notes backwards.

Specifically, when pivoting from forwards to backwards (around the drag start
point), the note length was too long.  Setting both the start and end x
coordinates of the rect every time to the right value does the right thing.

Fix error when create-dragging note backwards past start of region (#5934).

Don't make MIDI regions in note mode completely invisible.

This makes for extremely confusing behaviour, particularly when creating new
regions.  This is still probably too transparent, IMO, but at least isn't
totally mysterious.  "visible" is a bit of a misnomer in the code, now.

Fix crashes on various MIDI editing operations.

Assertions assumed old strict ordering, no longer reflecting reality of fuzzy
time comparison (introduced in 86f1b8).

Fix crash after deleting automation region views (#4718).

Fix crash when deleting overlapped regions.

Use RegionSelection for MIDI regions as well, since the old dumb stub didn't do
some things correctly.  There's probably no reason to have a separate class for
this at all, and some good ones for putting all regions in the same selection,
so we should probably do that.  For now they are still separate in the
selection but use the same base class.

yet another copy/paste typo :(

fix typo in b9fc616538

Linux-bundle: deploy all compiled backends

auto-initialize weak-jack

(not needed with ardour - but keep code in sync with
https://github.com/x42/weakjack)

prepare for JACK meta-data API

make new jack thread API functions optional

add symbol debug information to weak-jack

fix memory leak

install all additional binaries in $libdir

fix define

Don't iterate in 'ARDOUR::Route::get_control()' unless a ReaderLock was successfully obtained

Make sure we can't execute 'ARDOUR::Route::meter()' unless a ReaderLock was successfully obtained

don't use cached sudo password in un/installer

set umask in installer

see https://github.com/megastep/makeself/issues/6

yet another build fix

on x86_64 weak_libjack.c needs -fPIC which needs cflags which need 'c' :)

fix compilation with gcc-4.9.1

add popup message and error log for AU crash reports

Prepare AU-discovery crashlog.

stopgap solution to allow a user to detect which plugin caused a crash during discovery.

add AU plugin scan to Preferences UI

failsafe AU scanning part one:

Don’t scan for AU plugins unless users explicitly starts a
plugin scan. Disable automatic scanning if Ardour crashes
during AU plugin discovery.

Merge branch 'plugin-automation'

fixed automation *_all button crash

Merge branch 'plugin-automation'

Added Write All, Play All, etc. automation buttons to generic plugin UI

fix/amend b84b0bfd

clean up after ourselves

linux packaging: add some checks to reduce warnings

add 512px icon

osx packaging: add some checks to reduce warnings

osx build script: change default xjadeo/harvid cache folder

fix osx startup script /usr/local test

A vanilla OSX system has no /usr/local/ folder. Ardour can still
run with the coreaudio or dummy backend.

refactor MIDISceneChange color property addition by moving it into SceneChange to anticipate other types of SceneChange objects (e.g. OSC)

when creating a new MIDI scene change, check existing scene changes for equality and reuse their color (which may or may not be set)

serialize/deserialize MIDISceneChange color; put out of bounds color into static const

add color property to MIDISceneChange

add operator== to ARDOUR::MidiSceneChange

tweak canvas on_expose_event() handler to avoid unnecessary save/paint/restore

further work on new color manipulation code

add documentation about meter colors

fix signed/unsigned compiler warning

improve random seed (for dummy generators)

add required header for compilation

redraw entire canvas when ColorsChanged is emitted

set canvas background color

add new color variable for arrange canvas background

Conflicts:
gtk2_ardour/canvas_vars.h
gtk2_ardour/default_ui_config.in

add background color property to Canvas::Canvas ; rearrange expose handling and include background fill

Conflicts:
libs/canvas/canvas.cc
libs/canvas/canvas/canvas.h

Modifications needed to build ReasonableSynth with MSVC

Add a small dummy command to a couple of MSVC build projects

These are trivial (no-op) fudges - to ensure that if we can't copy the built executable for whatever reason, it won't get flagged up as a build failure.

Add some newly introduced source files to our MSVC project (cairocanvas)

Just rename problematic Variant::VOID type.

use new canvas colors header as necessary

move canvas colors code into its own file(s)

remove TimeAxisViewItem::RIGHT_EDGE_SHIFT; fix up x-coordinate for right edge of an audio fade out

simplify removal of outline from streamview base rect

simplify removal of outline from automation time axis base rect

fix up dimensions of ghostregion colored rect

when drawing only part of a Canvas::Rectangle outline, we only need to do the cairo-pixel alignment math for the axis perpendicular to the edge

don't created vestigial frame canvas item for TimeAxisViewItem unless actually needed.

They don't cost very much but there's just really no point cluttering up the scene graph with generally invisible
and insensitive items

VOID is a windows define (not MSVC specific)

tentative fix for canvas background/image surface

another hack for windows timers, DSP load calculation

add API to extract color from ArdourCanvas::Text and tidy up member declarations

Conflicts:
libs/canvas/canvas/text.h

changing the clamp_width of an ArdourCanvas::Text needs to potentially provoke an entire redraw

This fixes missing range marker text in sessions where the initial zoom value causes an initial clamp_width of zero

possible fix for DSP load calculation on windows

raising/lowering canvas items should trigger a redraw

add a selection frame to TimeAxisViewItem along with a slew of other height/positional fixes

various positional fixes for TimeAxisView, including moving the separator line to the bottom, and keeping it in the same coordinate space as the TAV group

streamview no longer outlines any part of its canvas rect (we leave that to time axis view separators)

Also remove commented "- 1.0" adjustments that were already commented

correct the rendering of Rectangle outlines, and the computation of their bounding box

add Rect::shrink(Distance) even though it arguably should be handled by Rect::expand()

add some more outline enums to ArdourCanvas::Rectangle

allow derived classes to access Item::{scroll,position}_offset(), mostly for debugging purposes

catch up with waves/tracks version of MIDI scene changer code

alter implementation of ArdourCanvas::contrasting_text_color() to use gamma-adjusted luminance values rather than simple contrast

add ArdourCanvas::Fill::set_pattern() to allow any filled canvas item to be filled using an arbitrary Cairo Pattern

fix insane design for Location::next_available_name() so that it has no effective limit (other than UINT32_MAX)

permit initial contents of a FloatingTextEntry

Add some newly introduced header files to our MSVC project (libs/ardour)

Add some newly introduced source files to our MSVC project (gtk2_ardour)

When building with MSVC undef type VOID if we're declaring an enum called VOID

Modify 'msvc32-fixup.pl' for building the VST scanner app resources

When building with MSVC, guard a declaration of 'strdup()' which conflicts slightly with the MSVC declaration

(maybe we should just #include <string.h> ?)

Remove an unneeded icon when building the VST scanner app with MSVC

When running an (MSVC built) VST scanner app, make sure we can utilise debuggable versions if we need to

Add some new modules, needed for building the VST scanner app with MSVC

(gtk2_ardour) Remove the reliance on libfst, which we no longer build

(fst's functionality has now mostly been moved into libardour)

use ArdourCanvas::TimeRectangle for regions, notes, markers