Re: typo/oversight in align-interface.cc and page-layout-problem.cc (issue 115770043 by thomasmorle...@gmail.com)

2014-07-22 Thread markpolesky 


https://codereview.appspot.com/115770043/diff/1/lily/page-layout-problem.cc
File lily/page-layout-problem.cc (left):

https://codereview.appspot.com/115770043/diff/1/lily/page-layout-problem.cc#oldcode38
lily/page-layout-problem.cc:38: Returns the number of footntoes
associated with a given line.
footntoes?  Not handnfingers?

https://codereview.appspot.com/115770043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: typo/oversight in align-interface.cc and page-layout-problem.cc (issue 115770043 by thomasmorle...@gmail.com)

2014-07-22 Thread James 

On 22/07/14 09:52, markpole...@gmail.com wrote:


https://codereview.appspot.com/115770043/diff/1/lily/page-layout-problem.cc 


File lily/page-layout-problem.cc (left):

https://codereview.appspot.com/115770043/diff/1/lily/page-layout-problem.cc#oldcode38 


lily/page-layout-problem.cc:38: Returns the number of footntoes
associated with a given line.
footntoes?  Not handnfingers?

or lookenpeepers

http://en.wikipedia.org/wiki/Blinkenlights

:)

Sorry, couldn't resist.

James

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

typo/oversight in align-interface.cc and page-layout-problem.cc (issue 115770043 by thomasmorle...@gmail.com)

2014-07-13 Thread thomasmorley65 

Reviewers: ,

Message:
please review

Description:
typo/oversight in align-interface.cc and page-layout-problem.cc

issue 4008

fix oversight in doc-string of align-interface.cc
fix typo in comment of page-layout-problem.cc

Please review this at https://codereview.appspot.com/115770043/

Affected files (+2, -2 lines):
  M lily/align-interface.cc
  M lily/page-layout-problem.cc


Index: lily/align-interface.cc
diff --git a/lily/align-interface.cc b/lily/align-interface.cc
index  
f9e0e06345187c2534cc69528d130e5bd40359fb..36bfec72a4854cfe7be89f50ceae08fde4b76bdb  
100644

--- a/lily/align-interface.cc
+++ b/lily/align-interface.cc
@@ -367,7 +367,7 @@ Align_interface::set_ordered (Grob *me)
 ADD_INTERFACE (Align_interface,
Order grobs from top to bottom, left to right, right to  
left

 or bottom to top.  For vertical alignments of staves, the
-@code{break-system-details} of the left
+@code{line-break-system-details} of the left
 @rinternals{NonMusicalPaperColumn} may be set to tune
 vertical spacing.,

Index: lily/page-layout-problem.cc
diff --git a/lily/page-layout-problem.cc b/lily/page-layout-problem.cc
index  
52720d54490738c1579e09d218b0a6cf688f786b..3a1a84cfc5add27da5165e2f8bcffe7399040bbf  
100644

--- a/lily/page-layout-problem.cc
+++ b/lily/page-layout-problem.cc
@@ -35,7 +35,7 @@
 #include text-interface.hh

 /*
- Returns the number of footntoes associated with a given line.
+ Returns the number of footnotes associated with a given line.
 */

 vectorGrob *



___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: typo/oversight in align-interface.cc and page-layout-problem.cc (issue 115770043 by thomasmorle...@gmail.com)

2014-07-13 Thread david . nalesnik 

LGTM.

https://codereview.appspot.com/115770043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel