[FFmpeg-devel] [PATCH 1/9] lavf: add AVIO_FLAG_EXCL flag system.
Reimar.Doeffinger at gmx.de
Sat Apr 19 08:47:35 CEST 2014
On 18.04.2014, at 23:35, Nicolas George <george at nsup.org> wrote:
> Le septidi 27 germinal, an CCXXII, Michael Niedermayer a écrit :
>>> libavformat/avio.c | 12 +++++++++++-
>>> libavformat/avio.h | 18 ++++++++++++++++++
>>> libavformat/url.h | 1 +
>>> 3 files changed, 30 insertions(+), 1 deletion(-)
> Thanks for the review. I will leave a few more days for people who may want
> to comment on the API itself, especially the EXCL / EXCL_LOOSE part, before
> requesting a merge.
I am not really sure we'll be happy with it in the long term, also because the "normal" default behaviour may differ e.g. when thinking of files on Windows.
But I don't have a better idea, so we'll just have to deal with it when we actually find issues.
> By the way, if people would have preferred the six trivial ones bundled
> together, just tell me for future similar cases.
I kind of prefer having things like adding new flags to existing structs as one patch, but it deosn't matter much.
More information about the ffmpeg-devel