lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Fix calculation of X-extent of AccidentalPlacement. (issue 94980043)


From: karol
Subject: Re: Fix calculation of X-extent of AccidentalPlacement. (issue 94980043)
Date: Sat, 3 May 2014 15:23:22 -0700 (PDT)

Hi David. Does your patch fix issue 612? 
  
https://code.google.com/p/lilypond/issues/detail?id=612 
  
Reviewers: , Message: Please review.  Thanks! Description: Fix calculation 
of X-extent of AccidentalPlacement. Previously, the offsets used for staggered 
placement of accidentals were not included in the calculation of the width of 
AccidentalPlacement grobs. Please review this at 
https://codereview.appspot.com/94980043/ Affected files (+1, -1 lines):   
 M lily/accidental-placement.cc Index: lily/accidental-placement.cc diff 
--git a/lily/accidental-placement.cc b/lily/accidental-placement.cc index 
  
f954defc652d89dd9cc354051c76d165fb399cb1..6fa61e967de6a74cf0e62cf89848bab3bbc1259a
   100644 --- a/lily/accidental-placement.cc +++ 
b/lily/accidental-placement.cc @@ -418,7 +418,7 @@ position_apes (Grob *me, 
  
        {             Real mh 
= ape->horizontal_skylines_[d].max_height ();         
    if (!isinf (mh)) -           
 width.add_point (mh); +           
 width.add_point (mh + offset);           } 
        last_offset = offset; 
_______________________________________________ lilypond-devel mailing list 
[hidden email]  https://lists.gnu.org/mailman/listinfo/lilypond-devel   

If you reply to this email, your message will be added to the discussion below: 
http://lilypond.1069038.n5.nabble.com/Fix-calculation-of-X-extent-of-AccidentalPlacement-issue-94980043-tp162038.html
 
To start a new topic under Dev, email address@hidden  To unsubscribe from Dev, 
click here .  NAML 








--
View this message in context: 
http://lilypond.1069038.n5.nabble.com/Fix-calculation-of-X-extent-of-AccidentalPlacement-issue-94980043-tp162038p162048.html
Sent from the Dev mailing list archive at Nabble.com.


reply via email to

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