Skip to content

Align version check #301

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 18, 2025
Merged

Align version check #301

merged 1 commit into from
Apr 18, 2025

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