KCoreAddons forward headers

David Faure faure at kde.org
Sun Dec 29 19:10:04 UTC 2013


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, faure at kde.org, http://www.davidfaure.fr
Working on KDE, in particular KDE Frameworks 5



More information about the Kde-frameworks-devel mailing list