Factor out copy-paste code.

Remove unused method.

Replace half-baked param metadata with descriptor.

Among other things, this means that automation controls/lists have the actual
min/max/normal/toggled of parameters, and not those inferred from the Parameter
ID, which is not correct for things like plugin parameters.

Pushing things down to the Evoral::ParmeterDescriptor may be useful in the
future to have lists do smarter things based on parameter range, but currently
I have just pushed down the above-mentioned currently used attributes.

Make it compile with C++11 support.
Reference : https://bugs.webkit.org/show_bug.cgi?id=59249

Relax VAMP version from 2.4 to 2.1

fix linux installer:

 * allow to override install dir with environment variable.
 * create /opt if if it does not exist and INSTALL_DEST_BASE env is unset.

Tidy up comments in Evoral::coverage

Remove old (already #if 0'ed) implementation of Evoral::coverage() and its
comments.

Tidy up the comment enumerating all the possible ways in which two ranges
can overlap, note the Evoral::OverlapType corresponding to each one, and add
comments to the if()s in coverage corresponding to the cases in the list of
overlap types.

Remove some commented-out assert()s that actually do happen, and re-instate
one that really shouldn't.

Fix a small typo (with -> within)

Simplify Evoral::RangeList::subtract(), and make it pass amended tests

The various conditionals in Evoral::RangeList::subtract() appear to have
been there to work around
 (a) coverage() not always returning the correct value, &
 (b) the test suite assuming that the ->to point lies outside the range

Now that these are both fixed, the implementation of subtract() becomes
quite a bit clearer. I replaced the if()s with assert()s for now, but these
shouldn't trip if coverage() is working as I expect.

Also (attempt to) clarify the comments in subtract.

Comments in various call-sites of coverage()

Comments in various call sites of Evoral::coverage() marking things I think
are dubious (with XXX). Also straightened up the alignment of some ASCII
art in libs/ardour/diskstream.cc

Add comment about parameters to coverage() including the end point.

Rework Evoral::coverage() to pass unit tests

Rewrite Evoral::coverage() to (hopefully) do what it's supposed to.

Return OverlapNone for invalid ranges: if either of the ranges passed to
Evoral::coverage() have negative length (i.e.  start > end), return OverlapNone
- it seems reasonable to say that a negative-length range can't overlap
anything. Also return OverlapNone from the fallthrough case, though this should
never happen.

Make tests of Evoral::Range functions include both endpoints in Range

Some of the tests for Evoral::RangeList::subtract() assume that ranges
don't contain their end (->to) point. This appears inconsistent with how
they are used elsewhere.

Add some ASCII art comments to the tests to try to clarify what they're
really testing for, and amend subtractTest1, subtractTest4, & subtractTest5
to incorporate the assumption that ranges include their end points.

exclude fader-reserve from range

Remove braindead Parameter inheritance abuse.

Replace thinning static state with parameter.

Remove dead code.

Remove bogus test and noisy output.

Fix test runner for tests that use files.

Trim the include tree.

Move EventRingBuffer to libardour.

This is not used anywhere in Evoral and is just a wrapper around the PBD
RingBuffer anyway.  Towards a (once again?) independently buildable/testable
Evoral and fewer cross-dependencies.

Add missing namespace qualifiers.

Clean up state tracking of raw MIDI.

Factor out duplicated track automation code.

Mute automation for MIDI tracks.

Save mute automation state.

update interface for processor-box faders step/page

see also 4d4c20b8c

CLA approved consistent fine-grained scale modifier

Process 'default_ui_config.in' when building with MSVC

gtk interface update for controller step/page size

consistent step+page sizes for fader+spinbox

use update_steps() instead of custom function

update_steps(), properly handle integer ranges

fix ladpsa default value.

because get_parameter_descriptor() is const, also wrap
default_value() in a const function.

fix LV2 default port value.

Fix write for boolean automation.

don't move faders on click.

for one, it can mess up the value when switching to
numeric-entry. It is also rather useless and not
Fader-like (faders are not scrollbars).
Besides, "stepvalue" is way to large and causes jumps
(sometimes step-value is even negative, see
"fast lookahead limiter", "release time"
 -> some other bug)

show values on faders (fixes bug in 55c6165)

change ownership of processor window-proxy

fixes crashes:
* If the Editor-Mixer shows a channel with a plugin that
has been edited in the Mixer, double-clicking the plugin
will try to bring up a 2nd instance of the plugin-UI.
* When closing Ardour both the Mixer and the Editor-Mixer try to delete the underlying plugin, resulting in a double free.

Set discrete by default for boolean parameters.

Somewhat working gain automation.

This probably isn't correct in several ways, but it works more than it did, so
I figure it's push worthy.

Still not working:

 * Saving mute automation list
 * Dragged control points are not snapped to model restrictions
   (boolean, in this case, but general problem)
 * Line goes funny if you record mute automation
   (as opposed to drawing it which works)

Show discrete/toggled automation as stepped line.

Fix crash when deleting all points from a line.

Use a button as controller for toggled parameters.

Remove redundant calls to start/end_touch.

The controllers do this automatically.

Set the toggled default metadata of parameters.

This is so generic code looking at the Parameter for gain, solo, and
rec-enable will get the correct value for toggled.

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

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

change extention of linux installers to .run

Remove dead code.

Fix calculation using uninitialized value.

Implicit "this" is the devil.

remove "port sources" management from MidiControlUI and Mackie support.

    This was necessary due to a bug/design issue between Glibmm and Glib (see https://bugzilla.gnome.org/show_bug.cgi?id=561885)
 but the problem needs to be managed by the *creator* of the IOSource and that has now moved inside CrossThreadChannel.

variable name change for slightly increased clarity

get correct ::distance() and ::delta() implementations for cases involving achromatics

continue tweaks on the color road

newly generated, quantized-hue relative color definitions

add back grays

add default values for base colors (configurable colors) to default_ui_config.in

save base colors (configurable colors) in normal way (code had been removed)

massive reworking of color selection implementation

move creation of UIConfiguration object to precede creation of ArdourUI.

This is pretty kludgy but we need the UIConfiguration object (or any color management object) to be the first to connect to
the ColorsChanged signal.

Might be better to make the UIConfiguration object a delegate for the ThemeManager.

ensure that Ardour recognizes its own MMC device ID as "all-call" when set to 0x7f (127)

Replace individual waf options for enabling backends with one --with-backends option

With printf() / snprintf() etc "%f" actually requests a double, not a float

Let's pass the correct type, since we can't rely on these functions to do intelligent conversion

Add test for Evoral::coverage()

Add a test function to test Evoral::coverage() with all possible overlap
types. The first test (line 161) that expects OverlapExternal will fail
with the current implementation of coverage().

There's possibly still a discussion to be had about what the overlap type of
ranges with negative lengths should be: there are currently places in the main
Ardour code base where coverage() is called with ranges where start > end.

Fix build of evoral tests

Fix compile errors in libs/evoral/test/, by explicitly calling
Evoral::MusicalTime::to_double() wherever a double value is required of a
MusicalTime.

Some of the double variables should probably really be made into MusicalTime
ones instead, but I don't want to mess with this too much.

takeFiveTest still fails for me after this, but a failing test is probably
more informative in the long run than a test that won't even compile.

Fix order of parameters to stop_touch() in Mackie::Control

Reverse the parameters of Mackie::Control::stop_touch() to make them
consistent with AutomationControl::stop_touch(), and fix up the call to
AutomationControl::stop_touch() to have the parameters in the correct
order.

Unfortunately, I don't possess any devices that speak the Mackie protocol, so
though the patch seems logical and correct to me, I have no way of testing it.
If anyone has a device with touch faders that speaks Mackie, I'd be glad of any
confirmation that it at least doesn't break anything.

Revert "Don't add non-clang flag regardless of platform."

Apple's g++ on 10.6. does not understand -Wno-unused-local-typedefs
either

This reverts commit 9dd8024e41f35e581b3db5f4a873126398db9d45.

fix indentation nightmare from 9559cf3947

fix case of included header file

[Summary] Fixing behavior of the FloatingTextEntry

quick sanity check

NOOP, fix some “missing sentinel” -Wformat

Check for clang with an actual code check.

Turns out Apple in its infinite wisdom installs clang as /usr/bin/g++, so
guessing based on executable name doesn't work.

Remove dead code.

Shup up clang about gtkmm/papertraits.h.

Don't add non-clang flag regardless of platform.

Fix clang check for versioned clang executables.

when dragging on the canvas, use x,y pointer coordinates to decide if motion has occured.

Using _last_pointer_frame breaks when dragging to the left of the canvas, because we clamp
the value of the frame to >= 0. Motion would step once the pointer crossed the left edge
of the canvas because the frame value would always be zero.

This is not a problem when using the pointer x,y values which end up appropriately negative
under all conditions.

fix OSX open URL in browser

Add a newly introduced header file to our MSVC project (gtk2_ardour)

Now that we're building pango's language modules statically (i.e. no longer as DLLs) it's not necessary to edit the user's 'pango.modules' file

Simulate 'trunc()' which isn't available in MSVC

(needed by 'gtk2_ardour/editor.cc')

Remove a commented out line in 'pbd/stacktrace.h'

(was probably just needed for testing at some stage)

Changes needed to accommodate building libevoral with MSVC

Note that class 'MusicalTime' needn't be exportable (with a little work, this could be reverted if it affects the other builds)

waf is fun

OSX gcc does not support local-typedef warning

also search for 32bit xjadeo on 64bit windows

amend c4533939b9

lost quote

hotfix windows-compilation (unfinished bfe49540b9)

..at least it compiles and starts. Mackie support etc
is probably still broken.

update harvid & ffmpeg binary path discovery on windows (64 & 32 bit)

make debug message more useful

fix/silence various compiler warnings.

* ifdef unused static functions
* brackets around assignment and comparision
* no return statement in function returning non-void
* boost concept_checks.hpp unused-local-typedefs

Fix build with a certain awful broken compiler.

-Wunused-function on windows

use NULL not 0 for c-functions (even in c++ code)

4.3 KBytes less compiler warnings.

remove use of CrossThreadChannel::selectable() to allow cross-platform implementation of CrossThreadChannel via Glib::IOSource

assign minsec_ruler_scale and minsec_mark_modulo when zoomed very far out

Avoids crash when computing % minsec_mark_modulo.

Fix quantization and other time-related ops.

Max is max and min is min and I am dumb.

Fix note resizing.