[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5719 Tie form

labels: --> Fixed_2_21_0
status: Started --> Fixed
Patch: push -->
Comment:

commit 937e413c0f399b2ec44785b7ca29d61cf7b24cff
Author: Dan Eble &lt;address@hidden&gt;
Date:   Fri Jan 31 13:48:35 2020 -0500

    Issue 5719/4: Tie_formatting_problem: [i] instead of at (i)

    ... in loops where it is clear that the index is in bounds.
    at () checks the index, which is unnecessary in these places.

commit 6321ffc39a276d0de4b13bfa105fbc7940278959
Author: Dan Eble &lt;address@hidden&gt;
Date:   Fri Jan 31 11:54:18 2020 -0500

    Issue 5719/3: ties-&gt;front () instead of ties-&gt;at (0)

    ... in contexts where ties-&gt;back () is also used.
    at () checks the index, which is unnecessary in these places.

commit f7fdee569ca0929d58bce6d75960f419e941d393
Author: Dan Eble &lt;address@hidden&gt;
Date:   Fri Jan 31 10:47:49 2020 -0500

    Issue 5719/2: Replace flower&#39;s Tuple with std::array

commit c4879542d28410081c7dcc6ab198897594054ffb
Author: Dan Eble &lt;address@hidden&gt;
Date:   Fri Jan 31 10:13:07 2020 -0500

    Issue 5719/1: int-&gt;vsize to fix tie-formatting warnings

[issues:#5719] Tie formatting maintenance

Status: Fixed
Labels: Fixed_2_21_0
Created: Fri Jan 31, 2020 09:22 PM UTC by Dan Eble
Last Updated: Tue Feb 04, 2020 06:15 AM UTC
Owner: Dan Eble

1: int->vsize to fix tie-formatting warnings

2: Replace flower's Tuple with std::array

3: ties->front () instead of ties->at (0)
... in contexts where ties->back () is also used.

4: Tie_formatting_problem: [i] instead of at (i)
... in loops where it is clear that the index is in bounds.

at () checks the index, which is unnecessary in those places.

https://codereview.appspot.com/581560049

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.

From:	Auto mailings of changes to Lily Issues via Testlilyissues-auto
Subject:	[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5719 Tie formatting maintenance
Date:	Tue, 04 Feb 2020 21:09:52 -0000