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

Issue 4535055: Fixes the assert problem coming from ledger-line-spanner.cc. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 years ago by MikeSol
Modified:
7 years, 11 months ago
Reviewers:
mike, carl.d.sorensen
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Fixes the assert problem coming from ledger-line-spanner.cc.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M lily/ledger-line-spanner.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3
MikeSol
This seems like a quick, easy, and reasonable fix for Issue 1652. Lemme know if ...
8 years ago (2011-05-12 20:06:59 UTC) #1
Carl
LGTM. Carl
8 years ago (2011-05-12 20:08:54 UTC) #2
mike_apollinemike.com
8 years ago (2011-05-12 23:30:53 UTC) #3
On May 12, 2011, at 4:08 PM, Carl.D.Sorensen@gmail.com wrote:

> LGTM.
> 
> Carl
> 
> 
> http://codereview.appspot.com/4535055/


Passes the regtests, so pushed as 69580da523384fc207043e4bad297d7560e24268.

Cheers,
MS
Sign in to reply to this message.

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