Re: [PATCHES] symbol visibility



Dan Williams wrote:
> 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.
> 

Ok, so would you actually prefer this approach then?
A common nm_* prefix for publicly exported API and renaming of the
private API to _nm_* (or __nm_*) while symplifying the version script
and using a regexp nm_* (which means the version script doesn't have to
be touched on API changes)?


Cheers,
Michael

-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?

Attachment: signature.asc
Description: OpenPGP digital signature



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