Authors: Updated Authors.itexi

Issue 4118

From June 2014 until September 2014

Based on all those contributors for 2.18.x
and current 2.19.x

Issue 4126: Provide a --enable-guile2 option in configure

For convenient experimenting with GUILEv2, we want to have everything in
master, so provide a configure option that will allow the switch to
GUILEv2 compilation.  Off by default since, well, lots of stuff does not
work yet.

Issue4129/3: Let safe-utility-defs.scm avoid bugs in (ice-9 curried-definitions)

The exporting definitions in the GUILEv2 module
(ice-9 curried-definitions) were buggy until quite recently.

Issue 4129/2: Use curried-definitions in titling-init and safe-utility-defs

Issue 4129/1: Load (ice-9 curried-definitions) in ly/init.scm and documentation-lib.scm

The load is made conditionally for GUILEv2 where curried definitions
need this library.

Issue 4125: Disable character set recognition

GUILE 1.8 presumably does not make use of the LC_CTYPE information and
just does its input/output/string processing in 8bit chunks.  For the
sake of getting comparable results at first when working on GUILE 2.0,
we switch off character set recognition here.

At a later point of time, integration with the character processing of
GUILE 2.0 may be attempted but since this is an entirely new can of
worms with its own performance pitfalls, we put this aside for now.

Issue 4121/2: Store markup command argument list names in docstring

For one thing, the doc string does not make a lot of sense without the
argument list and it cannot reliably be deduced from the doc string
itself.

For another, GUILEv2 does not do us the favor of storing the original
argument list anywhere where it could be retrieved with a useful amount
of labor.

While it would be nice to put the types of the arguments into the doc
string as well, the signature's actual predicates are only evaluated
after macro expansion and that is too late for generating the doc
string.

So we just stick the argument list (after uncurrying) as a line at the
top.

Issue 4121/1: Store music function argument list names in docstring

For one thing, the doc string does not make a lot of sense without the
argument list and it cannot reliably be deduced from the doc string
itself.

For another, GUILEv2 does not do us the favor of storing the original
argument list anywhere where it could be retrieved with a useful amount
of labor.

While it would be nice to put the types of the arguments into the doc
string as well, the signature's actual predicates are only evaluated
after macro expansion and that is too late for generating the doc
string.

So we just stick the argument list (after uncurrying) as a line at the
top.

Issue 4116: Disable ASSERT_LIVE_IS_ALLOWED in GUILEv2

Apparently the garbage collection in GUILEv2 can do a mark pass over
material without actually protecting it from collection, possibly
because the garbage collector can run in a separate thread.  At least,
the attempt to print such remaining live smobs still seen by a mark pass
leads to crashes, so disable this memory leak detection mechanism on
GUILEv2.

Issue 4115: Call (ice-9 format) with #f as second argument

Our own format in (lily) will substitute the #f by itself when necessary, but in
scm/document-context-mods.scm, we are explicitly asking for (ice-9 format)

Issue 4114: Rearrange load order of markup macros and definitions

The markup macro has to be defined before use, and any of its arguments
have to be defined before use as well.

Issue 4124: Move make-engraver macro to output-lib.scm

GUILEv2 is a bit picky about macros being defined before use, so we need
to define this one early.

Issue 4123: markup macro must not be used in output-lib.scm

It is not defined at that time, and GUILEv2 is not forgiving about using
macros before definition.

Updates instructions in CG for LilyPad development

Issue 4110: Patch up GUILE 2.0 SCM_SMOB_OBJECT*_LOC API, broken 2009-2014

Issue 4111: (scm accreg): don't use markup definition and markup macro in the same form

GUILEv2 is not happy about that.

Issue 4100: IR: Fix subproperty and default-child descriptions in context docs

Issue 4102: misaligned composer in LP 2.19.14

This problem surfaced in the wake of the fix for issue 3855.  It is due
to Stencil::translate not moving an empty expression even if it has
non-empty extents so that

\markup fill-line { "" "" "composer" }

which assembles a line piece by piece and determining the offset of the
next piece by looking at the end of the assemblage so far lost the
offset between the first and second "" in the line.

Remove redundant check for empty X interval

Issue4107/2: Score_performer::prepare: precomputed_recurse_over_translators should do START, not STOP

Issue 4107/1: Use START/STOP for *recurse_over_translators instead of the counterintuitive DOWN/UP

Fixes bog in Lookup::bezier_sandwich, tethering sandwiches at their extremal ends with a line (issue 4087)

Doc: NR and CG - fix some Doc Compilation warnings in logs

contributor.splittexi.log:
Fixed some sectioning warnings

notation.splittexi.log:
removed @rinternals reference to non-existant node in fr, es
and en changing-defaults.itely files.

Doc: LM - correct @example for Tweak Command

Issue 4101

Corrected existing 'long form' example for Syntax Consistency

Issue 4094: Check for Note_column interface before using grob.

Rest_collision was calling Note_column functions on grobs that did not
have the Note_column interface.  Now Rest_collision ignores such grobs
and Note_column will print a more accurate message if such an error
ever happens again.

