visibility compile flag

[Àlex Fiestas]

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.