Fix #786.

Fix #786.

Send a CompletizeExtenderEvent at the end of each lyrics block so that
the Extender_engraver can tie up loose ends with any pending extender.

* input/regression (display-lily-tests.ly):

  add extra space to \lyricsto and \addlyrics test to account for silenced
  display method

* lily/extender-engraver.cc:

  add listener for CompletizeExtenderEvent which will call completize_extender ()
  if necessary

  revert previous fix which checked for empty LyricText grob

* lily/hyphen-engraver.cc:

  revert fix as above

* lily/lyric-engraver.cc (process_music):

  don't create empty LyricText grobs

* scm/define-event-classes.scm (event-classes):

  add completize-extender-event as child of music-event

* scm/define-music-display-methods.scm:

  add display method for CompletizeExtenderEvent (returns empty string)

* scm/define-music-types.scm (music-descriptions):

  add CompletizeExtenderEvent

* scm/ly-syntax-constructors.scm (lyric-combine-music):

  insert CompletizeExtenderEvent at the end of the sequential music (i.e.,
  following the last lyric)

[Neil Puttock, 2010-04-05 23:59:09]

Hi,

This patch implements the suggestion outlined here:

http://lists.gnu.org/archive/html/lilypond-devel/2010-01/msg00120.html

Please review.

Thanks,
Neil

[Trevor Daniels, 2010-04-06 08:13:20]

Typo

http://codereview.appspot.com/885044/diff/1/2
File input/regression/display-lily-tests.ly (right):

http://codereview.appspot.com/885044/diff/1/2#newcode1
input/regression/display-lily-tests.ly:1: \version "2.13.8"
2.13.18

[Neil Puttock, 2010-04-07 22:37:20]

On 2010/04/06 08:13:20, Trevor Daniels wrote:
> Typo
> 
> http://codereview.appspot.com/885044/diff/1/2
> File input/regression/display-lily-tests.ly (right):
> 
> http://codereview.appspot.com/885044/diff/1/2#newcode1
> input/regression/display-lily-tests.ly:1: \version "2.13.8"
> 2.13.18

Done.

Thanks for checking, Trevor.