remove commented code

fix logic of RouteUI::set_color_from_route()

fix logic of PresentationInfo::color_set()

remove gtk_entry_* colors from themes except for cursor color

MotionFeedback vanished long ago

drop use of gtk_entry_* colors (except cursor); use original base/text choices but avoid using selected for entire bg

add the potential for a bit more debugging of signal connects

add the potential for a bit more debugging of signal connects

minor no-op hacks (unneeded or suboptimal code)

fix jack_port_rename() detection

some debug message for lua processors

fix lua filters for multi-channels processing

add (essentially random) color defs for text entries to other 3rd party themes

NO-OP: remove trailing whitespace

assign appropriate styles for text entry and spinbuttons

SpinButtons do odd things to draw the two arrow buttons. So for now, leave them with the default theme,
even though this makes their text entry component different from the other text entry widgets

remove non-zero xthickness from "small_entry" widgets (gain display, peak display in mixer strip).

Unnecessary and wrong once you can see the border in a different color from the base

new style for Gtk::Entry widgets

More legibility and easier to figure out what colors are used for what

use a more appropriate @color name for the cursor color in a text entry

add text entry cursor color to dark theme

add new colors to dark theme

no reason to make floating text entry be in STATE_SELECTED when constructed

use initialization, not explicit assignment

OSC: Select tweak.

OSC: Reworked select to follow Gui selected strip regardless of it's inclusion in bank

Revert "remove unnecessary duplicate loading of GTK RC file"

This reverts commit 694d805bb0dd7531a58caa7cf617b8ea117e7588.

fix color file install paths

We are supposed to remove the application name after copying

fix color file install paths

We are supposed to remove the application name after copying

fix cursor set path during linux build

consistent mixer sidebar layout (group, tracks  show/hide)

 * move checkbox to the left (unaffected by scrollbars)
 * use same expand & sizing strategy for both

remove unnecessary duplicate loading of GTK RC file

remove debug output and add a few suggestive comments

changes in logic used by source cleanup to avoid endless recursion in sessions with deeply nested/recursive compound regions.

This also fixes some potentially dangerous cleanup logic related to two sources with the same name (but different paths)

ignore automation for non-automatables.

Since 4.7-1477-g2c6d595 Ardour implements Control::NotAutomatable.
This can lead to situations where automation exists for parameters that
were not supposed to be automatable, which in turn caused a
"programming-error" abort() in
RouteTimeAxisView::add_processor_automation_curve()

some more translation fixes

fix german translation

'msgid' and 'msgstr' entries do not both end with '\n'

tweak b7e645ec6, use session start/end position

quick prototype to allow changing *all* gain automation

switching to "touch" implies existing automation.

This fixes an issue of Fader jumping to unity when switching from
manual to touch if there is no prior automation.
Likewise for plugin parameters jumping to "default"

add a debug mode (in synth) to track down lost note-on

BSD portability

Adopted from Michael Beer -- GH pull-request #232

Enable build for FreeBSD (part 1/2)

Adopted from Michael Beer -- GH pull-request #232 with minor changes:

*   rebased on master,
*   removed trailing whitespace,
*   don't explicitly change saved configuration defaults (wscript)
*   moved sys/wait (WNOHANG) to header include
*   separate changes in GUI and lib

Use correct type of std::map::count

Compiling Ardour commit ec8a4de01596c162c1529f3021dfc432bf66dfe8 with GCC
6.1.1 (on Fedora 24) gave this build failure:

    In file included from /home/sam/ardour/libs/lua/LuaBridge/LuaBridge.h:154:0,
                     from ../tools/luadevel/devel.cc:16:
    /home/sam/ardour/libs/lua/LuaBridge/detail/Namespace.h: In instantiation of ‘luabridge::Namespace::Class<std::map<K, V> > luabridge::Namespace::beginStdMap(const char*) [with K = std::__cxx11::basic_string<char>; V = std::__cxx11::basic_string<char>]’:
    ../tools/luadevel/devel.cc:89:60:   required from here
    /home/sam/ardour/libs/lua/LuaBridge/detail/Namespace.h:1666:30: error: no matches converting function ‘count’ to type ‘void (class std::map<std::__cxx11::basic_string<char>, std::__cxx11::basic_string<char>, std::less<std::__cxx11::basic_string<char> >, std::allocator<std::pair<const std::__cxx11::basic_string<char>, std::__cxx11::basic_string<char> > > >::*)()’
           .addFunction ("count", (void (LT::*)())&LT::count)
                                  ^~~~~~~~~~~~~~~~~~~
    In file included from /usr/include/c++/6.1.1/map:61:0,
                     from /home/sam/ardour/libs/lua/LuaBridge/LuaBridge.h:45,
                     from ../tools/luadevel/devel.cc:16:
    /usr/include/c++/6.1.1/bits/stl_map.h:1131:2: note: candidates are: template<class _Kt> decltype (((const std::map<_Key, _Tp, _Compare, _Alloc>*)this)->std::map<_Key, _Tp, _Compare, _Alloc>::_M_t._M_count_tr(__x)) std::map<_Key, _Tp, _Compare, _Alloc>::count(const _Kt&) const [with _Kt = _Kt; _Key = std::__cxx11::basic_string<char>; _Tp = std::__cxx11::basic_string<char>; _Compare = std::less<std::__cxx11::basic_string<char> >; _Alloc = std::allocator<std::pair<const std::__cxx11::basic_string<char>, std::__cxx11::basic_string<char> > >]
      count(const _Kt& __x) const -> decltype(_M_t._M_count_tr(__x))
      ^~~~~
    /usr/include/c++/6.1.1/bits/stl_map.h:1125:7: note:                 std::map<_Key, _Tp, _Compare, _Alloc>::size_type std::map<_Key, _Tp, _Compare, _Alloc>::count(const key_type&) const [with _Key = std::__cxx11::basic_string<char>; _Tp = std::__cxx11::basic_string<char>; _Compare = std::less<std::__cxx11::basic_string<char> >; _Alloc = std::allocator<std::pair<const std::__cxx11::basic_string<char>, std::__cxx11::basic_string<char> > >; std::map<_Key, _Tp, _Compare, _Alloc>::size_type = long unsigned int; std::map<_Key, _Tp, _Compare, _Alloc>::key_type = std::__cxx11::basic_string<char>]
           count(const key_type& __x) const
           ^~~~~

