Moved to KDE Review: KReport

[Adam Pigg]

Yes, I'll take a look, is there a list of problem classes?

On Mon, 10 Oct 2016, 23:18 Jaroslaw Staniek, <staniek at kde.org> wrote:

>
>
> On 11 October 2016 at 00:09, Albert Astals Cid <aacid at kde.org> wrote:
>
> El dissabte, 8 d’octubre de 2016, a les 7:49:21 CEST, Ben Cooksley va
> escriure:
> > Hi all,
> >
> > Just a heads up that at the request of Jaroslaw Staniek, Sysadmin has
> > moved the following repositories to KDE Review:
> >
> > - KReport (kreport)
>
> kreport seems to have some public/exported classes without d-pointer that
> may
> make keeping BC harder, is that on purpose?
>
>
> ​Good find!​
>
> ​@Adam Seems like an easy fix?​
> ​
>
>
> the use of ki18n_install seems wrong as said in the other email.
>
> The clazy report at https://paste.kde.org/ppbkincse
>
>
> ​​@Friedrich Would you be able to help with tr()?​
>
> Thanks
>
> --
> regards, Jaroslaw Staniek
>
> KDE:
> : A world-wide network of software engineers, artists, writers, translators
> : and facilitators committed to Free Software development - http://kde.org
> Calligra Suite:
> : A graphic art and office suite - http://calligra.org
> Kexi:
> : A visual database apps builder - http://calligra.org/kexi
> Qt Certified Specialist:
> : http://www.linkedin.com/in/jstaniek
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20161011/087ad689/attachment.htm>