Skip to content

Conversation

charliechiou
Copy link
Contributor

@charliechiou charliechiou commented Apr 11, 2025

The condition ">=" is used to align all version checks, improving code consistency and readability.

Summary by Bito

This pull request enhances code consistency and readability by aligning the version check for the class_create function in static_key.c with chardev.c and standardizing version checks across multiple files. The condition was modified from '<' to '>=', improving overall code quality, logical consistency, and maintainability across related files.

Unit tests added: False

Estimated effort to review (1-5, lower is better): 2

@charliechiou charliechiou changed the title Align class_create version check with chardev.c Align version check with chardev.c Apr 11, 2025
@jserv
Copy link
Contributor

jserv commented Apr 12, 2025

The condition was modified from '<' to '>=', improving overall code quality and ensuring logical consistency across related files.

You should revise all C source files to make the above statement practically fit.

The condition ">=" is used to align all version checks, improving code
consistency and readability.
@charliechiou charliechiou changed the title Align version check with chardev.c Align version check Apr 12, 2025
@jserv jserv merged commit 16e1b72 into sysprog21:master Apr 18, 2025
1 check passed
@jserv
Copy link
Contributor

jserv commented Apr 18, 2025

Thank @charliechiou for contributing!

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