Page MenuHomePhabricator

Refactor DataNode::GetPropertylist(..) for more const correctness
Open, LowPublic

Description

Interesting discussion. I agree that either all propertylists or none should mark the DataNode modified.

There is another tiny detail that could be cleaned up while on the subject: DataNode::GetPropertylist(..) const returns non-const property lists. This allows removing properties from const DataNodes without having to use a suspicious const_cast<>.

Event Timeline

floca triaged this task as Normal priority.Aug 31 2018, 5:19 PM
floca created this task.
nolden added a subscriber: nolden.

Discussed this in the MITK meeting, we agreed this would be an improvement

nolden lowered the priority of this task from Normal to Low.Oct 26 2018, 11:09 AM
kislinsk added a project: Auto-closed.

Hi there! 🙂

This task was auto-closed according to our Task Lifecycle Management.
Please follow this link for more information and don't forget that you are encouraged to reasonable re-open tasks to revive them. 🚑

Best wishes,
The MITK devs

floca removed a project: Auto-closed.