Re: Review Request: replace old kickoff with kickoff-qml

2012-09-19 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106448/#review19150 --- wow, what a large patch :-) I still see a lot of code written

Re: Review Request: Fix setting the size of the calendar of clocks

2012-09-19 Thread Ralf Jung
On Sept. 18, 2012, 10:13 p.m., Sebastian Kügler wrote: Patch looks good now, thanks a lot! Please commit it to KDE/4.9 and master. Sebastian Kügler wrote: Ah, crap. I missed your question about the possible BIC issue. Can someone answer this question? Is it

Re: Review Request: make plasmate able to load a project in the second time

2012-09-19 Thread Giorgos Tsiapaliwkas
On Sept. 18, 2012, 11:32 p.m., Aaron J. Seigo wrote: mainwindow.cpp, line 488 http://git.reviewboard.kde.org/r/106479/diff/1/?file=85743#file85743line488 this looks incorrect; probably what is missing is an || !m_part it works with !m_part - Giorgos

Re: Review Request: make plasmate able to load a project in the second time

2012-09-19 Thread Giorgos Tsiapaliwkas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106479/ --- (Updated Sept. 19, 2012, 11:03 a.m.) Review request for Plasma. Changes

Re: Review Request: hide the konsole on the close project

2012-09-19 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106478/#review19154 --- This review has been submitted with commit

Re: Review Request: The preview action shouldn't be visible all the time

2012-09-19 Thread Giorgos Tsiapaliwkas
On Sept. 17, 2012, 4:13 p.m., Sebastian Kügler wrote: mainwindow.cpp, line 975 http://git.reviewboard.kde.org/r/106477/diff/1/?file=85740#file85740line975 Do we setVisible(true) it when we change package? I don't see this in the code, but haven't tried. Something to

Re: Review Request: The preview action shouldn't be visible all the time

2012-09-19 Thread Giorgos Tsiapaliwkas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106477/ --- (Updated Sept. 19, 2012, 11:13 a.m.) Review request for Plasma. Changes

Review Request: fix crash in runner due to bookmarks

2012-09-19 Thread Giorgos Tsiapaliwkas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106504/ --- Review request for Plasma. Description --- krunner crashes due to

Re: Review Request: fix crash in runner due to bookmarks

2012-09-19 Thread Marco Gulino
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106504/#review19156 --- Why the if(m_favicon)? You could just delete m_favicon and set

Re: Review Request: The preview action shouldn't be visible all the time

2012-09-19 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106477/#review19157 --- Ship it! Ship It! - Sebastian Kügler On Sept. 19, 2012,

Re: Re: Status of QML Porting

2012-09-19 Thread Alex Fiestas
Assigned the Picture Frame for myself, I was going to do the Calculator instead but I found a blogpost by Davide saying that he was going to do it. So, last call... is anyone porting Picture Frame? Cheerz ! ___ Plasma-devel mailing list

Re: Review Request: Make Sonnet::Speller::setLanguage threadsafe, fixing segfaults in krunner

2012-09-19 Thread Zack Rusin
On Sept. 11, 2012, 7:39 p.m., Zack Rusin wrote: That's pretty bonkers. This class was never meant to be used like that. Holding an object in two threads is not going to work unless you make the entire communication synchronous effectively reducing all the multi-threading aspects to a

Re: Review Request: Make Sonnet::Speller::setLanguage threadsafe, fixing segfaults in krunner

2012-09-19 Thread Simeon Bird
On Sept. 11, 2012, 7:39 p.m., Zack Rusin wrote: That's pretty bonkers. This class was never meant to be used like that. Holding an object in two threads is not going to work unless you make the entire communication synchronous effectively reducing all the multi-threading aspects to a

Re: Review Request: Make Sonnet::Speller::setLanguage threadsafe, fixing segfaults in krunner

2012-09-19 Thread Simeon Bird
On Sept. 11, 2012, 7:39 p.m., Zack Rusin wrote: That's pretty bonkers. This class was never meant to be used like that. Holding an object in two threads is not going to work unless you make the entire communication synchronous effectively reducing all the multi-threading aspects to a

Re: Review Request: Make Sonnet::Speller::setLanguage threadsafe, fixing segfaults in krunner

2012-09-19 Thread Thomas Lübking
On Sept. 11, 2012, 7:39 p.m., Zack Rusin wrote: That's pretty bonkers. This class was never meant to be used like that. Holding an object in two threads is not going to work unless you make the entire communication synchronous effectively reducing all the multi-threading aspects to a

Re: Review Request: Make KDM compilation optional

2012-09-19 Thread Ralf Jung
On Sept. 15, 2012, 9:23 a.m., Martin Gräßlin wrote: given that everything else uses the optional macro that sounds like a valid improvement to me Great - if nobody objects, I'll push this tomorrow evening. - Ralf --- This is an

Re: Review Request: Fix setting the size of the calendar of clocks

2012-09-19 Thread Sebastian Kügler
On Sept. 18, 2012, 10:13 p.m., Sebastian Kügler wrote: Patch looks good now, thanks a lot! Please commit it to KDE/4.9 and master. Sebastian Kügler wrote: Ah, crap. I missed your question about the possible BIC issue. Can someone answer this question? Is it

Re: Review Request: Make Sonnet::Speller::setLanguage threadsafe, fixing segfaults in krunner

2012-09-19 Thread Zack Rusin
On Sept. 11, 2012, 7:39 p.m., Zack Rusin wrote: That's pretty bonkers. This class was never meant to be used like that. Holding an object in two threads is not going to work unless you make the entire communication synchronous effectively reducing all the multi-threading aspects to a