Skip to content

COMP: Remove NthElementPixelAccessor::operator= for VariableLengthVector#5911

Merged
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:mainfrom
N-Dekker:Remove-NthElementPixelAccessor-operator=-VariableLengthVector
Mar 10, 2026
Merged

COMP: Remove NthElementPixelAccessor::operator= for VariableLengthVector#5911
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:mainfrom
N-Dekker:Remove-NthElementPixelAccessor-operator=-VariableLengthVector

Conversation

@N-Dekker
Copy link
Copy Markdown
Contributor

@N-Dekker N-Dekker commented Mar 9, 2026

Let the assignment operator be implicitly defaulted (compiler generated), for the VariableLengthVector specific template specialization of NthElementPixelAccessor.

Aims to address warnings appearing on Linux/GCC 11.4.0 saying:

warning: implicitly-declared 'constexpr NthElementPixelAccessor::NthElementPixelAccessor(const NthElementPixelAccessor&)' is deprecated [-Wdeprecated-copy]

Let the assignment operator be implicitly defaulted (compiler generated), for the
VariableLengthVector specific template specialization of NthElementPixelAccessor.

Aims to address warnings appearing on Linux/GCC 11.4.0 saying:

    warning: implicitly-declared 'constexpr NthElementPixelAccessor::NthElementPixelAccessor(const NthElementPixelAccessor&)' is deprecated [-Wdeprecated-copy]

Follow-up to pull request InsightSoftwareConsortium#5849
commit 00ead18
"COMP: Let NthElementPixelAccessor follow the Rule of Zero"
@github-actions github-actions Bot added type:Compiler Compiler support or related warnings area:Core Issues affecting the Core module labels Mar 9, 2026
@N-Dekker N-Dekker marked this pull request as ready for review March 9, 2026 20:22
@hjmjohnson hjmjohnson merged commit 26a0767 into InsightSoftwareConsortium:main Mar 10, 2026
11 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 type:Compiler Compiler support or related warnings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants