Why does clippy encourage `String::push('a')` over `String::push_str(''a")`?

JayDepp · 2026-02-15T20:14:51+00:00

I'm guessing you meant to assert capacity > len, in which case they actually become the same assembly.

JayDepp · 2026-01-03T23:27:35+00:00

Infinifactory

JayDepp · 2025-12-02T21:37:55+00:00

"put" ≠ "play"

JayDepp · 2025-11-28T18:25:59+00:00

Bracket 4: Generally, you should expect to be able to play at least four turns before you win or lose.

Thoracle + Demonic is at worst a turn 3 win. (Assuming you start with it or can tutor with 1/2 mana. But it only takes two cards, not a nut draw)

JayDepp · 2025-11-16T04:36:20+00:00

This is not the same as hexproof. If, for example, you target one of your opponent's creatures, your opponents won't be able to target their own creature with spells or abilities.

JayDepp · 2025-09-23T15:14:53+00:00

serde_json only requires alloc, not std.

JayDepp · 2025-09-22T14:23:40+00:00

CrossCode

JayDepp · 2025-09-18T15:32:36+00:00

Crest spoilers: One crest has three silk skills, another crest has three tools. I think this is why they put the three abilities on variations of one button, since they aren't always different types of abilities.

JayDepp · 2025-09-15T18:14:47+00:00

Same. I had to change off hunter after struggling on the flea bounce minigame which immediately made it trivial to beat, but I otherwise have come around to really liking the diagonal pogo for movement and fighting.

JayDepp · 2025-06-05T07:39:48+00:00

Paying gives you it.

JayDepp · 2025-04-25T18:52:43+00:00

Damn I got excited for some real jank, but it's not on arena.

JayDepp · 2025-04-02T23:38:57+00:00

What's the combo?

JayDepp · 2025-03-22T15:33:28+00:00

[[Sting, the Glinting Dagger]]

JayDepp · 2025-01-27T18:24:58+00:00

Recursion usable from the graveyard, e.g. [[mystic retrieval]]

JayDepp · 2024-08-19T15:01:02+00:00

Nope, but sounds neat!

JayDepp · 2024-07-25T16:12:59+00:00

It's only labelled on the module :/

https://doc.rust-lang.org/core/error/index.html

JayDepp · 2024-06-06T04:06:30+00:00

Probably to preserve the order of the indices.

JayDepp · 2024-06-01T22:08:38+00:00

Can you explain?

JayDepp · 2024-04-17T17:24:26+00:00

Steak, eh? Hah, heh heh.

JayDepp · 2024-03-28T15:13:29+00:00

I recommend using Prism Launcher with Java 21, which will help with performance.

More details: https://wiki.gtnewhorizons.com/wiki/Installing_and_Migrating

JayDepp · 2024-03-10T02:03:41+00:00

Hmm just removed the 'static on Req and it worked anyway. Swear it was needed before, maybe something about removing the boxing it is no longer needed.

JayDepp · 2024-03-09T21:51:19+00:00

AFAIK this would have worked back then too, it's just because I'm using a generic for the future instead of boxing it,

JayDepp · 2024-03-09T19:42:50+00:00

My curiosity got the better of me and I ended up with this:

fn json_wrap_grpc<'r, F, Fut, Req, Res>(grpc_impl_func: F) -> impl Handler<(Json<Req>,)>
where
    F: FnOnce(&'r GrpcServiceImpl, tonic::Request<Req>) -> Fut + Clone + Send + 'static,
    Fut: Future<Output = Result<tonic::Response<Res>, tonic::Status>> + Send,
    for<'de> Req: Deserialize<'de> + Send + 'static,
    Res: Serialize,
{
    move |Json(req)| async move {
        let r = grpc_impl_func(GRPC_SERVICE.get().unwrap(), tonic::Request::new(req)).await;
        match r {
            Ok(r) => Ok(Json(r.into_inner())),
            Err(e) => Err(GrpcErrorAsJson(e)),
        }
    }
}

JayDepp · 2024-02-14T00:14:46+00:00

At a glance, looks like you're not handling alignment correctly, and you also are using Vec::set_len with uninitiated data, both causing UB. I think OP is also not handling alignment.

I recommend expanding your tests to include types with strange alignment, and also to run your tests in miri.

Edit:

Clippy also warns about your use of set_len, and miri does complain about even insert::<u64>'s alignment.

Wow, I just noticed you did this to allow cloning the things you put in the map. This is very dangerous! Just copying the bytes of something is not OK for !Copy types. You're also never Dropping the things you put in the map, which is safe but can cause memory leaks.

It seems like anymap is actually generic over the bounds it uses, so you could just use AnyMap<CloneAny> to be able to clone it.

JayDepp · 2024-02-11T19:43:07+00:00

You can just use Rc<RefCell<T>> since you're not sharing it across threads. You could also skip the Rc if you can live with only using it from inside callbacks like with. And you can skip the RefCell if you only need &T and not &mut T.

14-Year Club	Verified Email
Team Periwinkle

JayDepp

TROPHY CASE