[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] refactor Ledger_line_spanner::print (issue 29782

From: paulwmorris
Subject: refactor Ledger_line_spanner::print (issue 297820043 by address@hidden)
Date: Thu, 14 Apr 2016 15:42:58 +0000

Reviewers: ,

Please review.  I was working on a scheme version of the ledger line
print routine (for some custom ledger line patterns) and having figuring
out how it worked, I thought I would make some improvements.

Since I'm new to C++ is especially appreciated.


Description: refactor Ledger_line_spanner::print

Optimizations and removal of code duplication.

Only iterate through all note heads once, generating
all relevant data for just note heads involved in
ledger lines.  Then work with that data.

Merge overlapping ledger data before producing
ledger stencils so we only produce one stencil for
each ledger line.

Please review this at

Affected files (+142, -81 lines):
  M lily/

reply via email to

[Prev in Thread] Current Thread [Next in Thread]