Only assign once

This commit is contained in:
Andrew Murray 2019-09-30 21:18:52 +10:00
parent 290189596e
commit 887a7e503c

View File

@ -310,20 +310,18 @@ class IFDRational(Rational):
float/rational/other number, or an IFDRational float/rational/other number, or an IFDRational
:param denominator: Optional integer denominator :param denominator: Optional integer denominator
""" """
self._denominator = denominator if isinstance(value, IFDRational):
self._numerator = value self._numerator = value.numerator
self._val = float(1) self._denominator = value.denominator
self._val = value._val
return
if isinstance(value, Fraction): if isinstance(value, Fraction):
self._numerator = value.numerator self._numerator = value.numerator
self._denominator = value.denominator self._denominator = value.denominator
self._val = value else:
self._numerator = value
if isinstance(value, IFDRational): self._denominator = denominator
self._denominator = value.denominator
self._numerator = value.numerator
self._val = value._val
return
if denominator == 0: if denominator == 0:
self._val = float("nan") self._val = float("nan")