LGTM. Please feel free to ignore (most of) my remarks if you consider such nitpicking as unnecessary :-) https://codereview.appspot.com/563630043/diff/571770043/lily/include/page-bre... File lily/include/page-breaking.hh (right): https://codereview.appspot.com/563630043/diff/571770043/lily/include/page-bre... lily/include/page-breaking.hh:107: Read the large commennt at the top of page-breaking.cc for context. s/commennt/comment/ https://codereview.appspot.com/563630043/diff/571770043/lily/include/page-spa... File lily/include/page-spacing.hh (right): https://codereview.appspot.com/563630043/diff/571770043/lily/include/page-spa... lily/include/page-spacing.hh:101: we add lines. details details what? Looks like something is missing accidentally. https://codereview.appspot.com/563630043/diff/571770043/scm/page.scm File scm/page.scm (right): https://codereview.appspot.com/563630043/diff/571770043/scm/page.scm#newcode51 scm/page.scm:51: of layout settings just like markups inside the music" Final stop missing. https://codereview.appspot.com/563630043/diff/571770043/scm/page.scm#newcode96 scm/page.scm:96: "Add a annotation at the top to STENCIL and return new stencil." s/a/an/ https://codereview.appspot.com/563630043/diff/571770043/scm/page.scm#newcode119 scm/page.scm:119: "add annotations to a stencil, and return result" Add ... result. https://codereview.appspot.com/563630043/diff/571770043/scm/paper-system.scm File scm/paper-system.scm (right): https://codereview.appspot.com/563630043/diff/571770043/scm/paper-system.scm#... scm/paper-system.scm:38: "add stencils for notes to the main stencil, returning the result." s/add/Add/ https://codereview.appspot.com/563630043/diff/571770043/scm/paper-system.scm#... scm/paper-system.scm:89: "Y-ext and next-Y-ext are either skyline-pairs or extents" Final stop missing.
werner's comments
commit 0b9804f7ed4ac285a432c12b0dea5ab9c67a5eb0 Author: Han-Wen Nienhuys <hanwen@lilypond.org> Date: Fri Feb 28 14:01:38 2020 +0100 Add comments to code related to page breaking/layout https://sourceforge.net/p/testlilyissues/issues/5802 http://codereview.appspot.com/563630043