Re: Where to put kwayland integration plugins

2016-07-05 Thread Martin Graesslin
On Wednesday, July 6, 2016 12:16:09 AM CEST Albert Astals Cid wrote: > El dissabte, 2 de juliol de 2016, a les 15:25:39 CEST, Martin Graesslin va > > escriure: > > On Saturday, July 2, 2016 11:41:11 AM CEST David Faure wrote: > > > On lundi 6 juin 2016 11:26:58 CEST Aleix Pol wrote: > > > > On Mon

Re: Review Request 126738: Remove external dependencies from svgs

2016-07-05 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126738/#review96341 --- I made a cleanup off all breeze icons so it should now work w

Re: KMessageBox runtime dependency on FrameworkIntegrationPlugin makes it useless

2016-07-05 Thread Albert Astals Cid
El dissabte, 2 de juliol de 2016, a les 12:23:31 CEST, David Faure va escriure: > On mercredi 15 juin 2016 00:33:40 CEST Albert Astals Cid wrote: > > El dimarts, 14 de juny de 2016, a les 10:57:19 CEST, David Faure va > > escriure: > > > On mardi 14 juin 2016 01:00:09 CEST Albert Astals Cid wrote

Re: Where to put kwayland integration plugins

2016-07-05 Thread Albert Astals Cid
El dissabte, 2 de juliol de 2016, a les 15:25:39 CEST, Martin Graesslin va escriure: > On Saturday, July 2, 2016 11:41:11 AM CEST David Faure wrote: > > On lundi 6 juin 2016 11:26:58 CEST Aleix Pol wrote: > > > On Mon, Jun 6, 2016 at 8:32 AM, Martin Graesslin > > wrote: > > > > Hi framework-deve

Re: Review Request 127813: Reduce string modifications when calling translatePath

2016-07-05 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127813/#review97136 --- Ship it! Looks good to me, thanks. Extending the unittests

Re: Review Request 128369: Handle multiple gzip streams in a single file

2016-07-05 Thread Martin Tobias Holmedahl Sandsmark
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128369/ --- (Updated July 5, 2016, 11:25 a.m.) Status -- This change has been ma

Re: Review Request 128369: Handle multiple gzip streams in a single file

2016-07-05 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128369/#review97114 --- Ship it! Ship It! - David Faure On July 4, 2016, 10:33

Re: Review Request 128369: Handle multiple gzip streams in a single file

2016-07-05 Thread David Faure
> On July 4, 2016, 9:39 p.m., David Faure wrote: > > src/kgzipfilter.cpp, line 331 > > > > > > This is where you could add something like > > if (avail_in < 10) { > > return Ok; // come back