Further tweaks to .pending file removal

* only delete it after successful save
* show unlink notification in debug-builds

Prefer implicit deletion of .pending

Session destruction removes ".pending", no need to explicitly delete
the file.

Fix periodic backup saves

Retain ".pending" files until explicit save or session destruction.
Previously every transport-stop deleted them :(

Fix C++98 compat

Prefer "one track per file" as default

Don't print usage to stdout for invalid parameters

Tweak .clang-format

NO-OP: clang format, comments & whitespace

Remove unused EditorSources selection filter

Remove remnants of hack from 9f3f3738e7444ee3

This fixes multiple selection. The original code using
"selection_countdown" is no longer in place. There's no "counter" to arm.

Shift/Tab name edit works regardless.
"s/m/r" solo/mute/rec shortcuts in the treeview have been discontinued
since a long time as well. Keyboard shortcuts are per window, regardless
of grab_focus()

PS. This change makes it easier for a user to start a drag/drop
operation of multiple regions. That currently assert()s, only
the first selected region is used.

Fix transient removal -- #7785

Incorrect iterators were used for user-transients.

Remove unused EditorRoutes selection filter

Re-order editor sidebar: prioritize "Tracks & Busses"

(RegionList) Tweak column ordering + fix translation naming

Source List: Explicitly check whether a file is in the {Audio|MIDI} folder where it should be.
Anywhere else (even somewhere else inside the session folder) should be flagged "weird"

Source List: Source regions can be tagged, too.

Add Pref to write mixer-screenshot post-export

Consolidate: use PBD method to try hard-linking

Add PBD API to hard-link files

Allow export get_path() without format

This is in preparation for post-export screenshots:
A possibility to get a file-path, independent of the format.

NO-OP: whitespace

NO-OP: mark private variables, remove cruft

Initialize some more uninitialized variables

Prefer std::vector<> over C-style malloc/free**

This fixes a potential out of bounds read `if (_dbtp_plugin[c])`
if c is larger than the allocated channel count.

Initialize variable

Initialize variable

Fix middle-click note selection

Fix midi-note selection invert

Selecting Notes in a region does not select the region itself.

Invert-selection needs to iterate over midi-regions that contain
the selection.

(RegionList) Don't change selection for context-menu

Add Action to save mixer screenshot

Add Lua-Dialog support for FileChooserWidget (save file)

Add Lua bindings for convenient Glib methods

Fix auto capture alignment when bouncing metronome

Allow to inhibit screensaver/system sleep

Add types/enum for screensaver mode

Simplify screensaver inhibit API and fix OSX compatibility

Add/remove source(s) in our MSVC project (gtk2_ardour)

Prototype infrastructure to disable screensaver

ptformat: Update to upstream 8d29d79 (drop fades && fix offsets)

Add script to `xdot` plot the process graph

Add Lua bindings for Pango Alignment

NO-OP: Fix compiler warning (-Wreturn-type)

Add method to graphviz plot the process-graph

Gui should allow foldback bus to be mono.

foldback may be mono don't set panner type

OSC: foldback buses should be able to be mono

Amend 183d3fb49; implement ImportableSource API

Fix meter metric visibility on narrow strips - #7781

After session load, the meter numerics were always visible,
regardless of strip width.
MixerStrip::set_stuff_from_route() and MixerStrip::set_width_enum()
are called before Mixer UI is realized and calls show_all().

Implement missing required virtual fn -- fix Mac builds

Update plugins/addons GPL boilerplate and (C) from git log

Update utility and tools GPL boilerplate and (C) from git log

Update core library GPL boilerplate and (C) from git log

Update canvas/UI lib GPL boilerplate and (C) from git log

Update audiographer GPL boilerplate and (C) from git log

Update backend GPL boilerplate and (C) from git log

Update libardour GPL boilerplate and (C) from git log

Update GPL boilerplate and (C)

Copyright-holder and year information is extracted from git log.

git history begins in 2005. So (C) from 1998..2005 is lost. Also some
(C) assignment of commits where the committer didn't use --author.

Remove cruft

Add/remove source(s) in our MSVC project (gtk2_ardour)

Fix CD cue file Index positions -- #7780

According to the cdrdao man page, they should be defined as follows:
  INDEX MM:SS:FF
   Increments the index number at given position within the track.
   The first statement will increment from 1 to 2. The position is
   relative to the real track start, not counting an existing pre-gap.

prefer lowercase and no spaces, since this will appear in the filename.

(Source List) Catch new sources created by bounce-range-to-source-list

(Source List) Handle Session::RemoveSource

(Source List) Replace missing initializer; fixes a bug where Sources did not appear on startup.

(Source List) Implement insert-region action... now uses the Source List.

(Source List) Recover Regions feature:  WIP

(Source List) Revert to displaying whole-file Regions instead of Sources.
This preserves the stereo-ness of files that were imported or recorded in stereo.

ToDo  (maybe someday): provide a disclosure triangle that exposes the individual channels in a multichannel region

(Source List) Add forall_regions function, used by Region and Source lists.

(Source List) No need to show the path when file is internal to session.

(Source List) merge fixes

(Source List) Clean up the natural_position implementation (gtk2 part).

(Source List) Clean up the natural_position implementation (libardour part).

(Source List) Implement remove_selected_sources when the user clicks Delete key.

(Source List) NOOP: whitespace cleanup

remove debug output

(Source List) Fix drag-n-drop.

(Source List) RegionFactory should return an empty region here.

(Source List) Source list, Region List:  Resolve various action+behaviors between the 2 lists.

Duplicate the remove_unsued_region action in the Session->Cleanup menu, for discoverability.
Region list should use the same Region actions as the canvas...
OK, now resolve the change in use-cases of Region List and Source List.  tentative.
Rough-in remove_selected_sources (Delete in Sources list).
Column tweaks.

(Source List) Region Tags (gtk2 part)

(Source List) Region Tags (libardour part)

Rough-in: Region-Tags.

More correct implementation of tags property (libardour).

Region Tags (libardour part)

(Source List) Make clock change func more explicit.

(Source List) Region List rewrite (gtk2 part)

See:  https://docs.google.com/document/d/1sI7p9RoRVZtNx2n67RvBa0_16fuZblV_lNkkKN2g93s/edit?usp=sharing

(Source List) Region List rewrite (libardour part)

(Source List) Source property signals (gtk2 part)

See:  https://docs.google.com/document/d/1sI7p9RoRVZtNx2n67RvBa0_16fuZblV_lNkkKN2g93s/edit?usp=sharing

(Source List) Source property signals (libardour part)

(Source List) Take-ID: new (working) implementation.

Update clang-format to match ardour-style

Fix C++11 NSDMI

Update MIDI Event precedence

A patch-change that is concurrent with a note-on event should be sent
before the given Note-on event.

As follow up: CCs also need to be prioritize to send bank-select first.

see also https://discourse.ardour.org/t/midi-strangeness-patch-setting-and-first-notes/101415

Another try at fixing our 'spinlock_t' compatibility

Partially undo 6d4b94df13 for MSVC and C++98 compat

Fix random off-by-one issue when vertically zooming a waveform

To mitigate concurrent rendering, the waveform cache adds a random
range of pixels centered around the visible waveform.

Alignment is using integer half_width = width_samples / 2;
This always aligns the left-edge to the left-most cairo-pixel.

This fixes an issue with moving moiree patterns in waveforms when
zooming vertically (which invalidates the cache and uses a
different random number of pixels),

Fix rendering of region waveform left-edge

The left-most part may be a partial block of a given "sample per pixel"
range. Also previously there was an off-by-one [block] in case
sample_pos was an even multiple of samples_per_visual_peak.

NO-OP: whitespace

Reduce compiler warnings when boost uses std-atomics

This works around for compilers with non-static-data-member
initialization.

spinlock_t is-a struct { lockType _; } and BOOST_DETAIL_SPINLOCK_INIT
initializes the first member of the struct.
All defines of BOOST_DETAIL_SPINLOCK_INIT include c-style curly braces
to initialize the struct's data member.

However, modern C++ compiler interpret the braces differently resulting
in copy constriction of the initializer.

Yet another spinlock init hack for g++8 std::atomics

Perhaps we should rather implement this ourselves,
using <boost/atomic.hpp>

Prevent spinlock copy construction

Another attempt at improving spinlock init

Follow up d8ae3fd

Depending on implementation, d8ae3fd may only construct the spinlock
once to `sl_init`. Later it is only copy-constructed and that leads to
compile and/or runtmime errors.

e.g. gcc-8.3 fails to compile
error: use of deleted function ‘boost::detail::spinlock::spinlock(const boost::detail::spinlock&)’

MSVC requires an already initialized object to initialize our boost::detail::spinlock (rather than the simpler #define)

Hopefully this'll work for the other builds too.

Correctly initialize spintlock_t

Depending on underlying implementation, boost::detail::spinlock
needs to be explicitly initialized