[FFmpeg-devel] [PATCH] lavu/attributes: add ff_ prefix to attribute_deprecated
Anton Khirnov
anton at khirnov.net
Tue Mar 12 12:12:36 EET 2024
Quoting Andreas Rheinhardt (2024-03-12 10:21:52)
> Anton Khirnov:
> > Quoting Andreas Rheinhardt (2024-03-12 09:56:46)
> >> Anton Khirnov:
> >>> Otherwise it might conflict with caller-defined identifiers.
> >>>
> >>> ff because it is not supposed to be used by external callers.
> >>> ---
> >>
> >> This is public, so it should be av_attribute_deprecated.
> >
> > As the commit message says, this is not supposed to be used by external
> > callers. Or do you see any reason why it should be public?
> >
>
> It is public given that it is declared in a public header. That is what
> matters.
I agree that it is (implicitly) public now, but the question is whether
it should be. IMO it should not. Do you disagree?
> Furthermore, we have used the FF_ and AV_ prefixes for public macros and
> the av_ prefix for public symbols, but IIRC we do not use the ff_ prefix
> for public symbols at all, therefore I'd like us not to reserve another
> namespace.
We do reserve the ff namespace for private identifiers that need to be
exposed, e.g. in public headers or when linking statically.
--
Anton Khirnov
More information about the ffmpeg-devel
mailing list