Moved to KDE Review: KReport

[Albert Astals Cid]

El dimarts, 11 d’octubre de 2016, a les 11:28:36 CEST, Adam Pigg va escriure:
> Yes, I'll take a look, is there a list of problem classes?

Not from my side, i just opened a few of the installed headers and found some, thus i wrote that 
email, but didn't have an exahustive list.

Sorry.

Cheers,
  Albert

> 
> 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/1b0f067c/attachment.htm>