Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

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

Copy link
Copy link
Closed
@PhilippSalvisberg

Description

@PhilippSalvisberg
Issue body actions

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 workingSomething isn't workingenhancementNew feature or requestNew feature or request

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions

    Morty Proxy This is a proxified and sanitized view of the page, visit original site.