Re: Review Request 106300: add kimgio WebP image format plugin

2013-05-22 Thread Albert Astals Cid
> On May 18, 2013, 12:44 p.m., Commit Hook wrote: > > This review has been submitted with commit > > df845b06bc440fff5a43a9cdc149e8c88c29bc67 by Martin Koller to branch master. > > Christoph Feck wrote: > Just noticed this has been committed to kolourpaint instead of kdelibs. > Did I miss

KF5 Update Meeting 2013-w21

2013-05-22 Thread Kevin Ottens
Hello all, Yesterday we had the second update meeting on #kde-devel. Like the last time I'll provide a summary of the meeting. Were present for this first meeting: Aaron Seigo (aseigo), Benjamin Port (ben2367), David Faure (dfaure), Kurt Hindenburg (khindenburg), Marco Martin (notmart), Martin

Re: Review Request 110563: Crash fix: hide symbols from static lib QtUitools.a (generically by new macro KDE4_HIDE_SYMBOLS_FROM_STATIC_LIBS)

2013-05-22 Thread Thomas Lübking
> On May 22, 2013, 6:28 p.m., Alexander Neundorf wrote: > > The documentation for the macro should be at the top of > > FindKDE4Internal.cmake, where all the documentation is. > > > > For the technical side: this flag is new to me. Is it the only possible > > solution ? Thiago ? > For the tec

Re: Review Request 110563: Crash fix: hide symbols from static lib QtUitools.a (generically by new macro KDE4_HIDE_SYMBOLS_FROM_STATIC_LIBS)

2013-05-22 Thread Friedrich W. H. Kossebau
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110563/ --- (Updated May 22, 2013, 6:45 p.m.) Review request for Build System, kdelibs

Re: Review Request 110563: Crash fix: hide symbols from static lib QtUitools.a (generically by new macro KDE4_HIDE_SYMBOLS_FROM_STATIC_LIBS)

2013-05-22 Thread Alexander Neundorf
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110563/#review32984 --- The documentation for the macro should be at the top of FindKDE

Re: Review Request 110262: KRandomSequence::randomize: use the Fisher-Yates Algorithm to achieve O(N) complexity

2013-05-22 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110262/ --- (Updated May 22, 2013, 5:19 p.m.) Status -- This change has been mark

Re: Review Request 110262: KRandomSequence::randomize: use the Fisher-Yates Algorithm to achieve O(N) complexity

2013-05-22 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110262/#review32982 --- This review has been submitted with commit 183e59b8e9fe938c209

Re: Review Request 108845: add support for SSSE3 and SSE4.2 in cpufeatures and for msvc

2013-05-22 Thread Patrick Spendrin
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108845/ --- (Updated May 22, 2013, 3:31 p.m.) Review request for KDE Frameworks, kdeli

Re: Review Request 108845: add support for SSSE3 and SSE4.2 in cpufeatures and for msvc

2013-05-22 Thread Patrick von Reth
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108845/#review32971 --- tier1/solid/src/solid/backends/shared/cpufeatures.cpp

Re: Review Request 110043: Proposed fix/workaround for legacy encoded filename handling

2013-05-22 Thread Róbert Szókovács
> On May 7, 2013, 10:11 a.m., Róbert Szókovács wrote: > > The solution is intentionally "shy", I really don't want to fan the flames > > surrounding this issue. I just stumbled upon this location when it can be > > handled painlessly. Whether or not it should be turned on by default, in my > >

Re: Review Request 108845: add support for SSSE3 and SSE4.2 in cpufeatures and for msvc

2013-05-22 Thread Lukáš Tinkl
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108845/#review32960 --- tier1/solid/src/solid/processor.h

Re: Review Request 108845: add support for SSSE3 and SSE4.2 in cpufeatures and for msvc

2013-05-22 Thread Patrick Spendrin
> On May 6, 2013, 11:49 a.m., Kevin Ottens wrote: > > My knowledge of the changes in the MMX/SSE landscape are slightly too > > limited for me to be 100% sure. But it looks like a fine patch. I just at that patch again, and I just realized that I changed the flags for Linux too, so it might ch

Re: Review Request 110563: Crash fix: hide symbols from static lib QtUitools.a (generically by new macro KDE4_HIDE_SYMBOLS_FROM_STATIC_LIBS)

2013-05-22 Thread Friedrich W. H. Kossebau
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110563/ --- (Updated May 22, 2013, 11:19 a.m.) Review request for Build System, kdelib