SFML / SFML / 8793914308

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.

5479 of 11331 branches covered (48.35%)

Branch coverage included in aggregate %.

6 of 17 new or added lines in 4 files covered. (35.29%)

10347 of 18508 relevant lines covered (55.91%)

6159.41 hits per line