Skip to content

refactor: introduce custom bitflags! macro#393

Merged
Lencerf merged 1 commit intogoogle:mainfrom
Lencerf:refactor/bitflags
Feb 8, 2026
Merged

refactor: introduce custom bitflags! macro#393
Lencerf merged 1 commit intogoogle:mainfrom
Lencerf:refactor/bitflags

Conversation

@Lencerf
Copy link
Collaborator

@Lencerf Lencerf commented Feb 8, 2026

Define a custom bitflags! macro to simplify bitflag definitions and ensure compatibility with external derive macros like zerocopy.

This new macro wraps bitflags::bitflags!, automatically derives the Debug trait, and introduces a more concise and idiomatic syntax.

This change is applied throughout the codebase, replacing direct invocations of the bitflags::bitflags! macro.

@Lencerf Lencerf force-pushed the refactor/bitflags branch 3 times, most recently from fc53c09 to 7dfda39 Compare February 8, 2026 19:53
@coveralls
Copy link

coveralls commented Feb 8, 2026

Pull Request Test Coverage Report for Build 21804828362

Details

  • 5 of 5 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.005%) to 26.595%

Totals Coverage Status
Change from base Build 21794413518: -0.005%
Covered Lines: 3669
Relevant Lines: 14288

💛 - Coveralls

Define a custom `bitflags!` macro to simplify bitflag definitions and
ensure compatibility with external derive macros like `zerocopy`.

This new macro wraps `bitflags::bitflags!`, automatically derives the
`Debug` trait, and introduces a more concise and idiomatic syntax.

This change is applied throughout the codebase, replacing direct
invocations of the `bitflags::bitflags!` macro.

Signed-off-by: Changyuan Lyu <changyuanl@google.com>
@Lencerf Lencerf merged commit 0c2b31c into google:main Feb 8, 2026
5 checks passed
@Lencerf Lencerf deleted the refactor/bitflags branch February 8, 2026 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants