- **labels**: --> Fixed_2_21_1
- **status**: Started --> Fixed
- **Patch**: needs_work -->
- **Comment**:
Han-Wen accidentally pushed this, but it turned out that the latest patch set
already contained the modifications requested by Dan.
~~~
commit fb98d6be7ed06969a8eeab7b974e8734d8cd21b1
Author: Han-Wen Nienhuys <[email protected]>
Date: Sun Apr 12 13:32:01 2020 +0200
Shortcut Rational addition if either operand is zero
This is useful, because Moment contains 2 Rationals, but the grace
Rational is usually zero.
In a GProf profile of the Carver MSDM score, before 3.14% of
cumulative time was from Rational::operator+=(). Afterwards, it
represents 1.02% of cumulative time.
~~~
---
** [issues:#5894] Shortcut Rational addition if either operand is zero**
**Status:** Fixed
**Labels:** Fixed_2_21_1
**Created:** Sun Apr 12, 2020 12:34 PM UTC by Han-Wen Nienhuys
**Last Updated:** Fri Apr 17, 2020 12:07 PM UTC
**Owner:** Han-Wen Nienhuys
In a GProf profile of the Carver MSDM score, before 3.14% of
cumulative time was from Rational::operator+=(). Afterwards, it
represents 1.02% of cumulative time.
https://codereview.appspot.com/551690046
---
Sent from sourceforge.net because [email protected] is
subscribed to https://sourceforge.net/p/testlilyissues/issues/
To unsubscribe from further messages, a project admin can change settings at
https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is
a mailing list, you can unsubscribe from the mailing list._______________________________________________
Testlilyissues-auto mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
