Skip to content

Conversation

DavidMCerdeira
Copy link
Member

@DavidMCerdeira DavidMCerdeira commented Jul 11, 2025

This commit updates mainly memory and bitmap operations to use named macros like MEM_DONT_FREE_PAGES, MEM_ALIGN_PPAGES, and BITMAP_DONT_SET instead of raw boolean values on respective functions.

@DavidMCerdeira DavidMCerdeira force-pushed the ref/avoid_bool_literals_invoke branch 2 times, most recently from ab02e52 to c0aaec7 Compare July 11, 2025 16:39
@josecm josecm self-assigned this Jul 15, 2025
@josecm
Copy link
Member

josecm commented Jul 15, 2025

@DavidMCerdeira looks good so far. Please let me know when its ready for final review

This commit updates mainly memory and bitmap operations to use named macros
like MEM_DONT_FREE_PAGES, MEM_ALIGN_PPAGES, and BITMAP_DONT_SET instead
of raw boolean values.

Signed-off-by: David Cerdeira <[email protected]>
@DavidMCerdeira DavidMCerdeira force-pushed the ref/avoid_bool_literals_invoke branch from c0aaec7 to acf3c60 Compare July 15, 2025 22:42
@DavidMCerdeira DavidMCerdeira marked this pull request as ready for review July 16, 2025 07:40
@DavidMCerdeira DavidMCerdeira requested a review from danielRep July 16, 2025 07:51
@DavidMCerdeira
Copy link
Member Author

@josecm this is ready for review

@josecm josecm requested a review from miguelafsilva5 as a code owner July 25, 2025 10:27
Copy link
Member

@danielRep danielRep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work ! LGTM

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.

3 participants