On 06/06/15 10:04, "[email protected] on behalf of Thiago Macieira" <[email protected] on behalf of [email protected]> wrote:
>On Friday 05 June 2015 10:10:46 Frederik Gladhorn wrote: >> +#ifndef QT_NO_DRAGANDDROP >> + virtual void dragEnterEvent(QDragEnterEvent *); >> + virtual void dragMoveEvent(QDragMoveEvent *); >> + virtual void dragLeaveEvent(QDragLeaveEvent *); >> + virtual void dropEvent(QDropEvent *); >> +#endif >> + > >Is this a public libary? Yes. > > >Are the above overrides or not? If they are overrides, add Q_DECL_OVERRIDE. The methods are overrides. The change looks good except for the missing Q_DECL_OVERRIDE. So far the drag events have been catched in the generic ::event() reimplementation, so nobody could have subclassed these methods in any meaningful way anyway. Cheers, Lars _______________________________________________ Development mailing list [email protected] http://lists.qt-project.org/mailman/listinfo/development
