From: Misty De Meo <mistyde...@gmail.com>

> Just move allocation after all declarations.

OK, changed.

> Do you check that this condition is set in code?
>
> I mean, if user set codec for wrong case.

Oh, good point. I thought about this, and then realized that for mono audio,
there is no difference between planar and non-planar. If the muxer only accepts
planar then it'll work in both cases and there doesn't have to be a special
check. That also matches the behaviour of the demuxer, which marks streams as
planar regardless of channel layout.

> this is set but never used

Good catch - removed. Left over from an older version of the header generation.

Misty De Meo (1):
  Add Sega FILM muxer

 Changelog                 |   1 +
 libavformat/Makefile      |   1 +
 libavformat/allformats.c  |   1 +
 libavformat/segafilmenc.c | 377 ++++++++++++++++++++++++++++++++++++++++++++++
 4 files changed, 380 insertions(+)
 create mode 100644 libavformat/segafilmenc.c

-- 
2.16.2

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to