HomePhabricator
Diffusion MITK c7f54dfd1df1

BREAKING CHANGES!!!

Description

BREAKING CHANGES!!!
+ Changed signature of SliceInformation supporting WriteBackSegmentationResult to

WriteBackSegmentationResults (needed to avoid now compiler errors due to implicit
parameter conversion).

+ Aligned behavior of WriteBackSegmentationResults to the behavior of

other WriteBackSegmentationResult overloads. Until now WriteBackSegmentationResults
had a complete different implementation that. No it is streamlined and
WriteBackSegmentationResult overloads all call WriteBackSegmentationResults
and all behave like the former WriteBackSegmentationResult(const PlaneGeometry* ...)
version.

+ Made SliceInformation element const to avoid unintended manipulation.

Additionally:
+ improved const correctness
+ refactored redundant code into own function (RemoveContourFromInterpolator)
+ added UpdateSurfaceInterpolation overload to support the processing of

a SliceInformation vector at once.

+ Reworked LiveWireTool2D to use the new reworked WriteBackSegmentationResults

correctly

+ Fixed internal "bug"/inconsistency in working slice generation in LiveWireTool2D.

Problem was not obvious/relevant so far, because the bug got covered by an
other effect (maybe that was the reason of the very "special" own implementation
of the slice writing...). Would become a problem as soon as LiveWireTool2D
should e.g. correctly support Undo/Do.