SFML / SFML / 8736987266 / 10

60%

master: 60%

push

github

Revert change to `Drawable::draw` function signature

This change was made in 359fe90 due to recommendations from tooling.
On its face this change makes sense since it removes a copy that
isn't always necessary. In practice it caused ergonomic issues due
to now being forced to make a copy of the render states when needed.

The performance gains of eliding this copy are unsubstantiated. We
have not done any profiling to measure its impact. For lack of such
measurements I'd rather err on the side of improved user experience.
If future benchmarks prove this copy is rather expensive then we
can reconsider removing it with that evidence in mind.

1822 of 3138 branches covered (58.06%)

Branch coverage included in aggregate %.

6336 of 11730 relevant lines covered (54.02%)

1365.66 hits per line