Skip to content

COMP: Remove unused variable#50

Merged
jhlegarreta merged 1 commit intoInsightSoftwareConsortium:mainfrom
jhlegarreta:FixUnusedVariableWarning
Apr 14, 2025
Merged

COMP: Remove unused variable#50
jhlegarreta merged 1 commit intoInsightSoftwareConsortium:mainfrom
jhlegarreta:FixUnusedVariableWarning

Conversation

@jhlegarreta
Copy link
Copy Markdown
Member

Remove unused variable.

Fixes:

Modules/Remote/LabelErodeDilate/include/itkLabelSetUtils.h:260:37:
 warning: unused variable 'initPixelValue' [-Wunused-variable]

  260 |   const typename TInIter::PixelType initPixelValue = 0.0;
      |                                     ^~~~~~~~~~~~~~

raised for example in:
https://open.cdash.org/viewBuildError.php?type=1&buildid=10316578

Introducing the variable was an oversight in commit 108b0ba.

@jhlegarreta
Copy link
Copy Markdown
Member Author

My bad. Apologies.

Remove unused variable.

Fixes:
```
Modules/Remote/LabelErodeDilate/include/itkLabelSetUtils.h:260:37:
 warning: unused variable 'initPixelValue' [-Wunused-variable]

  260 |   const typename TInIter::PixelType initPixelValue = 0.0;
      |                                     ^~~~~~~~~~~~~~
```

raised for example in:
https://open.cdash.org/viewBuildError.php?type=1&buildid=10316578

Introducing the variable was an oversight in commit 108b0ba.
@jhlegarreta jhlegarreta force-pushed the FixUnusedVariableWarning branch from e405913 to 0e7998c Compare April 13, 2025 18:07
Copy link
Copy Markdown
Member

@blowekamp blowekamp left a comment

Choose a reason for hiding this comment

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

💯

@jhlegarreta jhlegarreta merged commit 8adf735 into InsightSoftwareConsortium:main Apr 14, 2025
23 checks passed
@jhlegarreta jhlegarreta deleted the FixUnusedVariableWarning branch April 14, 2025 12:37
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