diff --git a/cursors/kpCursorLightCross.cpp b/cursors/kpCursorLightCross.cpp --- a/cursors/kpCursorLightCross.cpp +++ b/cursors/kpCursorLightCross.cpp @@ -67,7 +67,6 @@ break; case Transparent: - default: colorValue = ColorWhite; maskValue = MaskTransparent; break; diff --git a/kpViewScrollableContainer.cpp b/kpViewScrollableContainer.cpp --- a/kpViewScrollableContainer.cpp +++ b/kpViewScrollableContainer.cpp @@ -120,15 +120,12 @@ { case kpGrip::Bottom: return Qt::SizeVerCursor; - break; // one day you'll forget case kpGrip::Right: return Qt::SizeHorCursor; - break; // one day you'll forget case kpGrip::BottomRight: return Qt::SizeFDiagCursor; - break; // one day you'll forget } return Qt::ArrowCursor; diff --git a/widgets/toolbars/options/kpToolWidgetFillStyle.cpp b/widgets/toolbars/options/kpToolWidgetFillStyle.cpp --- a/widgets/toolbars/options/kpToolWidgetFillStyle.cpp +++ b/widgets/toolbars/options/kpToolWidgetFillStyle.cpp @@ -114,23 +114,19 @@ // private QString kpToolWidgetFillStyle::fillStyleName (FillStyle fs) const { - // do not complain about the "useless" breaks - // as the return statements might not be return statements one day - switch (fs) { case NoFill: return i18n ("No Fill"); - break; + case FillWithBackground: return i18n ("Fill with Background Color"); - break; + case FillWithForeground: return i18n ("Fill with Foreground Color"); - break; + default: return QString(); - break; } }