Casting std::map::count to the correct type instead of a fake void()
type fixes the compile failure.

fix uninstaller for variants (Mixbus32C vs Mixbus)

fix typo

fix typos in prev commit

major internal plugin & processor API change:

Pass current (latency compensated) cycle times to plugin.
This fixes time-reporting to plugins and also fixes automation
and when bouncing (the session->transport* is not valid) etc.

fix vamp path:

* append, not replace
* make it work on wine/ardour

fix KeyEditor::print() for windows (hopefully, untested)

Fix counting thinko in tests.

- note that the negative framepos in
  FrameposPlusBeatsTest::singleTempoTest() will
  not pass. needs some thougt as to whether it is testing
  the correct thing.

Ignore numerals before "+" in tag

When building with MSVC, adapt the various font sizes to be a closer match to the GCC build

(the smaller fonts can't be an exact match or they end up looking too small).

fix ampersands in group names in generated binding files

include group descriptions when processing keybindings

more tweaks to HTML output when printing key bindings

show tooltips on favorite plugins -- closes #6902

OSC: master/monitor forced to sort at end

OSC: Fix math off by one, Check for all stripable controls because VCAs have less (crash fix)

another C++98 fix

fix gcc4 builds C++98

Flag is an enum -- not a class nor namespace

OSC: Might help if I used the right variable, Really fix select.

OSC: Fix out of range track. (crash)

OSC: Add strip type selection.

fix some more thinkos related to user color file management

remove debug output

remove old file

rename unastudia theme file to match per-program-name convention

fixes for color theme manager's handling of color file choices, notably while running from source tree

tweaks to how UIConfiguration::color_file_name() operates

fix default color file name in code default definition

remove unneeded declaration

fix incorrect color file name in default UI config file

use running_from_source_tree()

use running_from_source_tree()

provide ARDOUR_UI_UTILS::running_from_source_tree()

always run amp/trim LPF

This fixes "clicks" when switching fader automation from "manual"
to "play/touch" while the transport is rolling.

add comment button fill color alias for *all* themes

add tempo curve colors to unastudia theme.

Colors need adjusting by a theme owner/maintainer

add shuttle colors to unastudia theme.

Colors need adjusting by a theme owner/maintainer

new "studio one" theme from user straightothebar

show first word of comment on track's comment-button

use new keyboard modifier names, and replace GDK names with better versions

fix section naming in bindings file

provide some Keyboard:: methods to get access to other versions of modifier names

some modest improvements in the html rendering of key bindings, plus use of normal modifier names

fix comment-button highlight

Remove confusion over ambiguous symbols

libboost and libsigc++ both contain symbols called _1() / _2() etc. Forcing the use of namespace sigc causes us to use the ones from sigc++ when in fact, we want the ones from boost.

Hopefully this fix will work for all compilers.

Make sure MSVC knows which version of 'floor()' we want

Use Glib::usleep() for consistency

fix upsampling import of X-channel files where buffersize % X != 0

OSC: take care of track order changes too. Deal with mixer topology changes one time.

OSC: Blank some more feedback values when select observer is destroyed.

set latency of sidechain port

initial working version of "printing" keybindings (via a browser)

Register colour aliases for tempo curve.

no divider/full allocation of LHS of mixer "display" with no VCAs

avoid crash after removing a VCA

no debug output, either

no pingbacks if running a development version

mark mixer strip name label (button) as having occasional text

Sometimes it shows a track number, sometimes not, but this should never affect its size request

remove commented line