[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Lilypond-auto] Issue 4468 in lilypond: stencil-integrate.cc: root o
From: |
lilypond |
Subject: |
Re: [Lilypond-auto] Issue 4468 in lilypond: stencil-integrate.cc: root out "slopes" |
Date: |
Sun, 05 Jul 2015 08:35:42 +0000 |
Updates:
Status: Fixed
Labels: -Patch-push Fixed_2_19_23
Comment #9 on issue 4468 by address@hidden: stencil-integrate.cc: root
out "slopes"
https://code.google.com/p/lilypond/issues/detail?id=4468
Well, I trust myself. I ran all the files manually and compared manually.
It would just have been more convincing to any other reviewers of the
regtest differences if the skylines would have been there. But frankly, I
don't think there were any but you and me.
Pushed to staging as
commit 33e433beb994aaa770c552e714ae83823dfe3300
Author: David Kastrup <address@hidden>
Date: Tue Jun 30 07:48:24 2015 +0200
stencil-integrate.cc: wrap several overfull lines
commit 5ce3603640c820f4f18c3e0e32631d158e14fa72
Author: David Kastrup <address@hidden>
Date: Tue Jun 30 11:44:50 2015 +0200
Issue 4468/2: Avoid skyline interruptions at line caps
Line caps are made to slightly overhang into their corresponding line in
order to avoid numerics causing a gap. Such a gap is unlikely to be
relevant for positioning but is ungainly when using -ddebug-skylines.
commit 8cb9ecce8fe4de67c776603dacc9c7f270b3c8bf
Author: David Kastrup <address@hidden>
Date: Tue Jun 30 00:04:49 2015 +0200
Issue 4468/1: stencil-integrate.cc: root out "slopes"
They are quite unsuitable for the computational geometry performed here.
Their use rendered the code inefficient, obtuse, and unreliable.
Contains several other fixes like removing duplicate calculations and
wrong values. The C++ <complex> module is no longer used. It had only
been used for calculating sin and cos in an obscure manner.
commit f3f291c3c4411f9e73b59bfff94af0be60b43934
Author: David Kastrup <address@hidden>
Date: Mon Jun 29 10:48:29 2015 +0200
Robustify Offset::direction and Offset::length
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings