Remove unused macros ADD_TYPE_PREDICATE and DECLARE_TYPE_P

Issue 4033: All subsequent clef transpositions are marked as optional

This removes a bug cover-up in scm/parser-clef.scm, implements the
necessary functionality in display-lily-music and adds some regtests
for that functionality.

Issue 4035: Some refactoring and cleanup in lily/paper-book.cc

makelsr run on snippets from LSR

Phase two of pruning snippets/new

Doc: Appendix - Articulations and Ornamentation - part 2

Issue 1189

Create Texifo @multitable entries for the
List of Articulations appendix.

As there are a lot of separate scripts being
documented this Issue is going to be split
into multiple, smaller, parts to aid in the
review process. Part 1 was commit 8553021

This is part 2 and covers 'Fermata', 'Instrument
specific', 'Repeat signs' and 'Ancient' scripts.

Also, after more consideration, a third example
of each script has been added to show the
explicit differences between the up/down/neutral
positions. Not every script has a different
up/down glyph nor is positioned up/down when
placed on a note that is either top or bottom of
the Staff without explicit positioning. So this,
I hope just clarifies things (regardless whether
the position is musically 'correct' notation).

I have also sey the @code{} script examples into
the same column as the @lilypond examples; saving
a significant of wasted page space. Also I have
removed unnecessary lilypond-book variables making
the texinfo code less noisy.

Merge branch 'translation'

Merge remote-tracking branch 'origin/master' into translation

Make list of `non-staff lines' easier to find in IR.

Linking from the NR to the Axis_group_engraver IR entry
directs the reader to the list of staff contexts and
`non-staff line' contexts.

Remove warning when there are no noteheads to attach lyrics to.

The warning was added to address issue 248, but it is now moot after
issue 3254 (c73b41b3e7be6d7): attaching lyrics to a note-less context
(like Devnull) is perfectly legal now.

Issue 4021: Remove obsolete restrictions from define-*-function docstrings

Issue 4019: binding-offset not properly scaled along with paper size

This will unfortunately affect all previous uses of binding-offset which
likely were employed using trial-and-error.

Doc: NR - Pitches.itely - dodecaphonic-no-repeat text alt

Issue 4023

Came out of issue 3992

Corrected wording for dodecaphonic-no-repeat
accidental style.

CG: Update of Patchy instructions

Issue 4013

Added fuller explanatory information regarding Patchy.

Issue 4005: Set X-parent of TextScript to NoteColumn instead of PaperColumn

This makes TextScripts consistent with DynamicTexts and LyricTexts.
This is a follow-up to commit 59a842eba0f7ad78289a58a (Issue 2245).

Setting TextScript.cross-staff property to #f is required to ensure
that there are no collisions between TextScripts and cross-staff notes:

