QDialog - Change exec() and open() to virtual functions
authorThorbjørn Lund Martsum <tmartsum@gmail.com>
Wed, 7 Mar 2012 15:02:47 +0000 (16:02 +0100)
committerQt by Nokia <qt-info@nokia.com>
Thu, 8 Mar 2012 20:19:26 +0000 (21:19 +0100)
commitdbab994b2cdc5469cf53e3f6c5d09bc2d7b39ce9
treec4e68abf7a539b42e3e66d35c2b7c163f46dc1bb
parentc614bf25fef7e5a0e0fc7d74980f48a471b4406f
QDialog - Change exec() and open() to virtual functions

QDialog is meant for inheritance (it contains other virtual
functions) ...

The main reason for this is that we inside the dialog
could have a (datadepened) precondition that should
prevent the dialog from being shown at all - instead we
might just want to show a messagebox.

That is not easy solvable in Qt right now. It is possible
to reimplement setVisible - but calling reject from
setVisible does not work. There seems only to be clumsy
solutions to that problem - unless these functions are
made virtual

Beside it also creates a nice symmetry to done.

Change-Id: I51c29e1f7a4a5522f5c0f71bcf98c943580790b9
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@nokia.com>
Reviewed-by: Andreas Aardal Hanssen <andreas@hanssen.name>
Reviewed-by: Robin Burchell <robin+qt@viroteck.net>
src/widgets/dialogs/qdialog.h