marcus/scour
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Move Tree_of_Life_SVG.svg (revision 188) from unittests to fulltests. Whoops!

Browse source 
Partial fix for bug 603987: Allow curves representing straight lines to become LineTo commands (and possibly HorizontalLineTo and VerticalLineTo afterwards).

Fix a bug whereby an initial M0,0 in path data is lost, making the path unrenderable.
This commit is contained in:
Louis Simard 2010-07-16 22:04:22 -04:00
parent 939c7e1b97
commit 87b7908465
2 changed files with 4 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

0
unittests/Tree_of_life_SVG.svg → fulltests/Tree_of_Life_SVG.svg
View file

Side by side Swipe Overlay

Before

Width:  |  Height:  |  Size: 2.6 MiB

After

Width:  |  Height:  |  Size: 2.6 MiB

Before After
Before After

7
scour.py
View file

@ -1672,8 +1672,9 @@ def cleanPath(element, options) :
if cmd in ['m','l','t']: if cmd in ['m','l','t']:
if cmd == 'm': if cmd == 'm':
# remove m0,0 segments # remove m0,0 segments
if data[0] == data[i+1] == 0: if pathIndex > 0 and data[0] == data[i+1] == 0:
# 'm0,0 x,y' can be replaces with 'lx,y' # 'm0,0 x,y' can be replaces with 'lx,y',
# except the first m which is a required absolute moveto
path[pathIndex] = ('l', data[2:]) path[pathIndex] = ('l', data[2:])
numPathSegmentsReduced += 1 numPathSegmentsReduced += 1
else: # else skip move coordinate else: # else skip move coordinate
@ -1736,7 +1737,7 @@ def cleanPath(element, options) :
foundStraightCurve = True foundStraightCurve = True
else: else:
m = dy/dx m = dy/dx
if p1y == m*p1x and p2y == m*p2y: if p1y == m*p1x and p2y == m*p2x:
foundStraightCurve = True foundStraightCurve = True
if foundStraightCurve: if foundStraightCurve: