You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
While it is enforced by several rules that there must be a whitespace before / after certain tokens, it should also be enforced that there is only one. E.g.:
The line if ~strcmp(valid,'Valid')
is fixed to if ~strcmp(valid, 'Valid')
(adding the missing space), but if ~strcmp(valid, 'Valid')
is not touched at all and not marked as an style issue. Another constructions that have the same problem:
if conditionA && conditionB myVar = 25 / 14;
My guess is that this is a generalized thing. It also happens when tabs are replaced. The tabs are converted to tabwidth spaces, but then these extra spaces are left behind. E.g.: myVar = 25 /\t14;
is transformed into myVar = 25 / 14;
After having a quick look at the code, if I were to fix this I would start by changing the implementation of all the whitespace rules to ensure that there is one (and only one) whitespace.
Another option, probably a bit more "brute force", would involve removing all instances of more than 1 space in the code at the beginning of the style checks, normalizing all text to use a single whitespace character. Then, the code that does the alignment can add extra spaces wherever is right, at the beginning of a line.
Note: I would be happy to work on a fix, but thought of reporting first in case the approach is not optimal or there are better suggestions.
The text was updated successfully, but these errors were encountered:
MISS_HIT Component affected: Style checker
Environment: MATLAB R2022b
System: Windows
Python: Python 3.10.4
While it is enforced by several rules that there must be a whitespace before / after certain tokens, it should also be enforced that there is only one. E.g.:
The line
if ~strcmp(valid,'Valid')
is fixed to
if ~strcmp(valid, 'Valid')
(adding the missing space), but
if ~strcmp(valid, 'Valid')
is not touched at all and not marked as an style issue. Another constructions that have the same problem:
if conditionA && conditionB
myVar = 25 / 14;
My guess is that this is a generalized thing. It also happens when tabs are replaced. The tabs are converted to
tabwidth
spaces, but then these extra spaces are left behind. E.g.:myVar = 25 /\t14;
is transformed into
myVar = 25 / 14;
After having a quick look at the code, if I were to fix this I would start by changing the implementation of all the whitespace rules to ensure that there is one (and only one) whitespace.
Another option, probably a bit more "brute force", would involve removing all instances of more than 1 space in the code at the beginning of the style checks, normalizing all text to use a single whitespace character. Then, the code that does the alignment can add extra spaces wherever is right, at the beginning of a line.
Note: I would be happy to work on a fix, but thought of reporting first in case the approach is not optimal or there are better suggestions.
The text was updated successfully, but these errors were encountered: