When building with MSVC add a couple of new pragmas to cut down on the most frequent compiler warnings, namely:-

1) C4244 (conversion loses data) - e.g. when re-assigning a var of (say) int64_t to a lesser type, such as int32_t

2) C4275 (an exported class was derived from a class which is not exportable) - e.g. when deriving classes from std::set, std::exception or some other class which isn't declared with 'dllexport'

When building with MSVC, follow Robin's lead and add support for a new env var (SUIL_MODULE_DIR)

win-pkg, ignore missing suil modules

add accessor methods for single_exposure

_single_exposure is now a member variable for each GtkCanvas.
Gtk coalesces multiple exposes into a single combined rect.
If _single_exposure is disabled, we break apart the individual expose rects for the canvas rendering.

windows: deploy suil modules

Fix whitespace in pbd/debug.h

Add local PwdReset class to reset pwd in event of test failure

This fixes subsequent tests that rely on pwd but not PBD::path_is_within
breakage caused by commit 2689848e

Add some more test assertions so that tests fail when they should

remove naive attempt to avoid drawing first vertical pixel line of Canvas::WaveView

And add commented out attempt at more subtle attempt to get it right

remove use of TimeRectangle

simplify Canvas::Rectangle rendering to avoid unnecessary nonsense, and remove TimeRectangle

Canvas::Rect::contains() should treat its right/left coordinates as exclusive

When setting the duration of a TimeAxisViewItem, do not round before computing pixel width

Editor::sample_to_pixel() always returns an integer value, but it should use explicit rounding

fix midi-capture filter

diskstream reads directly from port, Route
use prefilled buffer-set.

use magic strings instead of magic numbers (allow profiles to leave out some track type options)

desensitize redirect-menu Send and Control items when they are not applicable

properly activating MIDI parsing for traced ports

infrastructure to allow tracing of all MIDI ports

skip drawing the first pixel of waves in an audio region view.

This means that they do no overwrite the frame border of the time axis item

add a sort-of hack to allow us to avoid drawing the first pixel of a waveview when necessary

fix previous commit

Track Mode is too vague; rename to Record Mode

flip the priority of input and output buttons and menus
left-click should show the simple input picker menu.
right-click shows the routing grid.
also reordered the menu to give lower precedence to {add/remove}-port

fix boot message always sticking on the last loaded route; it looked like an error

fixed collision with <SECONDARY>n binding

correct drawing of rectangle borders.

They need to use fractional coordinates, and the border position needed
generalizing for other border widths. See verbose comment for details

canvas items must be able to use fractional positions when rendering.

This is required to be able to draw precise single pixel lines, as described
in the Cairo FAQ

fix last cherry-pick

fix color and alpha of timestretch rect

put function into namespace

Fix sending out of bounds events to LV2 plugins.

After dragging from multiple tracks to the dropzone, create the right number of new tracks.

This does not address the visual flattening that occurs before the drop
is complete. Doing that is complex and there is no immediate solution
visible. The result after the drop is complete is correct, however.

Conflicts:
gtk2_ardour/editor_drag.cc

Properly deallocate memory when clearing selection_op_history and
before XMLNode*s.
Improve some comments.

fix 192k rate

when the user changes devices, notify the backend

fix (and comment) on subtle bug with audio file data width function

Make undo/redo work for:
undo (n) where n > 1
redo (m) where m < n
new transaction.

Previously the redo list was left untouched.
This would lead to utter nonsense in the redo list.

AFAICT this never worked.

fixup cherry-picked changes from Waves Tracks, and fix for Ardour

implement RangeMarkerBarDrag::abort

Conflicts:
gtk2_ardour/editor_drag.cc

implement MarkerDrag::abort()

do not attempt to call an empty/null boost functor in clear_events()

fix issue with region dragging while mouse pointer is in rulers/marker lanes

fix bad fix for cherry-pick conflict

make two obvously const methods actually be const

modify behaviour of session when updating skips to use new SessionEvent API

Conflicts:
libs/ardour/ardour/session.h
libs/ardour/session.cc

add SessionEvent::Action to libs/ardour enums registration

expand SessionEvent API to allow ::clear_events() to work correctly.

clear_events() must run in realtime context, which is likely to be asynchronous
with respect to the thread that calls it. So allow caller to pass in a functor
that will be executed (also in realtime context) after the clear is done.

Additionally, allow for a cross-thread callback to the event loop/thread which
initiated/allocated the clear event request so that it can flush its own pending
loop. This part probably isn't necessary but doesn't hurt and is a useful model.
The event would be placed back in the free list at the next event allocation
by the calling thread anyway.

expand PBD::Pool API and add additional DEBUG_TRACE output.

Expanded API splits apart some CrossThreadPool functionality, and provides
access to current pool status information (available(), total(), used(), pending_size())

set name of GUI event loop thread

ContentEdit drag on empty MIDI track should not create new midi regions

Rework previous commit to avoid extra property lookup (fwiw).

