Skip to content

STYLE: Standardize class and file naming in src/Developer - #483

Merged
hjmjohnson merged 1 commit into
InsightSoftwareConsortium:mainfrom
ritishkhichi:fix-developer-naming-inconsistencies
Sep 14, 2026
Merged

hjmjohnson merged 1 commit into
InsightSoftwareConsortium:mainfrom
ritishkhichi:fix-developer-naming-inconsistencies

Conversation

@ritishkhichi

@ritishkhichi ritishkhichi commented Sep 14, 2026 •

Copy link
Copy Markdown
Contributor

Closes #316

Fixes naming inconsistencies in src/Developer as discussed in #316.

Changes

  • Renamed ImageFilterMultipleOutputs.*, ImageFilterX.*, and ImageFilterY.* to prepend itk.
  • Updated classes inside itkImageFilterX.* and itkImageFilterY.* from ImageFilter to ImageFilterX and ImageFilterY.
  • Updated corresponding include guards and template includes.
  • Updated referencing source files in src/Core/Common/ and src/Developer/.

@github-actions github-actions Bot added area:Core Issues affecting the Core module type:Style Style changes: no logic impact (indentation, comments, naming) language:C++ Changes to C++ examples labels Sep 14, 2026

@dzenanz dzenanz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good on a glance.

@dzenanz
dzenanz requested a review from hjmjohnson September 14, 2026 17:00
@hjmjohnson
hjmjohnson merged commit 949f509 into InsightSoftwareConsortium:main Sep 14, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Core Issues affecting the Core module language:C++ Changes to C++ examples type:Style Style changes: no logic impact (indentation, comments, naming)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants