[Freeipa-devel] Hide private symbols in the bind-dyndb-ldap
Simo Sorce
simo at redhat.com
Wed Oct 17 15:16:58 UTC 2012
On Wed, 2012-10-17 at 17:06 +0200, Adam Tkac wrote:
> On Wed, Oct 17, 2012 at 09:58:36AM -0400, Simo Sorce wrote:
> > On Wed, 2012-10-17 at 13:04 +0200, Adam Tkac wrote:
> > > Hello,
> > >
> > > attached patch hides all symbols except dynamic_driver_{init,destroy}. Feedback
> > > is appreciated.
> >
> > Any reason not to use a simple export file ?
>
> This is also possible solution. However if I understand GNU build chain
> correctly, using export file only affects linker and doesn't allow compiler to
> perform more aggressive optimisations (i.e. inline hidden functions etc), does
> it?
It's linker only indeed, good point, I'll keep that in mind.
> > Anyway strong ACK, keeping private symbols private is good hygiene.
>
> Thanks, pushed to master.
>
> Regards, Adam
>
--
Simo Sorce * Red Hat, Inc * New York
More information about the Freeipa-devel
mailing list