Changeset View
Changeset View
Standalone View
Standalone View
lib/documentview/abstractrasterimageviewtool.cpp
Show First 20 Lines • Show All 48 Lines • ▼ Show 20 Line(s) | 48 | { | |||
---|---|---|---|---|---|
49 | delete d; | 49 | delete d; | ||
50 | } | 50 | } | ||
51 | 51 | | |||
52 | RasterImageView* AbstractRasterImageViewTool::imageView() const | 52 | RasterImageView* AbstractRasterImageViewTool::imageView() const | ||
53 | { | 53 | { | ||
54 | return d->mRasterImageView; | 54 | return d->mRasterImageView; | ||
55 | } | 55 | } | ||
56 | 56 | | |||
57 | void AbstractRasterImageViewTool::mouseDoubleClickEvent(QGraphicsSceneMouseEvent* event) | ||||
58 | { | ||||
59 | event->ignore(); | ||||
60 | } | ||||
61 | | ||||
muhlenpfordt: This is not needed since both derived classes override `mouseDoubleClickEvent()`.
But maybe we… | |||||
Yes, this was the intention. We should avoid having to submit something like D14286 again. Let's fix the rest too (whoever gets around to it first). rkflx: Yes, this was the intention. We should avoid having to submit something like D14286 again. | |||||
57 | void AbstractRasterImageViewTool::wheelEvent(QGraphicsSceneWheelEvent* event) | 62 | void AbstractRasterImageViewTool::wheelEvent(QGraphicsSceneWheelEvent* event) | ||
58 | { | 63 | { | ||
59 | event->ignore(); | 64 | event->ignore(); | ||
60 | } | 65 | } | ||
61 | 66 | | |||
62 | } // namespace | 67 | } // namespace |
This is not needed since both derived classes override mouseDoubleClickEvent().
But maybe we should explicitly set a default for all event functions here (of course in another patch). So I think it's good to keep it.