Reinstate lost style checks#1591
Conversation
|
Personally I’ve spent approaching 25 years being completely happy with plain We know I would really like |
| "-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.
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.
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.