\context PianoStaff <<
  \new Staff = "up" {
    b8[
    \change Staff="down"
    d'] ^"text"
  }
  \new Staff = "down" {
    \clef bass
    s4
  }
>>

(see also beam-cross-staff-auto-knee.ly)

As far as I can see, we don't want TextScript.cross-staff to be true
in any situation, because it would result in unwanted collisions.

Why it worked before: cross-staff property in this example evaluated to #f,
but only because of a bug in Script_interface::calc_cross_staff.  That
function should have marked the TextScript as cross-staff if the stem
of the note to which TextScript was attached was cross-staff, but it
didn't work correctly because it expected the parent of the TextScript
to be a NoteColumn, while it actually was a PaperColumn.  When I changed
the parent to be a NoteColumn, the function started working correctly
and marked the TextScript as cross-staff, so I had to change the default
value of the property.

Import snippets from LSR and delete initial batch in snippets/new

Fix ly:make-stream-event to receive a symbol list rather than a symbol as event class

Issue 4007: Fix parts of apparently dead code make-stream-event

It is not clear whether the code will be operative for its intended purpose
afterwards, but leaving it in broken state is not going to help.

typo/oversight in align-interface.cc and page-layout-problem.cc

issue 4008

fix oversight in doc-string of align-interface.cc
fix typo in comment of page-layout-problem.cc

Issue 4017: Define spacing from KeyCancellation to custos

Adds a space-alist rule from KeyCancellation to custos
to prevent "No spacing entry" programming error.

Issue 4015: Add \magnifyStaff.

* Add \magnifyStaff.
* Rewrite parts of the \magnifyMusic function so that it can share
  some scheme code with the new \magnifyStaff function.
* Add regtests.
* Update documentation.

Don't call Pitch::Pitch with more arguments than necessary.

Pitch::Pitch (int, int) is declared but not defined

This caters for the definition by making the last argument of
Pitch::Pitch (int, int, Rational) a default argument.

Merge branch 'dev/dodecaphonic_no_repeat' into staging

Issue 3992: Fix dodecaphonic-no-repeat

Incorporate renaming of localKeySignature to localAlterations.

The function dodecaphonic-no-repeat-rule now considers alterations.

A pitch which overlaps the same pitch begun earlier should not
receive an accidental.

Improvement to code comments.

Improve code formatting.

Issue 3999: Make tablature half-note stem-spacing adjustable.

This adds a new property to the Stem grob called
'double-stem-separation (default=0.5), that allows users to adjust
the space between the double-stemmed half-notes in tablature:

\new TabStaff {
  \tabFullNotation
  c4 c2 c4
  \override Stem.double-stem-separation = 0.3
  c4 c2 c4
}

It also centers the stems on the fret number and adjusts the
X-extent accordingly.

Issue 3995: Rename 'thin-kern to 'segno-kern.

Issue 3994: Fix typo `dash-details' => `dash-definition'.

Fix typo in the docs and translated .po files.
Also fix some whitespace errors.

Issue 3952: Automatically sort props within each grob-interface.

Ensures that "User settable properties" and "Internal properties"
are listed in sorted order in the "Graphical Object Interfaces"
section of the IR.

This was originally pushed with commit 84f0776,
but was accidentally reverted with commit 0ea80a5.

Run scripts/auxilar/makelsr.py

Issue 3998: Let "Mars" example in pitch snippets use isolated durations

This is one example quite benefitting from issue 3648.  One somewhat
incidental boon is that it would now work in relative mode without
requiring the make-relative macro.

Revert "Change dsi to di in LSR address; update version"

This reverts commit 4ee99b07605942b02e82c6526df98b68cbc03981.

NR: Adapt docs for inversions in chordmode

Issue 2617: simple chord inversions take 2 octaves

With this change, chord inversions transpose all notes following the
inversion from the original chord down along with the inversion, as long
as the result ends up below the root note of the uninverted chord.

So with \chordmode { c:11 } => < c' e' g' bes' d'' f'' >
we get \chordmode { c:11/e } => < e g bes c' d'' f'' >

In order to have the Chord_name_engraver reconstruct the correct
uninverted chord, all the additionally octavated note events are given
an octavation property like the inversion event itself.

An "inversion" on the chord root just transposes the chord root one
octave down, leaving the rest in place.

Issue 4001: Fix doc string of ly:pitch-diff

The documentation of ly:pitch-diff has got the sign wrong like forever.

Issue 4003: NR: warning for { \chordmode { c } c } obsoleted by issue 3797

Issue 4009: \retrograde fails on slurs

The section "Retrograde" in the Notation Reference is affected.
This became likely exposed with issue 2240; the corresponding
regtest does not cover it.  The problem was existent previously
for slurs and other spans inside of chords.

Add slur to modal-transform regtest motif

Change dsi to di in LSR address; update version

Mention backslash escaping in Scheme.

Clarify confusion about `@code{\foo}' vs. `@code{\\foo}'.

Doc: Appendix - Articulations and Ornamentation - part 1

Issue 1189

Create Texifo @multitable entries for the
List of Articulations appendix.

As there are a lot of separate scripts being
documented this Issue is going to be split
into multiple, smaller, parts to aid in the
review process.

This is part 1 and covers 'Articulation' and
'Ornament' scripts only.

Changes.tely updated - 2.19.x up to June 2014

Issue 3965

Since Feb 17th 2014.

Added Tracker issues
2752, 3937, 3913 and 3860

Included @lilypond examples where appropriate

Doc:NR Appendix - Clef Styles - added G2 and modernTab Clefs

Issue 3988

Added two Clefs to notation-appendices.itely - Clef Styles

\clef "G2" and \clef moderntab

Removed excessive brackets in the @lilypond constructions and
updated the '\clef tab' entry so that it engraves correct out
put by adding a \new TabStaff block in the @lilypond example.

Issue 4004: NR: remove a few warnings that do not apply any more

Issue 4000: NR: Non-working multi-staff beatStructure override example

Don't depend on features removed in Guile 2.1. (issue 3972)

* Replace scm_listify with scm_list_*. (It's been deprecated for a long time.)
* Use scm_protects only when available (Guile versions <2.1).

Release: bump VERSION.

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

Release: bump Welcome versions.

Release: update news.

PO: update template.

Release: bump VERSION_DEVEL.

Doc-fr: updates texidocs

Updates to GUB build instructions in CG

Issue 3996: Clarify 'thickness and 'line-thickness property docstrings.

Issue 3997: \magnifyMusic: don't modify nested properties.

Also reformat Scheme code.

Issue 3991: \magnifyMusic: surrender to issues 3987 and 3990.

Until issues 3987 and 3990 are fixed, automatic scaling
of horizontal spacing with \magnifyMusic is broken.  This
patch turns the feature off, and adds a note in the docs.

Issue 3958: Give examples of \applyContext usage.

Issue 2462: don't change ideal spacing when adding a rod

When objects like lyrics are added to the PaperColumns, LilyPond inserts
rods between these columns to ensure that the objects won't overlap.
However, the ideal distance should remain unchanged.  For example, in

\layout {
  ragged-right = ##f
}
\relative f' { \repeat unfold 8 a4 }
\addlyrics { \repeat unfold 4 la \repeat unfold 4 straight }

notes in 2nd measure have long lyrics attached to them - min_distance_
of the springs between these columns should be adjusted, but distance_
(the ideal distance) should not: it should be the same as for notes
with short lyrics.  When the line is stretched so much that the minimum
distances are not involved, the springs between notes in both measures
should behave (almost) identical.

Note that there are more such problems in the springs code, for example
in merge_springs.  The code should be rewritten, but that's not trivial.
Since this one-liner fixes some instances of the problem without any bad
side-effects, it makes sense to include it separately.

doc: update make-simple-closure example in Extending (Issue 3993)

After improvements in Self_alignment_interface::aligned_on_parent
(in particular after fixing issue 3254) it was possible to simplify
default X-offset values for several grobs, so that they didn't have
to use make-simple-closure (see e.g. commit 1d765020f8679958). This
made an example in Extending 2.7 obsolete, so I'm replacing it with
one that still uses make-simple-closure.

Revert "Doc: Appendix - Articulations and Ornamentation - part 1"

This reverts commit 0cf0757b026ea6d7435a9cd837019291c6dee3d6.

Pushed by mistake. This checkin is still under review.

My apologies

Doc: Updated Roadmap file

Issue 3963

Added new Catalan translation

Updated existing descriptions for previous entries
including minor formatting changes.

Doc: Appendix - Articulations and Ornamentation - part 1

Issue 1189

Create Texifo @multitable entries for the
List of Articulations appendix.

As there are a lot of separate scripts being
documented this Issue is going to be split
into multiple, smaller, parts to aid in the
review process.

This is part 1 and covers 'Articulation' and
'Ornament' scripts only.

Issue 2245: always align dynamics and lyrics on "main" notehead

Until now, LyricTexts and DynamicTexts had their X-parents set to
the first NoteHead in the NoteColumn.  This resulted in inconsistent
alignment - placement of lyrics and dynamics depended on the order
of notes in the input:

% this was aligned differently
{ <f' g'>1\p <g' f'>\p }
\addlyrics { la la }

By using NoteColumns themselves as the X-parents, we make sure that
the input order won't matter.  Since the NoteColumn contains all NoteHeads
(including suspended ones, which usually should be ignored when aligning),
as well as Flags and some other objects, we cannot use its X-extent directly -
instead, we add a function for calculating X-extent of the "main" part of the
NoteColumn, i.e. X-extent of the non-suspended NoteHeads (represented by the
NoteHead furthest away from the stem).

Doc: NR 1.2.1 - Tuplets add examples ...Up, ...Down & ...Neutral

Issue 3956

Added example showing the \tupletUp, \tupletDown and \tupletNeutral
@lilypond example.

The commands were mentioned in the @predefined and @funindex but
not actually illustrated with an @lilypond example.

Doc: NR 4.5.4 Change Node name from 'Line length' to 'Line width'

Issue 3982

Change Node name from 'Line length' to 'Line width'.

In the Notation Reference, this page is called
"Line length" but the property that it covers
(among others) is called "line-width."  Seems like
it would be better if the title of the page was
renamed "Line width" for greater consistency.

Issue 3984: Let \parallelMusic cope with \repeat .. \alternative

\parallelMusic's recursive decent only worked for comparatively simple
cases.  It now should be able to deal with complex constructs
gracefully.

Issue 3986: \displayScheme broken

    \displayScheme \markup \null

failed because a markup following an omitted optional argument (here the
optional output port) was backed up erroneously with the token type
LYRIC_ELEMENT only to be used in lyrics mode.

The optional argument was introduced with issue 2067 in version 2.19.0.
The actual underlying parser error triggered by this, however, is quite
older and originates from

    commit d4694123961690b173377b3c3a199d5a24c66a9d
    Author: David Kastrup <dak@gnu.org>
    Date:   Thu Nov 1 15:16:52 2012 +0100

parser/lexer: eliminate LYRIC_MARKUP as a token separate from MARKUP (2996/2)

in version 2.17.9.

Issue 3983: Avoid define-public and define*-public with curried definitions

Regarding curried definitions, GUILE has problems with define-public
before version 2.0.10, and with define*-public even later.

define-safe-public is implemented by LilyPond itself rather than the
(ice-9 curried-definitions) module and is unproblematic.

This is basically a cop-out since juggling with overriding bugfix
definitions of define-public is a distraction for getting GUILEv2
migration under way that we can do without.

Doc-fr: updates texidocs

Issue 3942: Scale slurs and ties when using \magnifyMusic.

Issue 3978: Merge alignment cleanup

Make DynamicTexts, TextScripts and many other grobs use aligned_on_parent,
which makes their behaviour consitent, predictable and thus more user-friendly.

define-grobs.scm: reorder properties alphabetically

TextScript, CombineTextScript: use aligned_on_parent

I think this makes more sense than just using "self-alignment".
It makes these grobs consistent with Lyrics and Dynamics, and
allows users to override their alignment more predictably.

Expected changes in output: TextScripts and CombineTextScripts
will be aligned slightly different when the user sets self-alignment-X
explicitly (parent notehead will be included in alignment, as in Lyrics).
Default placement should remain the same.

I'm also changing one regtest, just to make sure that despite slightly
changed alignment the markups will stick out far enough to the left.

Replace XY-offset closures with aligned_on_parent where possible

It doesn't make sense to specify placement using multiple added
callbacks or closures when we can use aligned_on_parent. What's
more, aligned_on_parent should expose more consistent interface
for the users - affected grobs should now behave similarly to
LyricTexts and DynamicTexts. For example, this

\relative c'' \context Voice {
  \compressFullBarRests
  \override MultiMeasureRestNumber.self-alignment-X = #RIGHT
  R1*100
}

will result in right edge of the MultiMeasureRestNumber being
aligned on the right edge of the MultiMeasureRest.

Affected grobs:
- AccidentalSuggestion
- ClefModifier
- DoublePercentRepeatCounter
- Fingering
- GridLine
- MultiMeasureRestNumber
- MultiMeasureRestText
- PercentRepeatCounter
- StemTremolo

Expected changes in output: none.

Clean up DynamicText horizontal alignment.

Until now, DynamicText alignment was messy, as there were 3 different
callbacks involved - some of them interacting in a confusing way:
 - in define-grobs.scm, X-offset property was initialized to
   ly:self-alignment-interface::x-aligned-on-self,
 - Dynamic_engraver called set_center_parent on every DynamicText
   attached to a note, so that half a NoteHead width was always added
   to its X-offset (producing confusing results for example when
   user requested dynamics to be left-aligned),
 - DynamicTexts living in a Dynamics context used a completely
   different offset callback, which aligned them on NoteColumns.

Since aligned_on_parent is now able to correctly align grobs
with PaperColumn parents (issue 3254), we can use it for all
DynamicTexts and have a single interface for the job.

Expected changes in output: DynamicTexts in Dynamics context
aligned to suspended noteheads may be placed up to 1/4 NoteHead
width further to the right.  This shouldn't be a problem.

Doc-fr: updates to NR

Reg Tests: XML 02a fix incorrect last note duration

Issue 3974

Unofficial MusicXML test suite shows incorrect last not duration
the last note was duration 2 (undotted 64th) but shows dotted 128th

Added some missing <type> elements
Corrected the durations of the last measure

Fixes provided by Patrick Schmidt

Reg Tests: XML 01b has an incorrect number of measures

Issue 3973

01b-Pitches-Intervals.xml from the Unofficial MusicXML
test suite contains only a single measure in MusicXML,
but there should be 41.

musicxml2ly is very tolerant here so the error doesn’t
show in the pdf.

Added missing <measure> elements
Fixed a typo in the title
Added missing <accidental> elements (They represent
the actual notated accidentals.  Without these elements
there shouldn’t be any visible accidentals. So this is
actually a bug in musicxml2ly. The bug doesn’t show here
because musicxml2ly interprets the <alter> element which
represents the sounding pitch of a note.)

Fixes provided by Patrick Schmidt

Release: bump VERSION.

Merge remote branch 'origin/master' into release/unstable

Makelsr.py run

For previous commit 7e23569bfbb6502bb2e3d6f09f089fdb968b6d55

Snippet: Update 'Create Delayed Turn' snippet from old syntax

Issue 3970

Updated all the ovverride AccidentalSuggestion settings so that they
now use the later accepted syntax supported by the parser.

font.size and script.size overrides will work with or without the '#'
symbol, so I have removed it in this example just to illustrate this.

Extending: 2.3.2 - Music Function usage

Issue 3955

Added words to sentences for clarity. (Hopefully) Improved sentence
syntax and some minor grammer nits.

Issue 3968: Improve output of scheme-engraver.ly regtest.

Rename argument to avoid preprocessor problems

Before this change, we were getting the following error with GUB:

/home/gub/gub/target/mingw/src/
lilypond-git.sv.gnu.org--lilypond.git-release-unstable/
lily/include/paper-column.hh:62:
error: expected identifier before ',' token
/home/gub/gub/target/mingw/src/
lilypond-git.sv.gnu.org--lilypond.git-release-unstable/
lily/include/paper-column.hh:62:
error: two or more data types in declaration of 'parameter'

David Kastrup suspects that the library/compiler used for bootstrapping
has a macro or reserved word defined for "interface" - changing the name
makes the problem go away.

Minor update to Gregorian section of NR

Issue 3966/2: Allows the user to override the text property of ChordName

This is a reimplementation of issue 2813.

Issue 3966: Revert "Allows the user to override the text property of ChordName"

This reverts commit 0d4c79ec0f5d728ac7fe58b87ccf51079ec33766.

Conflicts:
input/regression/chord-name-override-text.ly
lily/chord-name-engraver.cc

Reverts patch of issue 2813.

Release: update news.

PO: update template.

Release: bump Welcome versions.

Release: bump VERSION_DEVEL.

Adding new snippet generated from previous Makelsr

See commit 499ff80772530b66e59768a4251bde08bd77fab4

My mistake. Sorry.

Doc: Usage: Added a clarifying paragraph about Error Messages

Issue 3873

Added a clarifying paragraph explaining that (the same)
diagnostics message can occur in multiple places for
(the same) input depending on the type and its context.

Makelsr.py run

For previous commit aec840a735b33a4be38b10baeb40f8e660e3d55c

Doc: NR Section 4.3 - reformatting as per CG Guidelines

Start of issue 3960

This section does not adhere to  many aspects of the Contributors'
Guide Documentation policies.

This specific patch therefore is the 'first stage' of this issue
so as to clean up the section before then making changes to
the technical information. There is too much to do for a single patch,
so I expect to have to make a few more additional patches to tidy this
section up completely before the issue can be properly closed.

This patch includes:

* Correct Line lengths

* Added @seealsos where appropriate

* Re-ordered some of the @nodes so that they are grouped more logically

* Moved @subsections to @unnumberedsubsubsecs:
A few of these sections seem to 'talk through the code' so may
need to be improved later on. However, for now I have moved these
subsections to unnumberedsubsubsec under the main 'Page Breaking'
node. Moved one of these new unnumberedsubsucsecs around so that
the sections on the different breaks come before the turning one.

* Created new Snippet:
One subsection was really a glorified snippet so I have made it into
one.

Authors.itexi - Updated Developers list

Issue 3964

Added/updated to Devs Current, Translations and Core Current.

Based on checkins since Feb 2014 until June 2014

Doc-fr: updates LM

 - learning
 - common notation
 - fundamental
 - templates
 - tweaks

use X-parents to align MMRNumbers, MMRTexts and PercentRepeatCounters

It makes more sense to use X-parents than Y-parents for horizontal
alignment. To be able to do this, we have to ensure that X-parents of
MultiMeasureRestNumber, MultiMeasureRestText and PercentRepeatCounter
won't be overwritten by set_bound (these grobs are spanners).  We can
do this because parents of these grobs are spanners as well.

Expected changes in output: none.