HomePhabricator
Diffusion MITK a3d2e8fb0144

Description

Fixed T23741

Summary:
For mitk::UIDGenerator I have decided to base it internally on
boost::uuid (name_generator).

Reasoning:
I second @steint point, that it makes no sense to implement a UUID
generation code on its own.

Using boost can be a fast way to get a result, because it is
already included in the superbuild.
I think we should avoid being dependent on boost for too many
things, as the build has sometimes trouble with Boost?

This is not a valid argument in the current master as (1) boost is a
dependency of MITKCore anyways and (2) boost::uuid is nowadays header
only, so no new problems are introduced.
Further, I hided boost::uuid as an implementation detail so, we build
no further dependencies and can change it later one.

The other proposed options crossguid and stduuid would introduce further
additional 3rd party dependencies (like Microsoft GSL). So I see no
benefit in try to avoid boost (wich already is a core dependency) by
introducing even more dependencies.

DICOM UUID:
This we will also need. But I think it is cleaner to manage that on the
DICOM module level (and not the core) then we can also use DCMTKs or
GDCMs implementation.

Signed-off-by: Ralf Floca <r.floca@dkfz-heidelberg.de>

Test Plan: code review and unit test

Reviewers: O1 MITK Reviewer Group I, kislinsk

Reviewed By: O1 MITK Reviewer Group I, kislinsk

Subscribers: kislinsk, steint

Differential Revision: https://phabricator.mitk.org/D316

Details

Provenance
flocaAuthored on Jun 2 2020, 6:47 PM
flocaPushed on Jun 2 2020, 9:25 PM
Reviewer
O1: MITK Reviewer Group I
Differential Revision
Restricted Differential Revision
Parents
rMITK6e66c957ba95: Fixed review remarks
rMITK7cfb5d5d7209: Fixed T27259
Branches
Unknown
Tags
Unknown
Tasks
T23741: Specify default UID generation method

Event Timeline

floca added an edge: Restricted Differential Revision.
floca added a subscriber: steint.