[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: KDE/kdegames/ksirtet/ksirtet
From:       "Nicolas Roffet" <nicolas-kde () roffet ! com>
Date:       2007-06-30 22:24:24
Message-ID: 33468.86.158.67.150.1183242264.squirrel () roffet ! is-a-geek ! com
[Download RAW message or body]

Hi David,


Am Samstag, 30. Juni 2007 00:34:27 schrieben Sie:
> On Saturday 30 June 2007, Nicolas Roffet wrote:
> > +       if ( KApplication::kApplication()->isSessionRestored() )
> > +               kRestoreMainWindows<KSMainWindow>();
> > +       else {
> > +               KSMainWindow *mw = new KSMainWindow;
> > +               mw->show();
> > +       }
> > +       return KApplication::kApplication()->exec();
>
> Strange way of writing it. Surely you have kapplication instance in this
> block already, so this could be written much simpler+shorter(+safer, in
> case it doesn't have a kapp one day) as:
>
> +       if ( app.isSessionRestored() )
> +               kRestoreMainWindows<KSMainWindow>();
> +       else {
> +               KSMainWindow *mw = new KSMainWindow;
> +               mw->show();
> +       }
> +       return app.exec();

Thanks for your help.
Actually, the KApplication is been instanced in a factory class in the
library (libksirtet) and not in the application (ksirtet) itself. (I don't
know why and if it's a good idea...)
 -> So the only way to access it is "kapp" or "KApplication::kApplication()"
(which is the same and as I found the 1st way of writing more disturbing,
I chose to replace it by the 2nd).
:-/

-- 
Nicolas

[prev in list] [next in list] [prev in thread] [next in thread]