gpu-allocator icon indicating copy to clipboard operation
gpu-allocator copied to clipboard

Avoid allocating strings when formatting allocation byte-sizes

Open nical opened this issue 1 year ago • 3 comments

It's not particularly important but while I was adding integration in wgpu for the allocator reports and duplicating a bit of the formatting code in the process, it was rubbing me the wrong way to allocate strings in fmt_bytes when a Display impl could do the same without allocating, so I made the change there. Now I wrote the slightly better version, might as well contribute it back to its original spot!

nical avatar Jul 18 '24 16:07 nical

Seems quite coincidental that that Rust release notes suggest a nice way of writing this loop too:

https://blog.rust-lang.org/2024/07/25/Rust-1.80.0.html#exclusive-ranges-in-patterns

MarijnS95 avatar Jul 25 '24 16:07 MarijnS95

Seems quite coincidental that that Rust release notes suggest a nice way of writing this loop too:

Nice. Is it worth the msrv bump, though? wgpu has to lag a few rustc versions behind, so I'd appreciate if we shelf the nicer version for later.

nical avatar Jul 29 '24 07:07 nical

Nice. Is it worth the msrv bump, though? wgpu has to lag a few rustc versions behind, so I'd appreciate if we shelf the nicer version for later.

Wasn't intending to bump just to use .. (as the article suggests, ..= is already possible with a separate constant), just that it was so well timed :)

MarijnS95 avatar Oct 15 '24 09:10 MarijnS95