[PATCH wayland] wayland-egl: Ignore underscored symbols in ABI check

Daniel Stone daniel at fooishbar.org
Wed Mar 21 09:28:03 UTC 2018


Hi,

On 21 March 2018 at 08:27, Pekka Paalanen <ppaalanen at gmail.com> wrote:
> On Tue, 20 Mar 2018 11:46:32 +0000, Daniel Stone <daniel at fooishbar.org> wrote:
>> inconvenience of maintaining a list of every linker's implementation
>> detail on every platform, outweighs the risk of an exported
>> underscore-prefixed symbol slipping through review.
>
> how about we make it a coding style rule to not have any symbols
> starting with an underscore?
>
> I don't recall any already being in the code for either Wayland or
> Weston, but I didn't check. They certainly are not public ABI as the
> ABI check script would have listed such.

Sure, works for me.

> Aren't symbols (functions, variables) starting with an underscore
> reserved in C so should not be used anyway?

My recollection is that double-underscore and underscore-uppercase is
forbidden for use apart from the implementation / standard library,
and anything else underscore (underscore-number or
underscore-lowercase) can only be used for static symbols. But I'm
sure Simon will be along shortly to correct the record. :)

Cheers,
Daniel


More information about the wayland-devel mailing list