Page MenuHomePhabricator

Replace all ofstream based XML IO with TinyXML
Closed, WontfixPublic

Description

All ofstream based XML IO should be replaced with TinyXML to unify the XML in/output. The following classes in MITK are affected:

  • mitkPointSetWriter.cpp: Reader already usesTinyXML -> see bug-17765
  • mitkContourModelWriter.cpp: Reader already uses TinyXML
  • mitk::NavigationDataRecorderDeprecated.cpp: no time should be wasted on this class. Remains only for compatibility reasons in MITK
  • mitk::NavigationDataSetWriterXML: partially TinyXML based. Only header and some tags are written without it.
  • mitkEventRecorder.cpp
  • std::string MiniAppManager::CreateXML(): Names are written as std::string with std::ofstream

Event Timeline

This is the input for the filter

mitkInputImage.png (520×611 px, 37 KB)

(In reply to Eric Heim from comment #0)

  • mitkPointSetWriter.cpp: Reader already usesTinyXML -> see bug-17765

In T16115, I proposed a TinyXML based implementation of the PointSetWriterService. The only thing that is unsolved is that "TinyXML to stream" writes all XML nodes into a single line which makes the file pretty unreadable for humans with a simple text editor.

kislinsk claimed this task.
kislinsk added a project: Auto-closed.
kislinsk added a subscriber: kislinsk.

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

kislinsk removed kislinsk as the assignee of this task.May 26 2020, 12:05 PM
kislinsk removed a subscriber: kislinsk.