Skip to content

Add break statement to for loop after moving ColorPalette proxy

John Tourtellott requested to merge john.tourtellott/paraview:settings-logic into master

Adds a break statement to a loop in the pqSettingsDialog constructor that moves Color Palette to the end of the settings list. Once the proxy is found and moved, there is no need to continue looping. This change will avoid traversing the Color Palette twice, but more importantly, it also resolves an invalid pointer exception that gets thrown by CMB/ModelBuilder Windows executables.

Merge request reports