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

Rename a variable plus some editing of comments

Browse source
This commit is contained in:
Eduard Braun 2017-02-24 02:55:44 +01:00
parent a7e7b4c21d
commit 2ebe9741b2
1 changed files with 13 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

29
scour/scour.py
View file

@ -2554,13 +2554,13 @@ def serializeTransform(transformObj):
) )
def scourCoordinates(data, options, forceCommaWsp=False, cmd=''): def scourCoordinates(data, options, force_whitespace=False, cmd=''):
""" """
Serializes coordinate data with some cleanups: Serializes coordinate data with some cleanups:
- removes all trailing zeros after the decimal - removes all trailing zeros after the decimal
- integerize coordinates if possible - integerize coordinates if possible
- removes extraneous whitespace - removes extraneous whitespace
- adds spaces between values in a subcommand if required (or if forceCommaWsp is True) - adds spaces between values in a subcommand if required (or if force_whitespace is True)
""" """
if data is not None: if data is not None:
newData = [] newData = []
@ -2571,13 +2571,12 @@ def scourCoordinates(data, options, forceCommaWsp=False, cmd=''):
scouredCoord = scourUnitlessLength(coord, scouredCoord = scourUnitlessLength(coord,
needsRendererWorkaround=options.renderer_workaround, needsRendererWorkaround=options.renderer_workaround,
isControlPoint=cp) isControlPoint=cp)
# only need the comma if the current number starts with a digit # don't output a space if this number starts with a dot (.) or minus sign (-); we only need a space if
# (numbers can start with - without needing a comma before) # - this number starts with a digit
# or if forceCommaWsp is True # - this number starts with a dot but the previous number had *no* dot or exponent
# or if this number starts with a dot and the previous number # i.e. '1.3 0.5' -> '1.3.5' or '1e3 0.5' -> '1e3.5' is fine but '123 0.5' -> '123.5' is obviously not
# had *no* dot or exponent (so we can go like -5.5.5 for -5.5,0.5 # - 'force_whitespace' is explicitly set to 'True'
# and 4e4.5 for 40000,0.5) if c > 0 and (force_whitespace
if c > 0 and (forceCommaWsp
or scouredCoord[0].isdigit() or scouredCoord[0].isdigit()
or (scouredCoord[0] == '.' and not ('.' in previousCoord or 'e' in previousCoord)) or (scouredCoord[0] == '.' and not ('.' in previousCoord or 'e' in previousCoord))
): ):
@ -2588,13 +2587,11 @@ def scourCoordinates(data, options, forceCommaWsp=False, cmd=''):
previousCoord = scouredCoord previousCoord = scouredCoord
c += 1 c += 1
# What we need to do to work around GNOME bugs 548494, 563933 and # What we need to do to work around GNOME bugs 548494, 563933 and 620565, is to make sure that a dot doesn't
# 620565, which are being fixed and unfixed in Ubuntu, is # immediately follow a command (so 'h50' and 'h0.5' are allowed, but not 'h.5').
# to make sure that a dot doesn't immediately follow a command # Then, we need to add a space character after any coordinates having an 'e' (scientific notation),
# (so 'h50' and 'h0.5' are allowed, but not 'h.5'). # so as to have the exponent separate from the next number.
# Then, we need to add a space character after any coordinates # TODO: Check whether this is still required (bugs all marked as fixed, might be time to phase it out)
# having an 'e' (scientific notation), so as to have the exponent
# separate from the next number.
if options.renderer_workaround: if options.renderer_workaround:
if len(newData) > 0: if len(newData) > 0:
for i in range(1, len(newData)): for i in range(1, len(newData)):