Quoting Vittorio Giovara (2015-04-02 14:58:03) > On Thu, Apr 2, 2015 at 2:17 PM, Martin Storsjö <[email protected]> wrote: > > The previous documentation was very vague and almost misleading. > > --- > > libavcodec/internal.h | 9 +++++++-- > > 1 file changed, 7 insertions(+), 2 deletions(-) > > > > diff --git a/libavcodec/internal.h b/libavcodec/internal.h > > index a681329..c3dfaa1 100644 > > --- a/libavcodec/internal.h > > +++ b/libavcodec/internal.h > > @@ -34,11 +34,16 @@ > > #include "config.h" > > > > /** > > - * Codec is thread safe. > > + * The codec doesn't touch global variables in the init function, allowing > > + * running the init function without locking any global mutexes. > > How about "The codec doesn't access any global variable in the init > function, allowing to call the init function without locking any > global mutexes."?
That would not be true, since the codec does access global variables -- e.g. AVCodec instances are global variables. The flag is specifically with _modifying_ global variables, not just reading them. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
