by default, slint() could focus on code maintenance and actual errors, instead of on code style
Bug Description
slint() is the only scilab tool to detect deprecated functions to be urgently updated/converted, and to detect really problematic sequences like forgotten pause.
Currently, all its features are enabled by default. This make important results drowned in some much less important ones, like many code style criteria (by the way sometimes debatable as implemented).
IMHO, only the following criteria could be enabled by default:
- Redefinition of reserved keywords (2)
- Variable: valid name (3)
- Functions: valid names (4)
- Unassigned output argument (7)
- deprecated functions (22)
- Deprecated @ (not) operator (27)
- save/load with variables names (26)
- printf: number of data matches the format (14)
- illegal calls (35)
- Bad use of ':' operator (24)
- == instead of = (19)
- Implicit list a:b without %nan nor %inf bounds (24)
- struct: fields and data match (25)
Other features should be turned off in the default slint.xml
Edited by Samuel GOUGEON