match ID of Inline Scope following 2b7a89e

Save changes when toggling checkboxes in ExportDialog

I used the Widget::on_hide method in the ExportFileDialog to defer the saving
of changes in state of the analysis and soundcloud-upload checkboxes as it was
not possible to save the format xml state directly from the
ToggleButton::toggled() signal as it created a recursive loop and also to
prevent saving the state more than once.

Even though the ExportProfileManager::FormatListChanged signal is no longer
emitted when saving format state and the crash no longer occurs without this
change. I think it is worth saving explicitily from in the toggle callbacks to
reduce the complexity of understanding what is taking place and when even if it
is less efficient.

There is definitely more opportunity for refactoring and redesign.

Fix crash when ExportDialog is hidden

The issue is that when ExportDialog is destroyed
ExportFileNotebook::FilePage::on_hide is called which in turn calls
ExportProfileManager::save_format_to_disk() in order to save changes to the
analysis and soundcloud upload checkboxes. This then causes the
ExportProfileManager::FormatListChanged signal to be emitted, which in all
other cases is emitted when the format list is modified so that the GUI can
rebuild the format selector menu in ExportFormatSelector::update_format_list
when a format is added or removed.

The problem when doing this in the destructor is that some of the widgets have
already been destroyed, specifically the issue was in
ExportFileNotebook::update_soundcloud_upload trying to access the
soundcloud_selector member that had already had its destructor called.

As it is not necessary to call this signal in the first place and it just
causes unnecessary GUI updates remove the signal emission.

Fix indentation in ARDOUR::ExportProfileManager source file

VST threading: prevent concurrent effSetChunk and process()

This is a potential fix for unreliable preset load/restore.
(http://mixbus.harrisonconsoles.com/forum/thread-1970-post-21486.html#pid21486)

Since a Glib Mutex can't be copy-constructed an explicit copy c'tor
is needed.

NO-OP: whitespace

re-do previous commit

* do not include _by_name() API. Port names are locale dependent
* proper whitespace (after comma, before bracket) and styleguide

Revert "Add convenience Lua bindings to access plugin controls"

This reverts commit faf44386460b6cdda0aa249eb138e8fd82700194.

Add convenience Lua bindings to access plugin controls

remove cruft - don't pollute global namespace

* remove unused functions
* don't globally export C functions that are only used locally

Allow to query HTTP headers

 Modify our MSVC project to use the newly introduced 'libarchive' library (libardour)

Modify our MSVC project to use the newly introduced 'libarchive' library (libpbd)

Skip pin-management for MB channelstrip plugins

Do not delete selected MIDI notes while dragging. (See #6602)

This is not caused by commit_reversible_command, but because
NoteDrag::total_dx calls Evoral::Note<Evoral::Beats>::time()
with invalid MIDI note (deleted).

Fix passing ctrl-output data to lua inline display thread

Fix links to manual in windows installer

Send/show error message if g_stat fails in Session::cleanup_sources

let the user/developer know if this does occur rather than just silently
skipping the file.

Fix some whitespace issues while we are changing indentation.

Use g_strerror() instead of ::strerror() in Session::cleanup_sources

The error is generated by a glib function so use the glib version of this
function(it also consistent with usage in the rest the file).

Use g_rename() instead of ::rename() in Session::cleanup_sources

The paths are in UTF-8 encoding and ::rename expects paths in the system
codepage encoding so ::rename will fail for any paths that contain characters
that aren't in the system codepage.

This fixes Flush Wastebasket on Windows where paths contain characters that
aren't in the system codepage(usually most non-ascii characters).

Use g_strerror instead of strerror in ARDOUR::FileSource

For consistency when calling glib functions and to get UTF-8 strings for
display.

Use ::g_rename instead of ::rename in ARDOUR::FileSource class

The path is in UTF-8 encoding so use Glib functions to properly support paths
containing characters that aren't in the system codepage.

This fixes Clean-up Unused Sources on Windows where paths contain characters
that aren't in the system codepage(usually most non-ascii characters).

Add defines necessary for successful gcc/mingw compile with --cxx11 option

_USE_MATH_DEFINES is required for M_PI

WIN32 is no longer defined by gcc/mingw with --cxx11 option enabled(of course
_WIN32 still is) but as it is used in a liblo header(even though it is probably
incorrect to do so) define it.

Allow to bunde demo session(s)

bundle embeddable/nativeUI Harrison plugins

remove cruft from packaging scripts

implement file-[un]archive progress report

Restore state of analyze and soundcloud upload checkboxes in ExportDialog

This should be a proper fix for bug #7003, now that I've spent a bit more time
reading the code. The options are now stored as part of the export format
state.

Add analyse and soundcloud-upload properties of ExportFormatSpecification to xml state

Check a return value from 'g_stat()'

Some Mixbus users (on Windows) have reported seeing ludicrously high figures for the amount of disk space that'll be recovered if they choose to clean up unused sources. I can't see anything obviously wrong in Ardour's code - except for one situation where we don't check a return value after calling 'g_stat()'.

On Windows, the relevant path should be (hopefully!) in UTF8 format and the first thing that g_stat() does is to convert it to UTF16. If that conversion fails for some reason, g_stat() will return an error status and statbuf will be uninitialized - but at the moment, we're not checking this. As an experiment, let's check the returned value and find out if these user reports go away.

Unfortunately, if it does fix the problem then we've got an even bigger problem - because somehow, a Windows user can create source files with invalid names which can't be later deleted!!

Fixes 7004 - Ardour crash on Just Quit when AudioEngine is stopped. AudioEngine::stop() if running() is not true and _backend->stop() is true, mutex was being unlocked without being locked.

change all MIDI read-from-source to map all events into the loop-range for seamless looping (if using)

debug output formatting

do not call MidiRingBuffer::skip_to() when getting a MidiDiskstream's playback buffer, and do not use port offset to shift data in time either (it should be correct)

make MidiBuffer::read_from() use "dst_offset" correctly to alter the time of events read from the MidiBuffer

extend Evoral::Range<T> to offer ::length() and ::squish()

The latter maps a T into a range, using loop semantics

add back event offsetting into MidiRingBuffer::read() (even if we never really use it)

NOOP: spacing and indentation

show "this" in MidiRingBuffer::dump()

slight DEBUG_TRACE enhancement for MidiPort::flush_buffers() output

NO-OP: indentation

Delivery::run() now offsets data delivered to MIDI ports by the global port offset

if the output is then re-used, MIDI data is readjusted to not use the global port offset

use class-static method rather than pretend there is a per-member value for Port::port_offset()

provide Session::declick_out_pending()

second part of:  change offset arguments to various <T>Buffer::read/merge methods from framecnt_t to frameoffset_t

change offset arguments to various <T>Buffer::read/merge methods from framecnt_t to frameoffset_t

add a new DEBUG enum for MidiRingBuffer

copy demo sessions

add option to skip unzipping bundled demo sessions

Add support for built-in file/url unzip/untar

This introduces new build-dependency: libarchive
(http://www.libarchive.org/)

Rename Struct defined in freesound header to be less generic

This could potentially conflict with a struct declared in the
ardour/soundcloud_upload.h file.

Remove duplicate header includes in gtk2_ardour

Remove duplicate header include from libardour source file

Add missing header include guards in various gtk2_ardour headers

Add missing header include guards in canvas library

Put ARDOUR::SystemExec implementation inside ARDOUR namespace in source file

Add missing include guards in libardour header files

Tweak PR 269 for portable file-path

Cleaned up messed indents (tabs vs. spaces)

Lua script to export markers as mp4 chapters

Location markers, that are not xruns are exported as mp4 chapter
marks. This requires `Location::name()` to be exposed to Lua.

Expose Location::name() to Lua

Add a Lua wrapper to Glib::build_filename()

allow running from src-tree using system-wide gtk-themes w/color defs

Example script to prepare the session to record

Often one needs to make sure that all the relevant settings are made
before one actually starts recording. The Lua interface can take care of
this task easily, that the operater by just one action can make sure
that the relevant settings (e.g. recenables, playhead position, etc.)
are ready for recording. This example was written with a quasi live
podcast scenario in mind.

See `scripts/prepare_record_example.lua` for details.

Expose `Locations::remove()` to Lua

Clearing location markers is a frequent task and should be automizable. For
example it is handy to clear all the markers when restarting the recording
from scratch after a failed recording.

add request_play_loop() & get_play_loop() to luabindings

Fix #7015, Crash while aborting Region trim using Esc key

There were two issues:

The first is that TrimDrag::aborted was calling TrimDrag::finished with a null
GdkEvent which caused a segfault when dereferencing. So avoid that by passing
in a dummy event as we are just going to undo the operation in the next step
anyway so it shouldn't matter if it is valid(AFAICT).

The other is that TrimDrag:aborted was calling Editor::undo() which was in turn
calling TrimDrag::aborted leading to infinite recursion and stack overflow.
Calling Session::undo() directly seems to avoid that issue.

This fix feels like a bit of a hack...but it seems to work and is better than a
crash.

Fix #6774, Moving start/end markers doesn't set Session as modified/dirty

If a Session is saved and then the start or end session markers are moved,
closing the Session will not result in "Unsaved Session" dialog being presented
even though the Session has been modified. This change sets the Session as
modified when either the Session start or end markers are modified.

Fix #7003, Store/Restore state of checkboxes in Export dialog

closing the normalize dialog should not run normalize (#7016)

fix WM close handling for session lock dialog (#7017)

for files imported without copy, _origin is a full path. We should use this when looking for the file at session load

correct comment for FileSource::find()

add ActionManager::rec_sensitive_actions and put "add track/bus" in this group, which is disabled during active recording

use moved version of reset_focus()

Expose record enable/disable to Lua bindings

use moved version of reset_focus()

move kbd focus reset method into ARDOUR_UI where it (probably) belongs

Update clear-gray-ardour.colors

This commit changes item "gtk_bases" from light (color22) to darker (color88). The knob of "Enable/Disable MIDI input" in Edit List had the same colour and was not visible. This commit makes it clear visible, and it has no much influence on a general design.

use existing Editor::reset_focus() method to reset focus after audio clock editing is done

do not set the focus widget to be a toplevel window in Editor::reset_focus.

See comments in code for the reason why

Fix incorrect estimate of future revision.

Remove unused variable.

Load midi region length and start correctly in sessions modified by v5.0 -> 5.3-41

remove duplicate enum -- copy/paste typo

expose PeakMeter Lua bindings

Set start_pulse and length_pulse from state, neither need to be a PBD::Property.

Fix the tempo note-type hallucination.

- after reports that a tempo with non-4.0 note type
  would produce a ramp in the preceding section, an incorrect
  assumption was made leading to a bogus 'fix' that redefined
  Tempo's pulses_per_minute().
  the real fix was simple:
  the ramp's function constant was calculated using the note type
  of the proceding tempo rather than the current one.

- this patch reverts e82482e8e9c and a4d67279e0086258
  and maintains the old definition of Tempo::pulses_per_minute()

Sanity check for TempoMap::get_grid ()

Revert part of d50df8279966.

Fix graphical issues in tempo curve.

Move check for detected bundle earlier in install process.

Previously if the bundle being installed did not support the detected
system the user would get an error message about the .size file
being missing.

Fix various paste errors in tempo.cc. Should fix recent regressions.

Ensure the correct position is passed to MidiRegionView::get_grid_beats().

Use the stored beat to recalculate pulse when setting position as sub_num is irrelevent.

Editor::get_grid_type_as_beats() returns the quarter note equivalent of BBT beat and bar.

Consolidate the meaning of beat divisions.

- bar and beat snapping is BBT beat based while other
  music divisions are quarter note based. Reflect this in
  the way exact_beat/qn is calculated.

Fix a bug when formatting time strings

It seems that 'strftime()' (on Windows) works differently from its non-Windows counterparts. Specifically, some formatting options (e.g. %F) are not recognised in the Windows implementation.

Fortunately, glibmm comes to our rescue here! So let's use the glib implementation which will hopefully work the same on all platforms.

OSC: don't restore User preset before writing new one

Attempt to clarify tempo map comments.