[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-frameworks-devel
Subject:    Re: visibility compile flag
From:       Àlex Fiestas <afiestas () kde ! org>
Date:       2014-04-29 22:41:01
Message-ID: 4084242.7qzevbGKUN () minibad
[Download RAW message or body]

On Tuesday 29 April 2014 09:21:44 Kevin Ottens wrote:
> Hello,
> 
> On Monday 28 April 2014 19:50:51 Àlex Fiestas wrote:
> > I'm porting libkscreen to Frameworks and I found that we are passing (when
> > available) -fvisibility=hidden.
> > 
> > After reading this[1] really quick I thought I would send an email here so
> > people wiser than me can decide if it makes sense to enable it to all
> > frameworks by default, it sounds useful.
> 
> I'm confused... isn't it already the case?
> 
> At least KDECompilerSettings.cmake has the following line:
> set(CMAKE_CXX_VISIBILITY_PRESET hidden)
> 
> It should enable -fvisibility=hidden for all frameworks as they include that
> file.
> 
> Or you meant something else?

O I did check but apparently not enough :/

Nothing then! (more code that I can remove from kscreen cmake :p)

Cheers.
_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
[prev in list] [next in list] [prev in thread] [next in thread]