On Tue, Mar 28, 2017 at 07:37:54PM -0400, Ronald S. Bultje wrote:
> Fixes a reported (but false) race condition in tsan for fate-apng.
> ---
> libavcodec/pngdec.c | 39 +++++++++++++++++++--------------------
> 1 file changed, 19 insertions(+), 20 deletions(-)
>
> diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c
> index a4eb6cc..bbb9610 100644
> --- a/libavcodec/pngdec.c
> +++ b/libavcodec/pngdec.c
> @@ -45,7 +45,7 @@ typedef struct PNGDecContext {
> ThreadFrame last_picture;
> ThreadFrame picture;
>
> - int state;
> + int state, pstate;I think its clearer if both variable names differ from the prior one also the constants should be renamed consistently with the field names they are used with. Otherwise its a bit confusing what constant belongs to which state variable thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Breaking DRM is a little like attempting to break through a door even though the window is wide open and the only thing in the house is a bunch of things you dont want and which you would get tomorrow for free anyway
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
