Re: [PATCHES] symbol visibility

On Fri, 2008-08-15 at 23:06 +0200, Michael Biebl wrote:
> Marcel Holtmann wrote:
> > 
> > If the versioning script would handle regular expression by itself, then
> > that would be a lot simpler actually. At least I couldn't get that to
> > work.
> Afaik, this should work just fine. A version script like
> {
> global:
> 	conman_*;
> local:
> 	*;
> }
> should work in your case.
> Problem is, that within NM, the nm_* prefix is also used for private
> functions, so we can't use that, unless we change all those private
> functions to something like _nm_* (which would mean a lot of code changes)

I don't really mind renaming internal things since that's obviously the
"correct" thing to do.


> But as Dan already said, another benefit of explicitely listing all
> exported symbols is, that you have complete control over what is
> exported and what not and you can examine in one glance your complete
> API. (and we don't export any symbols by accident)
> Admittedly it is more maintenance work, keeping the version script
> up-to-date (and in sync with the installed header files). On the other
> hand, we should be really careful anyways, about changes to the API as
> soon as a stable version 0.7 is out. So this is imho actually a good
> thing, as it reminds when doing API changes.
> Cheers,
> Michael

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]