Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Default default-constructor of ConstantBoundaryCondition and other Core/Common class templates #3929

Merged
merged 17 commits into from
Feb 23, 2023
Merged

Default default-constructor of ConstantBoundaryCondition and other Core/Common class templates #3929

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
13aadd7
STYLE: Default default-constructor of ConstantBoundaryCondition
N-Dekker Feb 16, 2023
0c4de3f
STYLE: Default default-constructor of CompensatedSummation
N-Dekker Feb 16, 2023
4812c9f
COMP: Fix warning: unused variable 'compensatedSum' [-Wunused-variable]
N-Dekker Feb 17, 2023
222f049
STYLE: Default default-constructor of SpatialFunction classes
N-Dekker Feb 16, 2023
7dad3a3
STYLE: Default default-constructor of Image classes
N-Dekker Feb 16, 2023
d8ed914
STYLE: Default default-constructor of ImageRandom ConstIterator classes
N-Dekker Feb 16, 2023
38f4584
STYLE: Default default-constructor of ConstNeighborhoodIterator
N-Dekker Feb 16, 2023
b93358c
STYLE: Default default-constructor ImageVectorOptimizerParametersHelper
N-Dekker Feb 16, 2023
7e6d14e
STYLE: Default default-constructor of ImportImageContainer and Filter
N-Dekker Feb 16, 2023
f752fc4
STYLE: Default default-constructor of MinimumMaximumImageCalculator
N-Dekker Feb 16, 2023
5a200f9
STYLE: Default default-constructor of ObjectStore and MemoryBlock
N-Dekker Feb 16, 2023
5918bdf
STYLE: Default default-constructor of PointSet
N-Dekker Feb 16, 2023
76937ea
STYLE: Default default-constructor of PriorityQueueContainer
N-Dekker Feb 16, 2023
3de9994
STYLE: Default default-constructor of RegularizedHeavisideStepFunction
N-Dekker Feb 16, 2023
8e7a0de
STYLE: Default default-constructor of ThreadedImageRegionPartitioner
N-Dekker Feb 16, 2023
dca046a
STYLE: Default default-constructor of VariableLengthVector
N-Dekker Feb 16, 2023
d5fa8ae
STYLE: Default default-constructor of Versor
N-Dekker Feb 16, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions Modules/Core/Common/include/itkConstNeighborhoodIterator.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ class ITK_TEMPLATE_EXPORT ConstNeighborhoodIterator
using ImageBoundaryConditionConstPointerType = const ImageBoundaryCondition<ImageType, OutputImageType> *;

/** Default constructor */
ConstNeighborhoodIterator();
ConstNeighborhoodIterator() = default;

/** Virtual destructor */
~ConstNeighborhoodIterator() override = default;
Expand Down Expand Up @@ -612,11 +612,14 @@ class ITK_TEMPLATE_EXPORT ConstNeighborhoodIterator
* within the buffer. */
OffsetType m_WrapOffset{ { 0 } };

/** Default boundary condition. */
TBoundaryCondition m_InternalBoundaryCondition{};

/** Pointer to the actual boundary condition that will be used.
* By default this points to m_BoundaryCondition, but
* OverrideBoundaryCondition allows a user to point this variable an external
* boundary condition. */
ImageBoundaryConditionPointerType m_BoundaryCondition{};
ImageBoundaryConditionPointerType m_BoundaryCondition{ &m_InternalBoundaryCondition };
dzenanz marked this conversation as resolved.
Show resolved Hide resolved

/** Denotes which of the iterators dimensional sides spill outside
* region of interest boundaries. By default `false` for each dimension. */
Expand All @@ -636,9 +639,6 @@ class ITK_TEMPLATE_EXPORT ConstNeighborhoodIterator
/** Upper threshold of in-bounds loop counter values. */
IndexType m_InnerBoundsHigh{};

/** Default boundary condition. */
TBoundaryCondition m_InternalBoundaryCondition{};

/** Does the specified region need to worry about boundary conditions? */
bool m_NeedToUseBoundaryCondition{ false };

Expand Down
6 changes: 0 additions & 6 deletions Modules/Core/Common/include/itkConstNeighborhoodIterator.hxx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -206,12 +206,6 @@ ConstNeighborhoodIterator<TImage, TBoundaryCondition>::GetBoundingBoxAsImageRegi
return ans;
}

template <typename TImage, typename TBoundaryCondition>
ConstNeighborhoodIterator<TImage, TBoundaryCondition>::ConstNeighborhoodIterator()
{
m_BoundaryCondition = &m_InternalBoundaryCondition;
}

template <typename TImage, typename TBoundaryCondition>
ConstNeighborhoodIterator<TImage, TBoundaryCondition>::ConstNeighborhoodIterator(const Self & orig)
: Neighborhood<InternalPixelType *, Dimension>(orig)
Expand Down