Larry Ruane

Results 228 comments of Larry Ruane

Force pushed for needed rebase, then again for CI failure. @dergoegge can you take another look?

@rkrux - Thanks for the review and the suggestions, force-pushed them.

Please consider editing the description slightly, if my understanding is correct. >With this PR, if the inbound connection is on our whitelist we start the eviction process by selecting a...

Even though not needed for this PR, you may want to consider adding a commit to remove the unnecessary templating. It's perfectly reasonable not to make this change in this...

Force pushed to fix CI failure, thanks @fanquake

Force pushed (twice) to fix CI, thanks @MarcoFalke. One other small change I made in this force push is that if the category value is beyond the size (range) of...

Force pushed for [review comment](https://github.com/bitcoin/bitcoin/pull/26697#discussion_r1051382151), thanks @MarcoFalke!

Force pushed 40e1923e856661fdf68cf783ed9c6d1931dfbdf3 to address review comments (thanks, @stickies-v!): - remove the unnecessary comment on `#include ` line - simplify `EnableCategory()` and `DisableCategory()`, and remove the calls to `Assume()` -...

Thanks for the reviews, @ajtowns and @codobtc, and for the benchmarking @kouloumos. I think now that #26619 (closed in favor of the current PR) may actually be preferable because the...

Thanks, @ajtowns and @jonatack, I force pushed ac3c16c6978370c9c5ea4c591acbbbcf626804a6 to include Anthony's suggested atomic bitset, which is really nice. I only changed a few of the variable names to be a...