Skip to content
Snippets Groups Projects

Rename Get/Set Variadic[Named]Input[s]() by Get/SetInput() for easier reading in FunctorImageFilter

Merged Julien Michel requested to merge remove_variadic_fom_getset into develop

Summary

This MR proposes to rename the following methods in new filters in Functor module:

  • SetVariadicInput<N>() -> SetInput<N>()
  • GetVariadicInput<N>() -> GetInput<N>()
  • SetVariadicNamedInput<Tag>() -> SetInput<Tag>()
  • GetVariadicNamedInput<Tag>() -> GetInput<Tag>()
  • SetVariadicInputs() -> SetInputs()
  • GetVariadicInputs() -> GetInputs()

I also added a default value for template parameter N so that you can now call SetInput() and GetInput() to interact with first input.

Rationale

Easier to read, understand and more backward-compatible.

Additional notes

All ongoing refactoring with FunctorImageFilter will need to update their patches.

Copyright

The copyright owner is CNES and has signed the ORFEO ToolBox Contributor License Agreement.


Check before merging:

  • All discussions are resolved
  • At least 2 :thumbsup: votes from core developers, no :thumbsdown: vote.
  • The feature branch is (reasonably) up-to-date with the base branch
  • Dashboard is green
  • Copyright owner has signed the ORFEO ToolBox Contributor License Agreement

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading