Rule: avoidable-count

[anderseknert]

These two are equivalent, as -1 means "to the end of the string". Avoiding the count call is however slightly more efficient, and should be preferred if performance is a concern.

r1 := substring(s, 3, count(s))

r2 := substring(s, 3, -1)

r1

+-------------------------------------------+------------+
| samples                                   |      56862 |
| ns/op                                     |      19973 |
| B/op                                      |      12022 |
| allocs/op                                 |        218 |

r2

+-------------------------------------------+------------+
| samples                                   |      70899 |
| ns/op                                     |      16716 |
| B/op                                      |      10157 |
| allocs/op                                 |        177 |

[anderseknert]

As @srenatus found some time back, the caching of numbers in OPA doesn't work as it was expected. Perhaps this issue would be less relevant if it did, and if that'll be the case later, we won't have to do this.