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

List:       kde-commits
Subject:    Re: [kdelibs/frameworks] /: Move KTextBrowser to KDE4Support
From:       Anne-Marie Mahfouf <annemarie.mahfouf () free ! fr>
Date:       2013-05-15 12:54:58
Message-ID: 300455505.140345086.1368622498893.JavaMail.root () spooler3-g27 ! priv ! proxad ! net
[Download RAW message or body]

Hi,

I'll submit a patch and will check the other moves to add this as well if it's missing.

Anne-Marie

----- Mail original -----
> De: "David Faure" <faure@kde.org>
> À: kde-commits@kde.org
> Cc: "Anne-Marie Mahfouf" <annma@kde.org>
> Envoyé: Mercredi 15 Mai 2013 14:12:38
> Objet: Re: [kdelibs/frameworks] /: Move KTextBrowser to KDE4Support
> 
> On Wednesday 15 May 2013 12:31:49 Anne-Marie Mahfouf wrote:
> > @@ -44,7 +44,7 @@
> >   * @author Espen Sand (espensa@online.no)
> >   */
> >  
> > -class KDEUI_EXPORT KTextBrowser : public QTextBrowser
> > +class KDE4SUPPORT_EXPORT KTextBrowser : public QTextBrowser
> 
> This is missing an addition to the class documentation, like
> 
> @deprecated since 5.0, use QTextBrowser instead.
>   setNotifyClick() becomes ... ?
>   Replace <other method from KTextBrowser> with <new solution based
>   on
> QTextBrowser>.
>   etc.
> 
> 
> --
> David Faure, faure@kde.org, http://www.davidfaure.fr
> Working on KDE, in particular KDE Frameworks 5
> 
> 
[prev in list] [next in list] [prev in thread] [next in thread]