Release: bump VERSION.

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

Don't use incomplete template type Smob2 in Unpure_pure_container

Some compilers complain, and I think they might have a point.

Release: bump Welcome versions.

Release: update news.

PO: update template.

Release: bump VERSION_DEVEL.

Merge branch 'issue4097' into HEAD

Add regtest for \addlyrics on Staff context

Allow \addlyrics to work with arbitrary contexts

Revert "Convert \new Staff ... \addlyrics into \new Staff \new Voice ... \addlyrics"

This reverts commit 76f256075f0a4f481d7e11c19dd55ab09030fe5b.

Allow associatedVoice to be of different type than Voice

Derive melisma_busy state from child contexts when available

Merge branch 'issue4096' into HEAD

Add regtest for issue 4096, \addlyrics using existing contexts

Convert \new Staff ... \addlyrics into \new Staff \new Voice ... \addlyrics

Give \addlyrics the power of christening the associated context

With

\new Voice \with { \stemDown } { c1 } \addlyrics { Oh }

as opposed to

\new Voice = "upper" \with { \stemDown } { c1 } \addlyrics { Oh }

\addlyrics created its own context (consequently not using \stemDown)
since it had no context name for the Lyrics context to attach to.  In
this case (namely being preceded by an unnamed new Voice context), it
will now give a name to the context itself.

Fix bugs preventing \addlyrics from finding an existing context to attach to

Refactor grammar to give \addlyrics a good target to attach to

So far,

    \new Staff \new Voice { c1 } \addlyrics { Oh } \addlyrics { Ah }

was factored as

    \new Staff \new Voice { { c1 } \addlyrics { Oh } \addlyrics { Ah } }

which resulted in something like

   \new Staff \new Voice
     << \context Voice = "uniqueContext0" { c1 }
        \new Lyrics \lyricsto "uniqueContext0" { Oh }
        \new Lyrics \lyricsto "uniqueContext0" { Ah }
     >>

This made it impossible to work with the likes of

  \new Staff \new Voice \with { \stemUp } { c1 } \addlyrics ...

since the voice the lyrics attached to was always a new voice.
Fixing the grouping in the grammar is a first required step for getting
this under control.

Refactor grammar, folding re_rhythmed_music into composite_music

Refactor grammar, folding complex_music into composite_music

Add regtest for \tagGroup command

Basic documentation for \tagGroup command

Issue 4083: Implement \tagGroup command

Issue 346: monochord issues

This lets a reverse notehead at the stem start creep up closer to its
same-pitch unreversed partner in order to avoid a gap caused by notehead
slant.

Doc-fr: full update

Makelsr.py run

For commit ID a1c9c3d285a2f3e59cf79a375b67cfb58ceca7ba

Doc: NR Pitches.itely - Clef - updated snippets and text

Issue 3976

Added 2 new snippets:
1 snippet added to NR
1 snippet added just to Snippets

Modified 1 existing snippet:
This was already (and still is) included in the NR

Deleted 2 snippets (and incorporated them in the
main text in @lilyponds)

Added new @lilypond example that was originally
described (but not show)n in the 'Tweaking Clef
Properties' snippet to show how to remove
the clef glyph from the end of a line. As this
'tweak' is relatively simple (a single \set and
\unset) and more than likely to be used (IMHO) by
the general user than the listed items in the
snippet, I thought it better to be in the NR than
another example in either an already long snippet
or by creating yet another snippet.

Some minor linelength formatting as per CG.

