Doc: Bar_engraver is required for system-start clefs (2694)

Doc: Bar_engraver is required for system-start clefs (2694)

Add sentence to doc string of Bar_engraver

[Trevor Daniels, 2012-08-20 07:36:44]

This minor change hardly needs the full review
machinery, especially as I've checked it with
a make doc.  

Should I should push directly to staging?

Trevor

[dak, 2012-08-20 08:13:32]

http://codereview.appspot.com/6446160/diff/1/lily/bar-engraver.cc
File lily/bar-engraver.cc (right):

http://codereview.appspot.com/6446160/diff/1/lily/bar-engraver.cc#newcode119
lily/bar-engraver.cc:119: " is required to trigger the creation of system-start
clefs.",
system-start is either an identifier or identifier component, in which case it
should be written @code{system-start} or @samp{system-start}, respectively, or
it is plain English, in which case it should be written system start, namely
without a dash in the middle.

[Trevor Daniels, 2012-08-20 09:32:18]

On 2012/08/20 08:13:32, dak wrote:

> or it is plain English, in which case it should be written 
> system start, namely without a dash in the middle.

It /is/ plain English; system-start is a compound 
adjective of clef.  Whether it is hyphenated or not is
a matter of personal preference.  Rather than argue
such a minor detail I'll change it to "clefs at the start
of systems."  I'll not rerun make doc though, as this
is an overnight run on my laptop.

[dak, 2012-08-20 09:50:33]

On 2012/08/20 09:32:18, Trevor Daniels wrote:
> On 2012/08/20 08:13:32, dak wrote:
> 
> > or it is plain English, in which case it should be written 
> > system start, namely without a dash in the middle.
> 
> It /is/ plain English; system-start is a compound 
> adjective of clef.

Shouldn't it then be system-starting?

> Whether it is hyphenated or not is
> a matter of personal preference.  Rather than argue
> such a minor detail I'll change it to "clefs at the start
> of systems."

Even better.  This does not make the reader fear that he has been missing the
definition of a technical term.

[Graham Percival, 2012-08-21 11:53:45]

LGTM

[Trevor Daniels, 2012-08-24 11:27:45]

Pushed to staging as
bd73258c56a29a0e585c9653a2ba941390bd19d9
Closing