Tom Karwowski

Results 31 comments of Tom Karwowski

Made a simple improvement using `once_cell`, which should do the job. It's hard for me to think of a case where the value would change and caching the previous result...

Hey, I implemented this, simply copying the subset of rayon functionality for `DashMap` and relying on shards stored in `ReadOnlyVIew.map.shards`. Had to `pub(self)` or `pub(crate)` in 3 places in total,...

Decided to give this a go. Two things worth discussing: - Should we reserve space for `additional` entries in each shard, resulting in at least `self.shards.len() * additional` space allocated,...

I have some experience with doing exacltly the `reqwest -> hyper` transition. I also proposed very similar change to another project that suffered from the same issue https://github.com/open-telemetry/opentelemetry-rust/issues/843. You can...

It would seem that merging https://github.com/durch/rust-s3/pull/234 first, might simplify, or even remove the need for this work. I'll wait and see how that PR proceeds before working on this.

You can let me pick this up if you want. I'll be happy to do so.

@durch I did it, but now that I see it, I'm not sure if it's worth it. Your call.

At work, we use [just](https://github.com/casey/just/) with a command called `pre-commit`. It encompasses `fmt`, `clippy` and `test`. In general, tools like `just` should be preferred over pure bash scripts for more...

I figured out it would be easier for `cargo-binstall` to support versions without leading `v`, rather than go repo by repo and override the default paths. They added it straight...

I was surprised to see `hyper` not utilizing `clippy` to it's fullest. I usually run projects with `pedantic` enabled, and it has been nothing but valuable to me. I already...