[FFmpeg-devel] [PATCH 1/1] This change adds an encoder for Camera metadata motion. This is a type of sensor data associated with video, such as GPS, acceleration, gyro, and camera orientation. It does not encode video itself, but rather, this metadata.

Louis O'Bryan louiso at google.com
Wed Jul 12 19:16:43 EEST 2017


On Wed, Jul 12, 2017 at 12:50 AM, wm4 <nfxjfg at googlemail.com> wrote:

> On Tue, 11 Jul 2017 16:17:33 -0700
> "Louis O'Bryan" <louiso-at-google.com at ffmpeg.org> wrote:
>
> > If I need to write a new atom under stsd for my stream in the mov muxer
> > <https://github.com/FFmpeg/FFmpeg/blob/master/libavformat/movenc.c>
> > (mov_write_stsd_tag),
> > is it appropriate to indicate that through the AVStream metadata rather
> > than the codec_tag?
>
> It seemed to have lots of unrelated changes, but maybe I'm missing
> something. If those codec tag refactors are needed, they should
> probably be split into a separate patch.
>
> But it looks like most of those changes were unintended (Moritz
> suspected that too). The tag addition itself is probably fine.
>
> Also, please don't top post on mailing lists.
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

That file had unrelated changes that shouldn't have been there, please
ignore them.
Now that there is no codec associated with the stream, there shouldn't be a
codec tag at all, I would assume. (Another issue I need to deal with is
that the MOV muxer also doesn't support streams without a codec, but that
is separate.)


More information about the ffmpeg-devel mailing list