Suggested patch to LilyJAZZ.ily (bar number engraving)

[Jim Long]

To whomever the keeper of the flame is for LilyJAZZ.ily:

First, thank you!

My default stylesheet normally turns off bar number engraving.
But today I happened to be working on a jazz-style chart and
wanted to add it back, so I did that in the \layout {} block:

      \context { \Score \consists "Bar_number_engraver" }

No joy.  Still no bar numbers.

Some process-of-elimination led me to discover that LilyJAZZ.ily
defeats bar number engraving by nullifying the stencil.

IMO, the better way to do this would be to:

a) have LilyJAZZ be neutral about bar numbering, and not force it
on or off;

so that

b) the user can use or not use bar numbers by adding or removing
the bar number engraver.

I doubt the line numbers are exact, but I think this context diff
will be clear.  The patch could likely be improved by deleting
the line altogether.

Thanks again,

Jim


--- LilyJAZZ.ily        2013/10/07 02:25:22     1.1
+++ LilyJAZZ.ily        2013/12/28 23:19:07
@@ -346,17 +346,17 @@
 makeUnpurePureContainer =
         #(ly:make-unpure-pure-container
        ly:grob::stencil-height
        (lambda (grob start end) (ly:grob::stencil-height grob)))


 jazzOn = {
   \override Staff.Clef #'stencil = #jazz-clef
-  \override Score.BarNumber #'stencil = ##f
+%  \override Score.BarNumber #'stencil = ##f

   \override Staff.KeySignature #'stencil = #jazz-keysig
   \override Staff.KeyCancellation #'stencil = #jazz-keysig

   \override Staff.TimeSignature #'stencil = #jazz-timesig

   \override Staff.NoteHead #'stencil = #jazz-notehead
   \override Staff.NoteHead #'Y-extent = \makeUnpurePureContainer