https://codereview.appspot.com/22750045/diff/20001/lily/optimal-page-breaking.cc
File lily/optimal-page-breaking.cc (right):

https://codereview.appspot.com/22750045/diff/20001/lily/optimal-page-breaking.cc#newcode78
lily/optimal-page-breaking.cc:78: if (min_sys_count > ideal_sys_count)
// subtraction wrapped around
How about
if (min_sys_count > ideal_sys_count // subtraction wrapped around
   || min_sys_count <= 0)
  min_sys_count = 1;

Saves you the min thing.  <= instead of == just so that we don't really
need to know that vsize is unsigned.

https://codereview.appspot.com/22750045/

_______________________________________________
lilypond-devel mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to