Re: kemoticons is now in staging

2012-11-20 Thread Sune Vuorela
On 2012-11-19, David Faure faure+bluesyst...@kde.org wrote:
 Let me say this differently: look at the huge mass of code in Qt: almost none 
 of it uses QSettings to let the users configure things from underneath.
 Most of the time, the apps themselves have control over the various settings 
 offered by the Qt API.

 The exception is the stuff available in `qtconfig`, but that's quite limited.

 So maybe this setting should be left to the application, if that makes 
 sense.

I was actually going to do a mail suggesting the same.

/Sune

___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: kemoticons is now in staging

2012-11-20 Thread Kevin Ottens
On Tuesday 20 November 2012 23:03:50 Valentin Rusu wrote:
* Is the dependency on KService really needed? Or QStandardPaths would
be enough?

 kemoticons uses backend plugins. So it's not simply about QStandardPaths
 here.

Sure, sorry if I was unclear. KService vs QStandardPaths is about locating the
plugins part, for loading them you need some extra calls and that's
KPluginLoader vs QPluginLoader. We should take the decision on how we locate
the plugins though, hence why I pointed out QStandardPaths.

In other words: Does kemoticons really need the querying system of KService?
Or looking into a set of known folders to find the plugins would be enough?

 However, I think that the way I had to add the dependencies on kservice
 is somewhat ugly, as it requires 4 lignes in order to get kservice
 headers compiled when included from this framework. Is there any new KF5
 macro to add kservice inclusions in a single line?

You did it in the proper way for now. It'll get cleaner when kservice moves to
its final place.

* The only reason it depends on KConfig is to take a look in kdeglobals,
we might want to do that differently (QSettings could be a contender?)

 I'll check that.

Make sure to check the emails from David and Sune as well on that topic.
They're right that QSettings is likely not needed, and that it should/could be
something controlled by a symlink like the icon themes, or a similar mean.

Regards.
--
Kévin Ottens, http://ervin.ipsquad.net

Sponsored by BlueSystems and KDAB to work on KDE Frameworks


signature.asc
Description: This is a digitally signed message part.
___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel