diff options
| author | Chris <chris.sherlock79@gmail.com> | 2013-03-21 23:55:36 +1100 |
|---|---|---|
| committer | Fridrich Strba <fridrich@documentfoundation.org> | 2013-03-21 18:12:37 +0000 |
| commit | 3c61c08757208fb9735814dd62ac80f23cf51b86 (patch) | |
| tree | 8a24cb3c9d8f573f3c4ce7b1391390bd4764d0d2 | |
| parent | f9af9abe628dc8886e320fa27421a125886f37a6 (diff) | |
Move setupwizard modify handler code.
Move setupwizard code into the conditional below it. You can only work
on pPage if it isn't a null pointer anyway, to my mind this looks a
lot cleaner.
Change-Id: If8d910c15c5a1c1da5eb204b5922d074c454978b
Reviewed-on: https://gerrit.libreoffice.org/2890
Reviewed-by: Fridrich Strba <fridrich@documentfoundation.org>
Tested-by: Fridrich Strba <fridrich@documentfoundation.org>
| -rw-r--r-- | dbaccess/source/ui/dlg/dbwizsetup.cxx | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/dbaccess/source/ui/dlg/dbwizsetup.cxx b/dbaccess/source/ui/dlg/dbwizsetup.cxx index 338e7c0d9ff4..cb53dfd78904 100644 --- a/dbaccess/source/ui/dlg/dbwizsetup.cxx +++ b/dbaccess/source/ui/dlg/dbwizsetup.cxx @@ -587,13 +587,13 @@ TabPage* ODbTypeWizDialogSetup::createPage(WizardState _nState) break; } - if (pPage && (_nState != PAGE_DBSETUPWIZARD_INTRO) && (_nState != PAGE_DBSETUPWIZARD_AUTHENTIFICATION)) - { - pPage->SetModifiedHandler(LINK( this, ODbTypeWizDialogSetup, ImplModifiedHdl ) ); - } - if ( pPage ) { + if ((_nState != PAGE_DBSETUPWIZARD_INTRO) && (_nState != PAGE_DBSETUPWIZARD_AUTHENTIFICATION)) + { + pPage->SetModifiedHandler(LINK( this, ODbTypeWizDialogSetup, ImplModifiedHdl ) ); + } + pPage->SetServiceFactory( m_pImpl->getORB() ); pPage->SetAdminDialog(this, this); |