Skip to content

COMP: Fix warning: unused parameter 'argv'#93

Merged
dzenanz merged 1 commit intoInsightSoftwareConsortium:mainfrom
dzenanz:fixWarning
Mar 16, 2026
Merged

COMP: Fix warning: unused parameter 'argv'#93
dzenanz merged 1 commit intoInsightSoftwareConsortium:mainfrom
dzenanz:fixWarning

Conversation

@dzenanz
Copy link
Copy Markdown
Member

@dzenanz dzenanz commented Mar 16, 2026

ITK/Modules/Remote/Cuberille/test/CuberilleTest_Issue66.cxx:112:40: warning: unused parameter 'argv' [-Wunused-parameter]

ITK/Modules/Remote/Cuberille/test/CuberilleTest_Issue66.cxx:112:40: warning: unused parameter 'argv' [-Wunused-parameter]

int
CuberilleTest_Issue66(int argc, char * argv[])
CuberilleTest_Issue66(int, char **)
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.

[] -> * may cause issues

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It does cause issues. I wasn't sure whether *[] works without identified between them. I tried it locally, and it does. I pushed the update before I saw this remark.

@dzenanz dzenanz merged commit a152f89 into InsightSoftwareConsortium:main Mar 16, 2026
20 checks passed
@dzenanz dzenanz deleted the fixWarning branch March 16, 2026 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants