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

List:       kfm-devel
Subject:    Re: konqueror and no view ? #212586
From:       Frank Reininghaus <frank78ac () googlemail ! com>
Date:       2011-05-29 10:33:57
Message-ID: BANLkTikzLxKrf3+XCtouFg4fzW2rTmE0rg () mail ! gmail ! com
[Download RAW message or body]

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]