In T22322#96215, @maleike wrote: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<>.
Description
Description
| Status | Assigned | Task | ||
|---|---|---|---|---|
| Resolved | kislinsk | T22322 Changing a renderer-specific property list of a data node does not modify the data node | ||
| Resolved | kislinsk | T25357 Refactor DataNode::GetPropertylist(..) for more const correctness |
Event Timeline
Comment Actions
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
Comment Actions
This task was closed here on Phabricator since it was migrated to GitLab. Please continue on GitLab.