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.

Sign up for GitHub

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

Jump to bottom

HAL: remove all boolean fields from settings structures #60

Open

jnaulet opened this issue Jul 11, 2023 · 0 comments

Labels

enhancement good first issue

Owner

jnaulet commented Jul 11, 2023

For better readability, i suggest to remove all booleans from the xxx_setup() arguments.
This would make structure inits more readable for the casual reviewer, not having to guess what this "true/false" statement is all about.

This is a generalisation of issue #59

jnaulet added enhancement good first issue labels

jnaulet added a commit that referenced this issue


          uart hal: removed all booleans, issue #60

b4d5ef3

jnaulet added a commit that referenced this issue


          uart: fix all drivers according to issue #60

e706902

jnaulet added a commit that referenced this issue


          demo: fix uart hal changes following issue #60

8a29f48

jnaulet added a commit that referenced this issue


          gpio hal: fix accord to issue #60

41525a9

jnaulet added a commit that referenced this issue


          drivers: gpio fixed according to hal issue #60

afd146b

jnaulet added a commit that referenced this issue


          demo: gpio hal fix, issue #60

16afb8c

jnaulet added a commit that referenced this issue


          can: removed stdbool from deps, issue #60

ffdf569

jnaulet added a commit that referenced this issue


          dma: removed stdbool from deps, issue #60

2d356dc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment