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

List:       kfm-devel
Subject:    Re: konqueror and no view ? #212586
From:       jcl.weingarten () free ! fr
Date:       2011-05-30 7:05:59
Message-ID: 868170125.844751306739159921.JavaMail.root () zimbra55-e10 ! priv ! proxad ! net
[Download RAW message or body]

mr martin 
hi 
i'm a little bit all at the sea with the terms i discover and procedures .try to \
survive it s the beginning nice day jcl weingarten

----- Mail Original -----
De: "Frank Reininghaus" <frank78ac@googlemail.com>
À: kfm-devel@kde.org
Envoyé: Dimanche 29 Mai 2011 12h33:57 GMT +01:00 Amsterdam / Berlin / Berne / Rome / \
                Stockholm / Vienne
Objet: Re: konqueror and no view ? #212586

Hi,

2011/5/29 Martin Koller:
> So what about my proposed fix in
> QString KonqExtendedBookmarkOwner::currentUrl() const
> 
> 
> return m_pKonqMainWindow->currentView() ? \
> m_pKonqMainWindow->currentView()->url().url() : QString(); 
> I'd say it should not hurt.

I'd make it

QString KonqExtendedBookmarkOwner::currentUrl() const
{
   const KonqView* view = m_pKonqMainWindow->currentView();
   return view ? view->url().url() : QString();
}

to improve readability.

Cheers,
Frank


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