Skip to content

STYLE: Indentation related changes

Csaba Pinter requested to merge github/fork/cpinter/indentation-issues into master

  1. vtkMRMLNode::Indent member seemed unused in the core, except from the also unused ROI list node, so the member was removed

  2. In WriteXML functions, the increasing indentation was added between the XML attributes, which did not have major effect on the top level, but for example in the scene view section, where indentation was increased, it made the nodes look like this:

  <Selection
    id="vtkMRMLSelectionNodeSingleton"    name="Selection"    hideFromEditors="true"    selectable="true"    selected="false"    singletonTag="Singleton"

As the purpose of indentation is to add space in front of the lines, adding that space between the XML attributes seems unnecessary. With these changes, the same entry now looks like this:

  <Selection
    id="vtkMRMLSelectionNodeSingleton" name="Selection" hideFromEditors="true" selectable="true" selected="false"

Merge request reports