[FFmpeg-devel] [PATCH] Per-frame metadata

Nicolas George nicolas.george at normalesup.org
Wed Apr 13 14:04:49 CEST 2011


Le quartidi 24 germinal, an CCXIX, Stefano Sabatini a écrit :
> But:
> 
> if ((ret = ...             )<0) goto fail;
> ...
> if ((ret = av_size_mult(...)<0) goto fail;
> ...
> fail:
>     ...
>     return ret;

Makes sense.

> Isn't it already included? (mem.h includes avutil.h which includes
> error.h).

It was more subtle than I thought, in fact: there are cyclic dependencies:

	mem.h
	 `--avutil.h
	     +--common.h
	     |   `--mem.h
	     `--error.h

Thus, mem.h got parsed before error.h was really included.

Therefore, my objection to making mem.h depend on error.h is no longer
relevant. I'll commit the change in my working tree.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110413/12b8c3db/attachment.asc>


More information about the ffmpeg-devel mailing list