Re: Fix 1112. (issue1670042)

2010-06-22 Thread n . puttock
On 2010/06/21 18:28:52, joeneeman wrote: We read it in several places, but we never set it to anything other than zero. So it's ready to be used as soon as we decide to set it to something better than zero, but I don't know what that is yet... Ah, I see. So the same applies to between_sys

Re: Fix 1112. (issue1670042)

2010-06-21 Thread joeneeman
Reviewers: Neil Puttock, Message: Thanks, fixed. I'll push after "make check" finishes... http://codereview.appspot.com/1670042/diff/1/3 File lily/constrained-breaking.cc (right): http://codereview.appspot.com/1670042/diff/1/3#newcode384 lily/constrained-breaking.cc:384: Line_details for anyth

Fix 1112. (issue1670042)

2010-06-20 Thread n . puttock
LGTM. http://codereview.appspot.com/1670042/diff/1/2 File input/regression/page-breaking-min-distance.ly (right): http://codereview.appspot.com/1670042/diff/1/2#newcode1 input/regression/page-breaking-min-distance.ly:1: \version "2.13.22" 2.13.26 http://codereview.appspot.com/1670042/diff/1/2#