Skip to content

Change rule G-7240 to cover IN OUT mode only #81

Closed
@PhilippSalvisberg

Description

@PhilippSalvisberg

The title and the description of Rule G-7240 do not match. The description should concentrate on the combined mode IN OUT.

Title:_

G-7240: Avoid using an IN OUT parameter as IN or OUT only.

Description:

By showing the mode of parameters, you help the reader. If you do not specify a parameter mode, the default mode is IN. Explicitly showing the mode indication of all parameters is a more assertive action than simply taking the default mode. Anyone reviewing the code later will be more confident that you intended the parameter mode to be IN / OUT.

The implementation of the validator should change as well (currently it checks if IN or OUT is defined as mode). This lead to some confusion. See discussion in #46.

Metadata

Metadata

Assignees

Labels

bugSomething isn't workingenhancementNew feature or request

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions