aembke / redis-protocol.rs

A Rust implementation of RESP2 and RESP3
Apache License 2.0
41 stars 14 forks source link

Fix PartialEq implementation for RedisProtocolErrorKind #28

Closed rukai closed 7 months ago

rukai commented 8 months ago

Clippy pointed out that this line was fishy. image

On closer inspection it looks like an actual bug. So this PR fixes the amt to actually compare between self and other while cleaning up the match to make it easier to read.

aembke commented 7 months ago

Good find. This should be fixed in 5.0.0 too.