Re: [Lilypond-auto] Issue 4242 in lilypond: make \shiftOff useful

[lilypond]

Comment #5 on issue 4242 by address@hidden: make \shiftOff useful
https://code.google.com/p/lilypond/issues/detail?id=4242

Well, in ly/property-init.ly I read

    shiftOn   = \override NoteColumn.horizontal-shift = #1
    shiftOnn  = \override NoteColumn.horizontal-shift = #2
    shiftOnnn = \override NoteColumn.horizontal-shift = #3
    shiftOff  = \revert NoteColumn.horizontal-shift

I am sure that Data of Starship fame would agree that the proper naming is

    shiftO    = \override NoteColumn.horizontal-shift = #0

However, this is a bit too cute for my taste.

Taking a look at other forms of *Off in that file, I find

autoBeamOn  = \set autoBeaming = ##t
autoBeamOff = \set autoBeaming = ##f

(can't revert because not a layout property)

balloonLengthOn = {
  \override BalloonTextItem.extra-spacing-width = #'(0 . 0)
  \override BalloonTextItem.extra-spacing-height = #'(-inf.0 . +inf.0)
}
balloonLengthOff = {
  \override BalloonTextItem.extra-spacing-width = #'(+inf.0 . -inf.0)
  \override BalloonTextItem.extra-spacing-height = #'(0 . 0)
}

bassFigureExtendersOn = {
  \set useBassFigureExtenders = ##t
  \set Staff.useBassFigureExtenders = ##t
}
bassFigureExtendersOff = {
  \set useBassFigureExtenders = ##f
  \set Staff.useBassFigureExtenders = ##f
}

cadenzaOn  = \set Timing.timing = ##f

cadenzaOff = \set Timing.timing = ##t

easyHeadsOn = {
  \override NoteHead.stencil = #note-head::brew-ez-stencil
  \override NoteHead.font-size = #-8
  \override NoteHead.font-family = #'sans
  \override NoteHead.font-series = #'bold
}
easyHeadsOff = {
  \revert NoteHead.stencil
  \revert NoteHead.font-size
  \revert NoteHead.font-family
  \revert NoteHead.font-series
}

harmonicsOn =
#(define-music-function (parser location) ()
   (_i "Set the default note head style to a diamond-shaped style.")
   (override-head-style '(NoteHead TabNoteHead) 'harmonic))
harmonicsOff = \defaultNoteHeads  % is equal to  
(revert-head-style '(NoteHead TabNoteHead))

palmMuteOff = \defaultNoteHeads

xNotesOff = \defaultNoteHeads

deadNotesOff = \xNotesOff

So precedent is mixed.

At any rate, if we think the clashing notehead warning is useful, we need a  
command for getting it.  The current patch does not provide any setting  
fitting into the pattern of commands for this functionality.  And if we  
don't think it is useful, it should not be there.

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings