Skip to content

BUG: Use itk::SimpleFilterWatcher instead of itk::FilterWatcher.#11

Merged
blowekamp merged 1 commit intoSimpleITK:masterfrom
jhlegarreta:UseSimpleFilterWatcherInLieuOfFilterWatcher
May 15, 2018
Merged

BUG: Use itk::SimpleFilterWatcher instead of itk::FilterWatcher.#11
blowekamp merged 1 commit intoSimpleITK:masterfrom
jhlegarreta:UseSimpleFilterWatcherInLieuOfFilterWatcher

Conversation

@jhlegarreta
Copy link
Copy Markdown

Use itk::SimpleFilterWatcher instead of itk::FilterWatcher.

The itk::FilterWatcher class was removed in favour of
itk::SimpleFilterWatcher (and thus, its header file named
itkFilterWatcher.h was deleted) in this gerrit topic:
http://review.source.kitware.com/#/c/23415/

This bug was identified thanks to the following gerrit topic:
http://review.source.kitware.com/#/c/23428/

Use itk::SimpleFilterWatcher instead of itk::FilterWatcher.

The itk::FilterWatcher class was removed in favour of
itk::SimpleFilterWatcher (and thus, its header file named
itkFilterWatcher.h was deleted) in this gerrit topic:
http://review.source.kitware.com/#/c/23415/

This bug was identified thanks to the following gerrit topic:
http://review.source.kitware.com/#/c/23428/
@jhlegarreta jhlegarreta force-pushed the UseSimpleFilterWatcherInLieuOfFilterWatcher branch from afa4c6c to de7c663 Compare May 14, 2018 20:07
@jhlegarreta
Copy link
Copy Markdown
Author

Sorry, I forgot to add the itknamespace 😬 Hopefully afa4c6c fixes it.

@jhlegarreta
Copy link
Copy Markdown
Author

@blowekamp please review and merge if appropriate. Thanks !

@blowekamp blowekamp merged commit 8e6f364 into SimpleITK:master May 15, 2018
@blowekamp
Copy link
Copy Markdown
Member

Thanks for the update.

@jhlegarreta jhlegarreta deleted the UseSimpleFilterWatcherInLieuOfFilterWatcher branch June 8, 2018 11:59
blowekamp added a commit to blowekamp/ITKSimpleITKFilters that referenced this pull request Apr 23, 2021
COMP: Update GitHub Actions from ITKModuleTemplate
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