konqueror and no view ? #212586

[jcl.weingarten at free.fr]

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 at googlemail.com>
À: kfm-devel at 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