Skip to content

STYLE: Remove obsolete __GNUC__ macro checks for GCC version 4#3738

Merged
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:masterfrom
N-Dekker:Remove-__GNUC__-4-checks
Nov 19, 2022
Merged

STYLE: Remove obsolete __GNUC__ macro checks for GCC version 4#3738
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:masterfrom
N-Dekker:Remove-__GNUC__-4-checks

Conversation

@N-Dekker
Copy link
Copy Markdown
Contributor

GCC 4 support was dropped with pull request #2563 commit 4e812d6
"COMP: Require compiler versions that support C++14", June 3, 2021

GCC 4 support was dropped with pull request InsightSoftwareConsortium#2563
commit 4e812d6
"COMP: Require compiler versions that support C++14", June 3, 2021
@github-actions github-actions bot added area:Core Issues affecting the Core module type:Style Style changes: no logic impact (indentation, comments, naming) type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct labels Nov 14, 2022
@N-Dekker N-Dekker marked this pull request as ready for review November 14, 2022 13:06
Copy link
Copy Markdown
Member

@thewtex thewtex left a comment

Choose a reason for hiding this comment

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

Looks good, but let's wait until after v5.3.0 has been tagged to merge, please.

@hjmjohnson hjmjohnson added this to the ITK 5.3.1 milestone Nov 15, 2022
@hjmjohnson
Copy link
Copy Markdown
Member

/azp macOS

@hjmjohnson
Copy link
Copy Markdown
Member

/azp run ITK.macOS

@hjmjohnson hjmjohnson merged commit ce57f30 into InsightSoftwareConsortium:master Nov 19, 2022
@N-Dekker
Copy link
Copy Markdown
Contributor Author

@hjmjohnson Thanks for merging, Hans, I think the commit is fine as well, but wasn't it supposed to wait until after v5.3.0? Anyway, I don't expect any problem with this commit.

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:Style Style changes: no logic impact (indentation, comments, naming) type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants