Release: bump VERSION.

Merge remote-tracking branch 'origin/release/unstable' into HEAD

Update release/unstable with recent changes

Issue 4959: output-ps.scm: remove unused helper functions

Their presence makes debugging harder by laying false trails.

Issue 4957: parser.yy: loc_on_music -> loc_on_copy

The function loc_on_music has been renamed to loc_on_copy and has been
changed to create a copy for a number of different expression types.
This makes \xxx.yyy behave similar to \xxx-yyy with regard to copying
the original expression in most cases, and it also helps with default
arguments of music/void/event/scheme functions of more types than just
music.

Issue 4956/4: Add regtest for \retrograde

Issue 4956/3: Augment warning about \retrograde

Ties work now, but the documentation points out a lot of other
shortcomings.

Issue 4956/2: Improve retrograde-music function

This allows to work with ties and glissandi, makes somewhat useful
changes for overrides, changes crescendi to descrescendi and vice versa.

Issue 4956/1: Let \retrograde expand repeat chords/notes

Since those lose their point of reference, expanding them
prior to retrograding keeps the results sane.

Doc: CG update Indenting with vim section

Suggestions for .vimrc did not produce correct indentation of C++
code, and also included personal and irrelevant settings such as
statusline and incsearch. I have replaced them with the settings
in the GNU GCC Wiki which do correctly indent. Also changed
suggested Scheme settings to use setlocal, and added a section of
settings for Texinfo files.

Issue 4954/2: Changes entries for \= and in-chord slurs

Issue 4954/1: Change spanner-id to be a key instead of a string

Also changes argument \= to be a key.

Backport-by: David Kastrup <dak@gnu.org>

Release: bump Welcome versions.

Release: update news.

PO: update template.

Release: bump VERSION_DEVEL.

Issue 4952 Clearify ending of hairpins at bar line in NR

Issue 4953 Fix SchemeTextSpanEvent in scheme-text-spanner.ly

Adding 'post-event to the types of SchemeTextSpanEvent in
(define scheme-event-spanner-types ...)

Typos.

Doc: Extending - rewrite - LilyPond Variables

Issue 4948

Last para of Extending/LP variables
rewrite for clarity.

Doc: updates translation status

Doc-fr: Change Lilypond to LilyPond where it refers to the software

  and NR: show optional \afterGrace argument

Change Lilypond to LilyPond where it refers to the software

Change 'Lily Pond' to 'LilyPond' in two translations

Issue 4947: Link notes to dynamics in Dynamic_performer rather than
Staff_performer.  Dynamics in different voices are now independent.

Issue 4945/2: midi2ly should omit redundant chord durations

Issue 4945/1: midi2ly -e should not print durations in chords

Issue 4941/2: NR: show optional \afterGrace argument

Issue 4941/1: Optional fraction after \afterGrace command

\afterGrace had its fraction determining the position of the aftergrace
notes hardwired to be read from the parser variable afterGraceFraction.
This change here allows for optionally specifying it right as the first
argument of the \afterGrace command.

Issue 4853 - correct make syntax in CG

Doc-fr: sync NR with master

Merge branch 'master' of /home/jcharles/GIT/Lily/. into translation

Doc-it: fix error of translation

Doc: CG fix misleading location for copying tarbal

Issue 4933

Fix misleading location
for copying tarball.

Issue 4938 (3/3) Rename Midi_control_function_performer to Midi_control_change_performer

Issue 4938 (2/3) Refactor handling of MIDI control changes

Handle the MIDI control value initialization from context properties
(Staff_performer::new_audio_staff), control value changes
(Midi_control_function_performer::announce_function_value_change), and
value conversion for output
(Midi_control_function_value_change::to_string) in the new
Midi_control_change_announcer class.

All MIDI control changes are now encoded using
{Audio,Midi}_control_change items.  This change makes the old
{Audio,Midi}_control_function_value_change classes obsolete.

Issue 4938 (1/3) Add Audio_item and Midi_item subclasses for control changes

Release: bump VERSION.

Merge remote-tracking branch 'origin/release/unstable' into HEAD

Issue 4940/2: Replace texinfo.tex macro name

texinfo.tex ver. 2016-05-28.15 renames
commondummies to definedummies.
So we need to replace it which we use.

Issue 4940/1: Update texinfo.tex from upstream

texinfo.tex ver. 2016-07-20.14
This ver. fixes XeTeX PDF destination.

Partially revert 445bf3bb2fbd1f259fe43ade204fb34d68bdd581.

Release: update news.

Doc: NR Describe the "midititle" \header variable

Issue 4932

The "midititle" header variable
can be used to set the sequence
name for MIDI files independently
of the contents of "title".

Try changing near and far

Release: bump Welcome versions.

Release: update news.

PO: update template.

Release: bump VERSION_DEVEL.

Doc-ca: Catalan translation fixes

Doc-ca: Catalan translation fixes

Doc-ca: New Catalan translations and fixes

4944: Do not initialize static constants in class body (fix FTBFS on GCC 6)

Issue 4814: grob.cc segfaults with gcc6

From the release notes of GCC 6:

    Optimizations remove null pointer checks for this

    When optimizing, GCC now assumes the this pointer can never be null,
    which is guaranteed by the language rules. Invalid programs which
    assume it is OK to invoke a member function through a null
    pointer (possibly relying on checks like this != NULL) may crash or
    otherwise fail at run time if null pointer checks are optimized
    away. With the -Wnull-dereference option the compiler tries to warn
    when it detects such invalid code.

    If the program cannot be fixed to remove the undefined behavior then
    the option -fno-delete-null-pointer-checks can be used to disable
    this optimization. That option also disables other optimizations
    involving pointers, not only those involving this.

As a consequence, we cannot call a member function on a prospective null
pointer (which actually is a bad idea for a number of other reasons,
like when anything tries accessing the vtable) and then try sorting out
the condition in the routine itself.

This problem was first observed with Fedora 24.  The Ubuntu GCC6
prerelease does not show this problem; presumably the respective
optimization has been disabled in the Ubuntu/Debian packaging because of
affecting other programs.

Commit-message-by: David Kastrup <dak@gnu.org>
Signed-off-by: David Kastrup <dak@gnu.org>

Doc: In NR, use comma before and after `etc.' where appropriate.

Doc: Make `Predefined commands' sections ragged-right.

This greatly enhances the PDF output.

Doc: Various minor fixes to NR.

Doc: In NR, always write comma after `e.g.' and `i.e.'.