Issue 3855: programming error with \markup \fill-line { \vspace #10 }

Issue 4092: Limit tuplet bracket length to actual tuplet length

See the regtest for an example of the problem.  Without the patch, the
tuplet bracket extends over all of

  \tuplet 3/2 4 { g16 a b } c8

including c8, resulting in visual lengths not adding up.

Issue 4030: Lyrics associated to a Staff are placed on top of each other

The approach here is simply to avoid attaching to noteheads that have
actually started _before_ the current moment of time.  It is somewhat
irritating that includeGraceNotes cannot possibly be supported in
associated voices with its own timing (rather than the fully
synchronized \addlyrics or \lyricsto): Instead you need to use the same
durations, including \grace constructs, in order to match the note you
are aiming for.

Issue 3832: react sanely to ill-formed version strings

Mostly, this cleans up Lilypond_version

We no longer convert to int because it is weird, but we still want
version comparisons.  Conversion to bool for checking validity seems
nice in addition.

And bombing out because of invalid data to String_convert::dec2int
seemed definitely non-nice.  This version requires specifying at
least major and minor version number explicitly separated by dots.

It accepts a trailing dot, too.  A trailing extra_patch_string after
major.minor.patchlevel. is also accepted but not interpreted.

Don't verify result of String_convert::dec2int

The error behavior is not useful and triggers on things like "00".

Make "Don't know how to parenthesize spanners." warning, not programming error

Also add location information.

Issue 3849: \parenthesize\p generates programming error

Register a grob as the cause for DynamicLineSpanner rather than an event.

Issue 4028: Automatic beaming of grace notes inhibits beam subdivision

While the whole extend_mom_ logic seems rather fishy to me (it probably
should be reset to a sane value at some other location per-beam) and
I don't have an actual idea why the Grace_auto_beam_engraver would
be involved at all here, initializing extend_mom_ to a value before
all grace notes seems to do the trick for this report.

Issue 4086/6: Reimplement unpure-pure-containers in terms of Simple_smob and Smob1

Issue 4086/5: Introduce callable Smob infrastructure

Issue 4086/4: Reimplement Simple_closure using Smob1

Issue 4086/3: Reimplement Music_function in terms of Smob2

Issue 4086/2: Add include/small-smobs.hh for smobs without extra allocation

Issue 4086/1: Reimplement Input in terms of Simple_smob

Issue 4091: Lexer: don't go through int for fractions

GUILE can deal fine with things like
\void \displayScheme 6563546345635436/3654635463456
if one does not route through C code.

Replace internal_get_property with get_property where possible

Note that thanks to lily-guile-macros magic, we can use get_property
with both string and SCM input (see lily/include/lily-guile-macros.hh:48).

Initialize main_head to 0 (in Note_column::calc_main_extent)

Make sure that the pointer won't be assigned a random value.

Ottava Bracket bounds; issue 3690

Ottava Bracket: do not dash edges; issue 3866

note-collision: rework horizontal shifts; issue 1774,2196

Also allow duplicate 'horizontal-shift's without the "too many clashing
note-columns" warning.

Issue 4078: Doc: Use variables rather than instrument definitions

  Using variables for switching character names in vocal music
  rather than instrument definitions is easier to understand,
  easier to set up and quicker to type.  It also frees up the
  use of instrument definitions so they may be tailored for
  better use in instrumental compositions.

  Remove the mention of \transposition.  This is never required
  in vocal music.

  Adjust references to match the changes.

Merge branch 'issue4082'

Revert "Issue 4082/2: Add scripts/auxiliar/smob-convert.sh for one-time C++ conversion"

This reverts commit b91616a79ea28aed15a3c7e9c801a843ae38753f.

The script is no longer needed.

Issue 4082/3: Run scripts/auxiliar/smob-convert.sh

Issue 4082/2: Add scripts/auxiliar/smob-convert.sh for one-time C++ conversion

This script is used for the mechanical stage of converting the Smob
machinery from working via the C preprocessor to working via C++
templates.

Issue 4082/1: Reimplement Smobs via templates rather than preprocessor

This creates the underlying code for the new implementation without
converting the bulk of the code (which is left to a script committed
separately).  The include file ly-smobs.icc is removed completely.  A
new file smobs.tcc contains generic template instantiations.  The GCC
implementation works by loading it unconditionally in smobs.hh.
Depending on the underlying template mechanism, it might be feasible to
include it into just one compilation unit.

Where the previous implementation referred to class names passed into
macros, the typeinfo mechanism of C++ is employed for deriving the
respective name.  The GCC implementation uses some cursory demangling of
the resulting type id, converting "3Box" into "Box" and similarly.
Other APIs might warrant different polishing but the type names are
mostly used for internal purposes and differences are not all that
problematic.

Don't assign 'safe' values to self_scm_ outside of smobify_self

Minor style fixes to some class definitions.

Release: bump VERSION.

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

LSR updates

Update release process in CG

Release: bump Welcome versions.

Release: update news.

PO: update template.

Release: bump VERSION_DEVEL.

Doc-fr: full update

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

Issue 4075: Doc: Correct the instrumentTransposition setting in NR 2.1.6

  The value of instrumentTransposition affects the MIDI
  pitch.  As all the notes in this example are entered
  at the correct sounding pitch no instrument transposition
  is required.  Previously Kaspar sounded an octave too low.

Doc: CG - Added Ralph Palmer and Federico to Bug Squad

Issue 4080

Doc-ca: Priority II Catalan translation

Remove tied accidentals after line-breaking

Simplify the NullVoice context, and move it under Score

issues 3834 3825 3734
The Tie_performer and similar give their timing information to
Lyric_performer without any need for Note_performer.

Simplify the NullVoice context

issues 3834 3825 3734

Issue 4071: scheme-tutorial.itely: missing closing paren

But it turns out that the "formal" definition of a list given there is
unsuitable for determining the list status of circular lists because the
definition, well, turns out to be circular.

Use aligned-on-x-parent instead of other callbacks for some grobs

Now that we have an easy way of specifying different alignments for
grob and its parent (see 0c4f221e5d), we can use aligned-on-.-parent
in places that we previously used .-aligned-on-self.  With this change,
the users have more control over the placement of grobs.

Affected grobs:
- InstrumentSwitch
- Script
- SostenutoPedal
- SustainPedal
- UnaCordaPedal

Fix bugs with aligning on main part of the NoteColumn

NoteColumns may contain either noteheads or rests, so the function
was changed to handle the case when NoteColumn contains only rest.

ly_scm2bool returns true when the argument is unset - that's not
what we want: unset properties should evaluate to false.

Fix ugly code in ClefModifier alignment