[ImageJ-devel] Labeling - Revised Branch

[Tobias Pietzsch]

Hi,

I tried to separate reformatting changes from actual changes to the
implementation. I did this by reformatting Lee's original code as
well as the "labeling-revised" changes Christian and Martin. For the
time being I undid the renaming of the template parameter T to L in
"labeling-revised" because to me it did not seem consistent across
all source files.

I pushed the result to the branch "labeling-revised-revised". This has
to commits, one for the reformatting (which is just Eclipse doing it's
thing) and one for the actual changes. I think it is now much clearer
what has been changed.

(I also cc'ed imagej-devel -- maybe this is of interest to others as well.)

best regards,
Tobias

On 03/19/2012 10:05 PM, Christian Dietz wrote:
> Hi Tobi & Lee,
>
> I created a new branch with some changes of Martin Horn and me
> concerning the Labeling (labeling-revised).
> Actually the we changed only a little.
>
> The main differences are:
>
> - Generic Labeling of any IntegerType (Bit, Byte, Short, Int and so on)
>
> - Labeling is not an Img<LabelingType<?>> anymore, but still
> IterableInterval<LabelingType<?>> and RandomAccessible<LabelingType<?>>
>
> - The Labeling is created a little different now (Img<? extends
> IntegerType<?>> is provided in the constructor as the storage).
>
> Please see this as some suggestions from our side and we are pretty sure
> it's not perfect or even good ;) But maybe it's a good starting point
> for some future discussions.
>
> Thank you for your comments in advance,
>
>
> Martin & Christian :)
>