This especially improves the formatting of `notation.pdf'.

Doc-de: fix div class

Thanks Marc for the report:
http://lists.gnu.org/archive/html/lilypond-devel/2016-07/msg00116.html

Issue 4048 (5/5) Add midi regtests covering depart-return crescendo performance

Issue 4048 (4/5) Handle multiple (de)crescendi in depart/return groups

Issue 4048 (3/5) Add midi regtest decrescendo-single-contrary-target

Issue 4048 (2/5) Dynamic_performer: represent dynamics as a piecewise
linear function rather than a collection of discrete points

Issue 4048 (1/5) Add some midi regtests

Replace crescendo-abutting with variations crescendo-return-crescendo
and crescendo-return-unspecified-target.

Issue 4926: Remove warning to use \harmonics in chords only

This restriction has been abolished with issue 2240, "Don't wrap
EventChord around rhythmic events by default."

Issue 4924: Fix the warning message for OTC fonts

This commit fixes the warning message for OTC fonts.

Doc: specify which \header blocks generate PDF metadata

Issue 4921

Doc-fr : sync with master

Issue 4925: Don't use @code inside of @example

@example is already set in monospace font (use @display if you don't
want to change fonts).  This replacement was done mechanically using
the script

    sed -i '/@example *$/,/@end example *$/{s/@code{\(\([^@}]\|@[@{}]\)*\)}/\1/g;s/ \+$//;}' $(git grep -l '^@end example$' Documentation)

so not all replacements are really formatted best.  But overall this
looks like an improvement.

Issue 4923/2: Add closing a file in `scale-down-image`

This commit adds `close-port` for closing a file
in `scale-down-image`.

Issue 4923/1: Add closing a file in `is-collection-fonts?`

This commit adds `close-port` for closing a file
in `is-collection-fonts?`.

Issue 4918: Fix CFF flavored font embedding for multiple output

For multiple output (e.g. `-dbackend=eps`),
conflict checking needs to be done within each output.

Otherwise, the font which is embedded in the prior output
is not embedded in the subsequent output.

4915: NR: Mention standalone accidentals in figuremode

Issue 4917: Remove some set_location calls from parser

Older versions of Bison (current is 3.0.4) had problems assigning
location data to rules with empty production, possibly related to the
definition of YYLLOCA_DEFAULT when N is zero.

This lead to several workarounds in the code base.  A number of them
dropped through the floor in the course of refactoring without
apparent problem, and the original problem does not appear to be
reproducible with the current versions of Bison.

This removes the remaining instances.  Should the original problem
reoccur at some point of time (or with some versions of Bison), it
would be noticeable as bad point-and-click messages and/or error
messages with bad location data.

Issue 4916: Let \overrideProperty use ApplyOutputEvent.symbol

Issue 4907: Midi_walker::do_start_note: skip ignored notes in stop_note_queue

For each semitone pitch value, stop_note_queue is likely supposed to
contain at most one Midi_note event with its "ignore_" flag set to
false, and the comparisons between notes of equal semitone pitch to be
always done between the input note and this unique queued note that is
not (yet) being ignored.

If notes which are already being ignored are not skipped in the loop,
the task of raising the "ignore_" flags for note events of equal
semitone pitch (overlapping in time) which stop before the maximum
stopping time of these notes may, due to breaking out of the loop,
fail to work if the queue grows to contain three or more notes of equal
semitone pitch, leading to the emission of premature "note off" events
for this pitch, as demonstrated, for example, in
<http://lists.gnu.org/archive/html/bug-lilypond/2016-06/msg00042.html>.

Merge branch 'issue4914'

Issue 4914/3: Add Output_property_engraver back to Score

It was removed by a simplistic convert-ly rule.

Run scripts/auxiliar/update-with-convert-ly.sh

Issue 4914/2: convert-ly rule for removing Output_property_engraver

It's highly unlikely that users will redefine the Score context from
scratch, so the convert-ly rule just removes every occurence of
Output_property_engraver from user source.  Obviously, when running the
rule on the LilyPond code base, we will need to fix up the Score
engraver manually to retain the Output_property_engraver .

Issue 4914/1: Move Output_property_engraver to Score level

This has the advantage of needing only one instantiation of the engraver
and not having \applyOutput mysteriously refrain from having an effect
in contexts without Output_property_engraver .

Due to the hierarchical nature of acknowledgers, acknowledgers in lower
contexts will now get to see the grobs before applyOutput has done its
work.  However, grobs are still unfinished (except for type, properties
initialized via context properties and cause) at the time they are
announced, with other details only getting filled in by the engraver
after announcement, so the potential for trouble seems low.
Acknowledgers should usually just register a grob (or write grob data)
with any actual reading of grob data occurring at the end of the
timestep instead or in the process-acknowledged phase.

Issue 4911/4: Fix comments in context-mod-with regtest

Issue 4911/3: Don't treat context modification identifiers special

This requires using \with before context modifications like
\RemoveEmptyStaves in order to make the syntax get along
with fewer special cases and exceptions.

Run scripts/auxiliar/update-with-convert-ly.sh

Issue 4911/2: convert-ly rule to put \with before context mods

Issue 4911/1: \with #*unspecified* should be equivalent to \with { }

Release: bump VERSION.

Merge remote-tracking branch 'origin/release/unstable' into HEAD

Trivial typo fix in doc-work.itexi

Release: bump Welcome versions.

Release: update news.

PO: update template.

Release: bump VERSION_DEVEL.

Issue 4910/3: Avoid crash when g_spawn_sync () fails

When g_spawn_sync () fails,
`standard_output` and `standard_error` may not be set.
This commit makes to check whether or not the value is valid.

Issue 4910/2: Replace the warning message output method in ly_run_command ()

This commit replaces
the warning message output method in ly_run_command ()
from fprintf () to warning ().

Issue 4910/1: Fix memory leak in ly:spawn ()

This commit makes free the memory allocated by g_spawn_sync ().

Reimplement issue 4781 for musicxml2ly more literally. Reimplementation
was necessary as part of the implementation of issue 4751, but some of
the code changes for 4781 were omitted. This reproduces all the 4781 changes.

Issue 4912: Fix output definition use in \book and \bookpart

The only explicit output definition blocks allowed in \book and
\bookpart blocks were paper blocks.  Output definitions supplied with
Scheme expressions were erroneously interpreted like global output
definitions, accepting all output definition types and overriding the
global defaults with them.

Now the only output definitions accepted as Scheme expressions are
paper blocks.  As opposed to previously, they actually set the paper
block of the respective book or bookpart.

Issue 4908/7: Add Changes entry for \header expressions

Issue 4908/6: Use \markupId and \stringId in identifiers regtest