konqueror and no view ? #212586

[Frank Reininghaus]

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