Re: Review Request: Fix KCategorizedView race

2011-12-08 Thread Christoph Feck
On Dec. 6, 2011, 6:39 p.m., Commit Hook wrote: This review has been submitted with commit 08325ba32b72326030004cc28430431193d82fc2 by Thomas Lübking to branch KDE/4.7. This commit causes systemsettings to go into endless recursion, see http://privatepaste.com/0e43ad214c zemo on

Re: Review Request: Fix KCategorizedView race

2011-12-08 Thread Jaime
The new endless loop (causing a stack overflow here when I resize systemsettings view) this time is in: QListView::resizeEvent(QResizeEvent *e) that calls QAbstractItemView::resizeEvent(e); QAbstractItemView::resizeEvent(QResizeEvent *event) that calls KCategorizedView::updateGeometries();

Re: Requesting freeze exception for JtG

2011-12-08 Thread David Faure
On Wednesday 23 November 2011 00:28:07 Pau Garcia i Quiles wrote: 1) Changes to ui_standards.rc This is a high-risk change. It affects every application that uses XMLGUI and KParts plugins. I did quite a lot of work in this area in KDE 3.x and I know that changes to this file could have

Re: Review Request: Fix KCategorizedView race

2011-12-08 Thread Thomas Lübking
On Dec. 6, 2011, 6:39 p.m., Commit Hook wrote: This review has been submitted with commit 08325ba32b72326030004cc28430431193d82fc2 by Thomas Lübking to branch KDE/4.7. Christoph Feck wrote: This commit causes systemsettings to go into endless recursion, see

Re: Review Request: Fix KCategorizedView race

2011-12-08 Thread Tomaz Canabrava
On Thu, Dec 8, 2011 at 2:33 AM, Thomas Lübking thomas.luebk...@web.dewrote: This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103335/ On December 6th, 2011, 6:39 p.m., *Commit Hook* wrote: This review has been submitted with commit

Re: Review Request: Fix KCategorizedView race

2011-12-08 Thread Thomas Lübking
Am Thu, 8 Dec 2011 07:59:43 -0800 schrieb Tomaz Canabrava tcanabr...@kde.org: What about jaime patch? yeah, static policies work - we know that ;-) This one is quite different (recursion, not race - though maybe both), from the first rough look there seem two recursions, one direct (easy to

Re: kactivities update

2011-12-08 Thread Kevin Kofler
Aaron J. Seigo wrote: yes, i skipped the details. done properly it should be linking libkactivities. this is an acceptable short term work-around given it's in a branch, while doing it right requires the modularization to be complete for at least libkactivities dependencies. Or it'd require

Re: kactivities update

2011-12-08 Thread Kevin Kofler
Aaron J. Seigo wrote: yes, i skipped the details. done properly it should be linking libkactivities. this is an acceptable short term work-around given it's in a branch, while doing it right requires the modularization to be complete for at least libkactivities dependencies. Or it'd require

Re: Review Request: Fix KCategorizedView race

2011-12-08 Thread Thomas Lübking
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103335/#review8802 --- - Thomas Lübking On Dec. 8, 2011, 6:04 p.m., Thomas Lübking

Re: Review Request: Fix KCategorizedView race

2011-12-08 Thread Thomas Lübking
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103335/#review8803 --- kdeui/itemviews/kcategorizedview.cpp

Re: Right location for test config files?

2011-12-08 Thread David Faure
On Monday 21 November 2011 20:27:28 Andriy Rysin wrote: In regards to defect below, what's the right location for some xml files I use in keyboard daemon tests? Currently the files I have is to test the code for parsing /usr/share/X11/xkb/rules/evdev.xml but some others could be added later...

Re: Review Request: Allow the user to choose sftp or fish on knetattach

2011-12-08 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103199/#review8804 --- Ship it! Ship It! - David Faure On Nov. 21, 2011, 8:11

Re: Review Request: make KFileWidget keep current extension if possible

2011-12-08 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102986/#review8808 --- Thx, looks better. Just one comment for code readability, feel

Re: Review Request: GSoC: Errors handling during file transfer.

2011-12-08 Thread David Faure
On Nov. 17, 2011, 2:33 p.m., David Faure wrote: David Faure wrote: Cyril, can you rebase your changes on top of the kdelibs frameworks branch, compile, and commit there? Cyril Oblikov wrote: Hi David. I'm currently working on this dialog together with Björn

Re: kactivities update

2011-12-08 Thread Thomas Zander
On Thursday 08 December 2011 17.12.05 Kevin Kofler wrote: Aaron J. Seigo wrote: yes, i skipped the details. done properly it should be linking libkactivities. this is an acceptable short term work-around given it's in a branch, while doing it right requires the modularization to be

Re: Review Request: Fix KCategorizedView race

2011-12-08 Thread Thomas Lübking
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103335/ --- (Updated Dec. 8, 2011, 6:04 p.m.) Review request for kdelibs, Rafael

Re: Requesting freeze exception for JtG

2011-12-08 Thread David Faure
On Thursday 08 December 2011 16:56:16 David Faure wrote: khelpmenu seems to have been ported from qt3 without much thinking, the actions in menu() could just use kstandardaction too, instead of duplicating the action details... but that's for frameworks 5 :) Hmm, actually that's just

Review Request: Make the action names of KRecentFilesAction not be extremely long

2011-12-08 Thread Albert Astals Cid
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103360/ --- Review request for kdelibs and David Faure. Description --- Cut the