|
From: | Auto mailings of changes to Lily Issues via Testlilyissues-auto |
Subject: | [Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5894 Shortcut Rational addition if either operand is zero |
Date: | Fri, 17 Apr 2020 12:19:50 -0000 |
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 <address@hidden> 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 address@hidden 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 address@hidden https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
[Prev in Thread] | Current Thread | [Next in Thread] |