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

Issue 320320043: Fix dashed line errors

Can't Edit
Can't Publish+Mail
Start Review
Created:
7 years, 1 month ago by david.nalesnik
Modified:
7 years, 1 month ago
Reviewers:
lemzwerg
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Fix dashed line errors There are two errors in lily/line-interface.cc which cause dashed lines to be drawn inaccurately. (1) Line-thickness is improperly subtracted from the "off" value in Line_interface::make_dashed_line. This was done, presumably, to account for the line cap of each dash. In fact, PostScript discounts the line cap when constructing the pattern. (2) Dashed lines are constructed so that they begin and end with a dash. Thus, a dashed line is built of dash + whitespace units and a last lone dash. Period is not adjusted in Line_interface::line for this last dash, which can lead to a noticeable difference in its length compared to other dashes. Correcting these flaws ensures that dashed lines end with a dash rather than whitespace, and that terminating dashes are not clipped. Also, the number of dashes drawn reflects the number calculated.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -8 lines) Patch
M lily/line-interface.cc View 2 chunks +13 lines, -8 lines 0 comments Download

Messages

Total messages: 2
david.nalesnik
Please review. Thanks!
7 years, 1 month ago (2017-03-07 04:46:34 UTC) #1
lemzwerg
7 years, 1 month ago (2017-03-07 13:28:33 UTC) #2
LGTM.
Sign in to reply to this message.

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