Create rule S7173: "GoSub" statements should not be used #4580
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
sebastien-marichal
changed the title
sebastien-marichal
force-pushed
the
rule/add-RSPEC-S7173
branch
from
6d97e37 to
05cc2be
Compare
rules/S7173/vb6/rule.adoc
Outdated
| === Exceptions | ||
|
|
||
| `On Error GoSub` statements are ignored as correct error handling. |
There was a problem hiding this comment.
It is an exception for GoTo so I kept it but I am not sure if this valid or even still relevant.
There was a problem hiding this comment.
If I'm not mistaken error handlers have to be inside a function / sub to be available with GoTo so I would strongly guess that GoSub is not possible in this context.
rules/S7173/vb6/rule.adoc
Outdated
| === Exceptions | ||
|
|
||
| `On Error GoSub` statements are ignored as correct error handling. |
There was a problem hiding this comment.
If I'm not mistaken error handlers have to be inside a function / sub to be available with GoTo so I would strongly guess that GoSub is not possible in this context.
|
|
You can preview this rule here (updated a few minutes after each push).
Review
A dedicated reviewer checked the rule description successfully for: