Re: From kdelibs4 to KDE frameworks... how to make KDE more cross platform...

2012-02-13 Thread Pau Garcia i Quiles
2012/2/13 Ingo Klöcker > Are there more important reasons than "it scares Windows users"? Windows > does already run so many services on its own. Why do a few more > processes matter? Why do the users care anyway? IMNSHO, they should stop > looking at their task manager or process explorer and s

Re: From kdelibs4 to KDE frameworks... how to make KDE more cross platform...

2012-02-13 Thread Shaun Reich
hate to chime in as well, but i think replacing the Windows shell should definitely be something that's looked at. imho it makes a lot of sense. face it, the Windows shell sucks. why are we replacing their apps and adding our own (dolphin kicking explorer's butt)? because the default ones are ter

Re: From kdelibs4 to KDE frameworks... how to make KDE more cross platform...

2012-02-13 Thread Christoph Feck
On Monday 13 February 2012 22:57:56 Aaron J. Seigo wrote: > On Saturday, February 11, 2012 20:00:01 Alexander Neundorf wrote: > > It is cool to be able to replace the Windows shell... but does it > > make sense > > fwiw, it's always been my opinion that it does not make sense to do > this. > > it

Re: From kdelibs4 to KDE frameworks... how to make KDE more cross platform...

2012-02-13 Thread Ingo Klöcker
On Saturday 11 February 2012, Alexander Neundorf wrote: > Hi, > > KDE is based on Qt, and Qt is very cross platform. > While there are Windows and also OSX builds of KDE4, they are not > really successful. > I mean, it's not like everybody is running amarok today under > Windows, or kate, or kdeve

Re: From kdelibs4 to KDE frameworks... how to make KDE more cross platform...

2012-02-13 Thread Aaron J. Seigo
On Saturday, February 11, 2012 20:00:01 Alexander Neundorf wrote: > It is cool to be able to replace the Windows shell... but does it make sense fwiw, it's always been my opinion that it does not make sense to do this. it doesn't solve a real problem on the windows platform, and as such does not

Re: Review Request: New KPart extension for manupilating a browser engine's settings

2012-02-13 Thread Dawit Alemayehu
> On Feb. 13, 2012, 7:44 p.m., Laszlo Papp wrote: > > Is it acceptable to send new features/extensions against KDE4.x ? I had the > > impression only bugfixes, and I tried to push my changes against the > > frameworks branch instead. I may have been wrong with that though. This is not a new fe

Re: Review Request: New KPart extension for manupilating a browser engine's settings

2012-02-13 Thread Laszlo Papp
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103973/#review10609 --- Is it acceptable to send new features/extensions against KDE4.x

Review Request: New KPart extension for manupilating a browser engine's settings

2012-02-13 Thread Dawit Alemayehu
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103973/ --- Review request for kdelibs and David Faure. Description --- This patc

Re: Review Request: Allow protocol-specialized ThumbCreators to be used for folder previews

2012-02-13 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103882/#review10607 --- This review has been submitted with commit 3487ca03f0e6a88a604

Re: Review Request: Report file errors when extracting files using karchive

2012-02-13 Thread Mario Bensi
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103808/#review10596 --- If you can, i prefer you use an enum Error (you need to create

Re: Review Request: Allow protocol-specialized ThumbCreators to be used for folder previews

2012-02-13 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103882/#review10598 --- This review has been submitted with commit 92e8ec0bf91ceead579

Re: From kdelibs4 to KDE frameworks... how to make KDE more cross platform...

2012-02-13 Thread Pau Garcia i Quiles
On Sat, Feb 11, 2012 at 11:39 PM, Andre Heinecke wrote: > The perception that dbus is a problem on Windows is outdated. Dbus was a > huge > problem on windows but currently the situation is pretty good, we reall > have > no known issues with it. We can run multiple instances with different > appli

Re: From kdelibs4 to KDE frameworks... how to make KDE more cross platform...

2012-02-13 Thread Pau Garcia i Quiles
On Sat, Feb 11, 2012 at 8:00 PM, Alexander Neundorf wrote: > > Pau had the idea to write a fake libdbus for Windows, which internally > doesn't > talk to a dbus daemon, but which uses the Windows messaging service. > On Linux DBUS is no problem. On Mac ? I don't know. Probably better than > under

Re: From kdelibs4 to KDE frameworks... how to make KDE more cross platform...

2012-02-13 Thread David Faure
On Saturday 11 February 2012 20:00:01 Alexander Neundorf wrote: > kded... for what things is this needed when running only a single > application ? The main reason currently is "so that it watches desktop files and keeps ksycoca up-to-date". The other uses of KDED are on-demand (kssl, timezones

Re: Review Request: New KDE Macro for to wrap the noreturn attribute

2012-02-13 Thread Andreas Pakulat
On 13.02.12 12:49:08, Pau Garcia i Quiles wrote: > Hi, > > The MSDN documentation explains this: > > http://msdn.microsoft.com/en-us/library/k6ktzx3s%28v=vs.100%29.aspx Hmm, maybe that changed in more recent MSVC's but MSVC6 always error'ed out from the compile if a branch was missing a return s

Re: Review Request: Make KFileWidget provide default filename even when the protocol doesn't support listing

2012-02-13 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103642/#review10594 --- Ship it! Ship It! - David Faure On Jan. 6, 2012, 9:54 p.m.,

Re: Review Request: Fix the KApplication '--config' option

2012-02-13 Thread Cristian Oneț
> On Feb. 13, 2012, 11:41 a.m., David Faure wrote: > > Ship It! I would prefer that a developer that currently works on kdelibs made the commit since I'm not familiar with the kdelibs development process so I would not know in which branch to push this. Thanks. - Cristian

Re: Review Request: New KDE Macro for to wrap the noreturn attribute

2012-02-13 Thread Pau Garcia i Quiles
Hi, The MSDN documentation explains this: http://msdn.microsoft.com/en-us/library/k6ktzx3s%28v=vs.100%29.aspx On Mon, Feb 13, 2012 at 12:30 PM, David Faure wrote: >This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/103832/ > > Out of curiosity,

Re: Review Request: New KDE Macro for to wrap the noreturn attribute

2012-02-13 Thread Tomaz Canabrava
On Mon, Feb 13, 2012 at 3:30 AM, David Faure wrote: >This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/103832/ > > Out of curiosity, if the method returns void anyway, why is this attribute > necessary? It's not like the the compiler is going to w

Re: Review Request: make Speller::spellCheckerFound() return true when the dict for the current language is found (important when there was no dict for default language)

2012-02-13 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103729/#review10592 --- Ship it! Looks good. But unless isValid() is a long operation,

Re: Review Request: Fix the KApplication '--config' option

2012-02-13 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103693/#review10591 --- Ship it! Ship It! - David Faure On Jan. 14, 2012, 9:27 a.m.

Re: Review Request: New KDE Macro for to wrap the noreturn attribute

2012-02-13 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103832/#review10589 --- Out of curiosity, if the method returns void anyway, why is thi

Re: Review Request: Add recentdocuments:/ kio slave to kde-runtime.

2012-02-13 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103849/#review10588 --- kioslave/recentdocuments/recentdocuments.cpp

Re: Review Request: Add recentdocuments:/ kio slave to kde-runtime.

2012-02-13 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103849/#review10587 --- kioslave/recentdocuments/recentdocuments.cpp

Re: Review Request: Allow protocol-specialized ThumbCreators to be used for folder previews

2012-02-13 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103882/#review10585 --- Ship it! OK I understand. This should actually replace m_remot