Skip to content

DOC: Avoid using initialization lists in coding style snippets#201

Merged
jhlegarreta merged 1 commit intoInsightSoftwareConsortium:masterfrom
jhlegarreta:AvoidUsingInitializationListsInSnippets
Jul 17, 2023
Merged

DOC: Avoid using initialization lists in coding style snippets#201
jhlegarreta merged 1 commit intoInsightSoftwareConsortium:masterfrom
jhlegarreta:AvoidUsingInitializationListsInSnippets

Conversation

@jhlegarreta
Copy link
Member

Avoid using initialization lists in coding style code snippets: initialization lists are not encouraged as per current ITK style guidelines.

Similarly, avoid using a boolean ivar setter method in the constructor, as ivars should be initialized when they are declared.

@github-actions github-actions bot added area:Appendices Issues affecting the Appendices part language:LaTeX Changes to LaTeX code type:BookStyle Changes to book style files type:Documentation Documentation improvement or change labels Jul 16, 2023
Avoid using initialization lists in coding style code snippets:
initialization lists are not encouraged as per current ITK style
guidelines.

Similarly, avoid using a boolean ivar setter method in the constructor,
as ivars should be initialized when they are declared.
@jhlegarreta jhlegarreta force-pushed the AvoidUsingInitializationListsInSnippets branch from 9d21cdc to 35a15c2 Compare July 16, 2023 23:56
@jhlegarreta jhlegarreta merged commit 8804cfe into InsightSoftwareConsortium:master Jul 17, 2023
@jhlegarreta jhlegarreta deleted the AvoidUsingInitializationListsInSnippets branch July 17, 2023 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Appendices Issues affecting the Appendices part language:LaTeX Changes to LaTeX code type:BookStyle Changes to book style files type:Documentation Documentation improvement or change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants