Differential D19412 Diff 53371 plugins/tools/selectiontools/kis_tool_select_elliptical.cc

Changeset View

Standalone View

View Options

plugins/tools/selectiontools/kis_tool_select_elliptical.cc

Context not available.
52 Q_UNUSED(roundCornersY); 52​ Q_UNUSED(roundCornersY);
53 53​
54 KisCanvas2 * kisCanvas = dynamic_cast<KisCanvas2*>(canvas()); 54​ KisCanvas2 * kisCanvas = dynamic_cast<KisCanvas2*>(canvas());
55 Q_ASSERT(kisCanvas); 55​ KIS_SAFE_ASSERT_RECOVER_RETURN(kisCanvas);
dkazakovUnsubmitted
Done

recover-version of the assert is expected to have a recover-branch, like

KIS_SAFE_ASSERT_RECOVER(kisCanvas) {
    // do something to recover
}

I guess in this case you wanted to use KIS_SAFE_ASSERT_RECOVER_RETURN(kisCanvas);

dkazakov: recover-version of the assert is expected to have a recover-branch, like ```…
sourabhbossAuthorUnsubmitted
Not Done

Done

sourabhboss: Done
56 56​
57 KisSelectionToolHelper helper(kisCanvas, kundo2_i18n("Select Ellipse")); 57​ KisSelectionToolHelper helper(kisCanvas, kundo2_i18n("Select Ellipse"));
58 58​
Context not available.