On Sunday 29 December 2013 16:18:23 Christoph Cullmann wrote: > > On Friday 27 December 2013 19:36:57 Aleix Pol wrote: > > > Hi, > > > Here's a patch adding CamelCase headers to KCoreAddons (attached). > > > > > > If we agree that it's the proper way of doing it, I'll proceed to do it > > > on > > > the rest of modules. > > > > Looks good to me. I like the fact that we can't forget to add forwarding > > headers like before - if we do, the lowercase header isn't installed > > either, so we'll notice faster :) > > I mean, a single operation is needed to install a new header, instead of > > two, one of which I always forgot. > > Hmm, does it really install the normal headers, too? > > I use it now in kate.git/ktexteditor/include/CMakelists.txt and need still > to install the normal headers manually.
Well, Aleix's patch removes the installation of the lowercase headers, doesn't it? -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE, in particular KDE Frameworks 5 _______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel