Fix likely thinko in bbt ruler.
This fixes a bug for me:
  Measure lines appear & disappear during playback, at certain zoom scales.
  I think this is because the unecessary offset was causing beat_density to vary unnecessarily while rolling.

compute_bbt_ruler_scale should not be necessary for every redraw of the timeline.  Works OK here without it...

fix incorrect property change in AudioRegion::set_fade_out()

- should fix 7283

Fix crash when removing time from track with automation

Resolves : #7287

Faderport uses the Mix and Proj buttons to summon mixer and editor; there is no longer a Toggle.

fix potential crash in LV2Plugin::connect_and_run() when music starts after 0

Allow Edit -> Crop to work with edit point set to mouse

Crop is currently only dependent on having a valid time/range selection.

Resolves : #7278

Allow Separate using Loop/Punch range to work with edit point set to mouse

These operations are not dependent on the mouse position.

Resolves: #7276

fix a-reverb, cut tail on deactivate, plug memory-leak

Consolidate code

The return of the Mixbus Send-options (allow aux-feedback override)

Don't show Group UI for the master-bus

Prevent adding "master" to a group

Mixer sidebar: show group context menu on right-click only

Pop down dialog before re-scanning VSTs

VST UI sizeing rework

* prefer signal over timer-poll for VST size changes
* properly offset VST window (Windows)
* constrain window size (Windows)
* explicitly notify re-parent'ed child window (Linux)

Prepare moving to a dedicated x-thread signal for VST UI resizing

(this changes the internal API, pending follow-up GUI update)

remove cruft

Fix copy/paste typo -- MacOSX builds

snapped pencil drawing of new midi regions cannot be zero (7271).

- fixes a couple of other oddities in the behaviour.

two new action scripts (no icons yet)

Rework VST initialization:

Set the AEffect* plugin pointer before calling effOpen.
effOpen may call back into the host (masterCallback) and invoke
actions which depend on _plugin (eg. to call back into the plugin again)

allow setting end bpm in new tempo from tempo dialog.

rework tempo dialog formatting, add end bpm entry.

fix crash when dragging region from list.

Fix save-as progress dialog.

ScopedConnection and label,bar went out of scope.

announce what changed correctly in MidiRegion::model_shifted()

more linked midi region trim drag fixes.

- freeze the correct playlists on fiorst move.

- only update the start offset of linked regions once the drag
  has finished.

More fun with LinuxVSTs & threading

Add support for VST effSetBypass #7266

This mechanism re-uses internal abstraction (plugin-enable).
Other plugin-specs designate a control-port, for VST we 'emulate'
a control-port.

There is still debug-output to stderr, here while testing.

Close the LXVST X11 connection for good measure

Work around a crash at exit for Linux VST UIs abusing pthread-keys.

Fix AFL/PFL from MIDI tracks without audio (zero buffers)

Revert "some further debugging, just in case"

This reverts commit 04ed05ecc4629d85d8f1fd3d6756669a42fb762b.

some further debugging, just in case

Prevent duplicate self-delete.

debug-prints indicat it was at least possible for the same Window to
receive  FloatingTextEntry::entry_focus_out twice (without idle)

NO-OP: whitespace

ensure no movement when draggin regions with x constrained.

- should fix 7260

fix undo if a midi region start trim changed the source offset.

- this diffs all playlists that use the same source.

- the secons part hould be a no-op, but needs testing
  (trim drag undo on audio and midi regions)

Remove space between _ and (

fix uninitialised variable (fixes linked midi region negative start oatch)

no-op - comment/whitespace fixes.

tempo editing updates

- a tempo marker may now be set to always continue (clamped)
  this means that the end tempo of the previous section will
  track the start tempo during tempo ops.
  it mimics the behaviour in 5.8, with the gui indicating
  the curves to be changed.

Notify UI when plugin state is copied/replaced

Don't allow GMCP to override loading_state_version

- GMPC does not use it directly
- when GMPC was enabled Stateful::loading_state_version was set to 1000
  (the protocol is at 1.0.0)
- it messes up session-loading, particularly various plugin states and
  templates (e.g  Processor::set_state_2X was used)

undoing a create note drag restores screen position.

rework percussive hit spraying.

- the spray is limited to the first entered note.

- all new notes are selected (requires a selection change to
  MRV::create_note_at()

spraying percussive hits has one undo step per drag.

- should fix 7134

remove bogus variable from last commit

Add reset region gain action for selected regions

offset linked regions to compensate for negative start after trim drag.

- should fix 7105

Add 7890 aka tom- to contributor list

Fix crash when inserting time with split intersected regions

Don't try to auto-start engine for new sessions

Work around logic-flow in EngineDialog which should already have
done that, but may no longer heed this special-case if sample-rate
dropdown is refreshed to only devices supported ones.

Ship system_config with bundles again.

This partially reverts commit ac40945ff96f27606ff5d3600cf06b0ba663a5e1.
re-add system_config, but not instant.xml

Empty system_config (pre-seed XML nodes, use built-in settings)

Hotfix work around:
 1) Launch without existing user-config and no system_config
 2) Load or create session.  save/quit.
 3) re-load session. save.   user-config file is corrupted in <Extra>

ArdourButton: if _act_on_release is true, only trigger actions/signal clicks on key release, and vice versa

This should really be split in separate semantics for key and button events.

Fixes a subtle but nasty bug in the EngineDialog where the change of the default focus
widget from a Gtk::Button (acts on press) to an ArdourButton (acts on release) caused
events occuring after a dialog has grabbed focus to trigger button clicks

Fix crash when adding a plugin immediately after re-order (via script)

The re-order is queued to be performed click-free in realtime-context.
Meanwhile adding plugins will result in a race-condition.

Don't automatically open generic UIs for plugins w/o controls

Don't display "channel-selector" for plugins only pretending to be instruments

If a plugin has 0 audio-outputs, and has the meta-data "InstrumentPlugin",
Ardour would interpret is as "variable number of output synth.

Fix audition with monitor-section.

When a monitor-section is added, the auditioner automatically connects to
its input. However on session-reload, the connection is explicit
(by saved port-name) and  Auditioner::needs_monitor() was not set.
Session::process_audition() didn't run the monitor-route. Silence.

Fix implicit selection when operating track-header ctrl buttons

Stop event propagation. Otherwise, on release the event is passed
up and handled by the TAV which scrolls the editor viewpoint
and the button is no longer under the mouse.

c89 compat (gcc4, mingw..)

Tweak/optimize VST callback:

The audioMasterAutomate callback from plugin to host does include
the parameter-value.

Previously there was a redundant call
Plugin::parameter_changed_externally() -> get_parameter
-> VSTPlugin::get_parameter()   back into the plugin to query the value.

calling back into the plugin from the callback, may explain
oddities and/or crashes with some VSTs.

when destroying a MidiSource, invalidate any existing MidiCursors

(such as those held in a MidiPlaylist's RegionTrackers

initialize some uninitialized vars

Fix loading SMF meta-data > 127 bytes (no more g_critical abort)

allow continuation of tempo via right-click 'Continue' where appropriate.

- this is the opposite of 'Ramp to Next'.
  it removes discontinuities between the
  last end tempo and the current by altering the current one.

Update GUI when playing mute-automation

Fix touching mute automation (mostly)

controllable->set_value() from GUI context will eventually queue
a session-rt command. By the time the realtime command runs, the
controllable needs to be in "touch" mode.

The AutomationController "toggle" UI worked around this by directly using
the underlying API (ignoring slaved-controls and ignoring groups).
The RouteUI's GUI wasn't able to write mute-automation at all.

This commit is a compromise: press + hold (touch) + release button.

(it may need further special-casing for _desc.toggled in
AutomationControl::actually_set_value(), also undoing automation-writes
is currently  not working correctly)

Likely fix for bug: cannot change track colors in older 4.x sessions.

probable fix for importing tempo maps that use non-quarter note pulse.

- actually i'm not certain that this is correct,
  but it works ok Beethoven's fifth.

_group, group.. who cares?

clicking on curve text does the right thing.

initial rework of tempo text display (numbers are curve items)

- move tempo numbers to a layer above the curve.

- the offset of the text box in Marker is horrible here.
  what to do?

fix crash in pinch drag

remove scroll wheel editing nonsense (tempo)

fix compilation

remove 'mouse:' verbose cursor when shift-dragging a curve.

only allow 'ramp to next' if the next tempo is actually different.

poor man's display of end tempo for ramps.

harden TempoMap::next_tempo_section () a bit.

Tempo Marks : set point colour by discontinuity, display non-quarter pulse if we are non-quarter.

- a jump in tempo by more than 1 ntpm results in a red
 tempo mark pointer.

- ignore UIConfiguration::get_allow_non_quarter_pulse()
  when displaying note type in the marker text (only display
  note type if we are non-quarter).

remove commented line

fix tempo marker ramp to next.

amend last commit (don't depend on 0.0)

hopefully fix legacy session loading (tempo)

add right-click 'Ramp to Next' tempo marker menu item.

back to using shift-ctrl for pinch drag. set colours of affected curves.

add TempoMap::next_tempo_section()

fix TempoMap::frame_at_tempo().

highlight the tempo curve that is to be altered, modify text to suit.

fix verbose cursor display when shift-dragging end tempo.

restore pinch tempo. it now operates via shift + alt drag on the curve.

remove tempo end drag (the control drag on the curve). a;ter tempo marker drag.

- holding down shift before initiating a tempo mark drag
  alters the end tempo of the previous one as before, but
  this is now a separate drag.
- restore vertical dragging of the tempo mark to alter start/
  end tempo. shift during the makrker drag alters start tempo.
  xontrol + shift during the drag alters end tempo.

remove "Make Ramped" from right click tempo marker menu.

fix formatting of bbtruler drag verbose cursor.

complete changes to tempo type.

- this implements in the intention behind the previous commit.
  a tempo mark is constant until its end has been changed by a
  shift-drag on the next marker.

compile fix

make new (ramped) tempi indidstiguishable from constant to the novice user.

- stretch drag (using shift on the curve) operates on
  the true meaning of constant. in other words,
  the only way you can make a tempo ramped is by adjusting the end
  tempo (shift drag on the *next* mark). from this point onwards.
  shift-drag the curve will change the start tempo and ctrl-drag
  the curve will change the end (needs work). to reset the
  curve to constant, 'make constant' should simply set the end
  tempo (unimplemented).