Make sure we have the property rather than the value of patch change id.
(don't crash whenever someone adds a patch change and reloads the session).

Clean up Session's _current_trans when aborting a drag.

Make redo action insensitive after new command.
Don't erroneously send first_move more than once after autoscroll drag.

remove duplicate callback

mcontrol->Changed is [also] subscribed to further below (input & output).

tweak to help with uncoalesced renders

render canvas using the GDK region rather than the GDK area.

The region is the un-coalesced set of rectangles that were requested for redraw. The area
is the coalesced single rectangle. In the worst cases, the coalesced rectangle could span
the entire window even though just two pixels in opposite corners were to be redrawn.

There is a problem with the verbose cursor as it is dragged across MIDI tracks. TO BE
FIXED.

Add an extra test to prevent 'PBD::path_is_within()' from looping infinitely on Windows

The infinite loop would happen if the 2 supplied paths were on different Windows drives - for example if one was on drive C:\ and the other on drive E:\

I don't think this new test will be detrimental to the other platforms but if it is, we could easily separate it out with a '#ifdef PLATFORM_WINDOWS' directive.

OSX packaging: fix disk-image "Resource busy"

see http://lists.apple.com/archives/darwin-dev/2008/Sep/msg00030.html

Remove remaining LEVEL4 bindings

Fix Session Export keybinding to call correct action (ExportAudio).
Still broken though, due to ongoing Mod4><Super binding confusion.

make it possible to bundle harrison plugins; osx version (disabled for now)

fix cursor color for text entries

remove locate before loop, which was causing intermittent loop behavior

use Gtkmm2ext::possibly_translate_mod_to_make_legal_accelerator() in all cases where we use gtk_accel_groups_activate()

Because that function needs it, whereas the old gtk_window_activate_key() did not.

If 'force' is TRUE (when using 'Location::set_start()' or 'Location::set_end()') make sure we do actually force the value to get set.

This gets us around a problem when converting a session from the old (Ardour2) format - where the Session Range (start) value was getting incorrectly set if we hadn't already set the end value.

fix “invalid use of incomplete type jack_client_t”

introduced in 66eef25

clarify sync-lock and disable it by default.

amend 070818f

stop transport when process_routes fails, even when using the parallel graph execution model

fix incorrect return value from sndfile_data_width()

take down backend specific threads with backend

fix jack thread joining

at the time the graph gets around to takes down
client threads, the jack-backend’s jack_client has been reset.
But never mind: libjack does not care about it, anyway.

return to old default disk chunk read size

don't wipe out SHIFT from event state when processing keyboard events, as done in a recent commit

in mixbus, treat the canvas rulers just like the canvas wrt mouse-wheel-zoom

consolidate using SECONDARY+number bindings for region properties, such as gain, mute, etc

switch WINDOW bindings to use SECONDARY instead and displace any bindings that conflicted

fix up and comment fmt-bindings tool

use gtk_accel_groups_active() in preference to gtk_window_activate_key() since the latter behaves incorrectly

The window version will not activate bindings like <Mod4><Super>-space, whereas the former one works correctly. We already used
the former one just above, when handling "fakekeys".

Also add a utility function to show modifier state.

fix -Wreorder in 6377fe89

allow bundling of lv2core

Don’t statically initiate the lv2 world, use explicit call after
scanning bundles.

lilv_world_load_specifications() and lilv_world_load_plugin_classes()
are only ever called after lilv_world_load_all(), so we postpone
the call to it.

Map window bindings to alt.

As they were before, got caught in the crossfire of super key bindings fix.

guess what?!

add lv2core to bundles

The ontology is needed for plugin classification.

readtest: calculate running average and stddev

read-test format directly for gnuplot

ALSA-backend separate report for error and x-run

ALSA-Backend prefer ppoll()

fix #6117 - fader prelight

towards fixing #5711

Don’t call ::output() [here: SilenceTrimmer::process()]
with no data to process.

If (position + N * period-size) % chunksize == 0;
frames_left == 0 before the last call to ::output().
chunker.h:60 keeps the ProcessContext<T>::EndOfInput
flag and the SilenceTrimmer will already have done ‘in_end’ processing.

[Summary] Adding missing initialization of the date &time formats. “Init trashes” caused a bit ruined logic of the export dialog.

fix printf format-security warning.

 warning: format string is not a string literal (potentially insecure) [-Wformat-security]

Don't try to dereference a NULL ptr (the function 'ARDOUR::IO::nth()' can return an invalid ptr, so let's make sure we test for it)

Note 1: Potentially we could have the same issue with 'AutomationLine::nth()' (although the current code seems okay)

Note 2: This (specific) problem seems mostly to affect Mixbus3 when it tries to load an old (version2) session from Ardour2. Mixbus can mark certain IO ports as being 'unused'. While loading sessions, function 'ARDOUR::IO::ensure_ports_locked()' deletes any unused ports. But later, the function 'ARDOUR::IO::make_connections_2X()' was crashing while trying to connect those (now NULL) ports. This commit fixes that crash but there's some evidence that it might only have moved the problem elsewhere. The version2 sessions now open successfully - but an MSVC (Debug) build now crashes whilst closing them (the Release build however, is okay). Someone who's more familiar with the code than me should review the Mixbus3/Ardour2 loading procedure to make sure it's safe.

ignore context menu click events, notably during drag but also for a couple of other purposes

Conflicts:
gtk2_ardour/editor_canvas_events.cc

remove unnecessary commented-out line.

Session is marked as in-deletion during deletion itself

on OS X, do not require click to activate the application

amend e3da18fd