- **labels**: --> Fixed_2_21_0
- **status**: Started --> Fixed
- **Patch**: push -->
- **Comment**:
```text
commit 937e413c0f399b2ec44785b7ca29d61cf7b24cff
Author: Dan Eble <nine.fierce.ball...@gmail.com>
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.
```
```text
commit 6321ffc39a276d0de4b13bfa105fbc7940278959
Author: Dan Eble <nine.fierce.ball...@gmail.com>
Date: Fri Jan 31 11:54:18 2020 -0500
Issue 5719/3: ties->front () instead of ties->at (0)
... in contexts where ties->back () is also used.
at () checks the index, which is unnecessary in these places.
```
```text
commit f7fdee569ca0929d58bce6d75960f419e941d393
Author: Dan Eble <nine.fierce.ball...@gmail.com>
Date: Fri Jan 31 10:47:49 2020 -0500
Issue 5719/2: Replace flower's Tuple with std::array
```
```text
commit c4879542d28410081c7dcc6ab198897594054ffb
Author: Dan Eble <nine.fierce.ball...@gmail.com>
Date: Fri Jan 31 10:13:07 2020 -0500
Issue 5719/1: int->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 testlilyissues-a...@lists.sourceforge.net 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
testlilyissues-a...@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto