-
-
Notifications
You must be signed in to change notification settings - Fork 51
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
Reinstate lost style checks #1591
Conversation
Personally I’ve spent approaching 25 years being completely happy with plain We know I would really like |
alire_common.gpr
Outdated
"-gnatyu", -- no unnecessary blank lines | ||
"-gnatyx", -- no extra parens around conditionals | ||
"-gnaty-s"); -- relax fwd decl | ||
"-gnatyg", -- Standard GNAT style = -gnatyydISuxz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One potential risk with gnatyg is that its definition can change in the future, which can bring some unwanted style errors.
Why not use our default style check switches for Alire crates?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess I prefer to make an sporadic fix if that happens rather than being unaware of recommended style changes.
No love for |
But I'm seeing a different problem now which is that different compilers understand different switches, so it's better to go for a explicit list in the end, so I'll take Fabien's suggestion. |
It seems the changes in #1497 removed quite more style checks than intended. In regard to @simonjwright concerns in that PR, I think we can just disable that specific check once it is enabled by default in
-gnatyg
.