Skip to content

Commit

Permalink
enum Av1BlockIntraInter: Remove #[repr(C)], reducing size by 4 by…
Browse files Browse the repository at this point in the history
…tes (#1333)

This is a simpler, fully safe alternative to #1331. It reduces
`Av1BlockIntraInter` from 28 to 24 bytes, and `Av1Block` from 36 to 32
bytes, just like #1331.

I haven't checked memory usage, though, since I'm not sure how to do
that, but since it reduces the type sizes the same, I assume it would
have the same memory savings.
  • Loading branch information
kkysen authored Jul 22, 2024
2 parents 2bd01dc + b62f2cf commit 5f55ed0
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion src/levels.rs
Original file line number Diff line number Diff line change
Expand Up @@ -532,7 +532,6 @@ pub struct Av1BlockInter {
pub tx_split1: u16,
}

#[repr(C)]
pub enum Av1BlockIntraInter {
Intra(Av1BlockIntra),
Inter(Av1BlockInter),
Expand Down

0 comments on commit 5f55ed0

Please sign in to comment.