Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(82)

Issue 576110043: break substitution cleanup

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 weeks ago by hanwenn
Modified:
3 weeks, 6 days ago
Reviewers:
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Break substitution cleanup * Remove global var from break substitution This assumes that we don't need to be that conservative of stack space. This is reasonable because mutable properties and objects live in different alists these days, limiting the amount of recursion. * Remove some conditional checks that have no coverage in the regtest. * Always call do_break_processing(), moving the is_live check into spanner.cc

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -8 lines) Patch
M lily/include/system.hh View 3 chunks +1 line, -2 lines 0 comments Download
M lily/one-line-auto-height-breaking.cc View 1 chunk +0 lines, -1 line 0 comments Download
M lily/one-line-page-breaking.cc View 1 chunk +0 lines, -1 line 0 comments Download
M lily/page-breaking.cc View 1 chunk +0 lines, -2 lines 0 comments Download
M lily/paper-score.cc View 1 chunk +1 line, -2 lines 0 comments Download
M lily/system.cc View 1 chunk +2 lines, -0 lines 0 comments Download

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b