Skip to content

BUG: Default parameters allowed incorrect usage

Hans Johnson requested to merge github/fork/BRAINSia/FixDefaultParametersIncorrectUsage into master

Slicer/Modules/Loadable/Models/qSlicerModelsModule.cxx:165:44: warning: 'this' pointer cannot be null in well-defined C++ code; pointer may be assumed to always convert to true [-Wundefined-bool-conversion] QStringList() << "vtkMRMLModelNode", this)); ^~~~

The 'this' pointer was interpreted as boolean. To ensure complete proper usage, the default parameters were removed, thus forcing compiler errors that were easy to identify and fix.

Merge request reports