[FFmpeg-devel] [PATCH] Add FITS Encoder

[Paras Chadha]

On Fri, Jul 21, 2017 at 1:45 AM, James Almer <jamrial at gmail.com> wrote:

> On 7/20/2017 4:46 PM, Paras Chadha wrote:
> > diff --git a/libavformat/img2enc.c b/libavformat/img2enc.c
> > index 1297b1a..25283cc 100644
> > --- a/libavformat/img2enc.c
> > +++ b/libavformat/img2enc.c
> > @@ -237,6 +237,7 @@ AVOutputFormat ff_image2_muxer = {
> >  AVOutputFormat ff_image2pipe_muxer = {
> >      .name           = "image2pipe",
> >      .long_name      = NULL_IF_CONFIG_SMALL("piped image2 sequence"),
> > +    .extensions     = "fits",
>
> This is probably wrong. Did you intend to add this extension to
> ff_image2_muxer instead?
>

No, if there are multiple images, i want them to be appended one after
other. I think image2pipe is the correct muxer to use.


>
> Also, libavformat changes should be in their own separate commits.
>

Okay, i will send separated patches in a few days (waiting for any other
suggestion in the encoder).


>
> >      .priv_data_size = sizeof(VideoMuxData),
> >      .video_codec    = AV_CODEC_ID_MJPEG,
> >      .write_header   = write_header,
> > --
> > 2.4.11
> >
> > _______________________________________________
> > ffmpeg-devel mailing list
> > ffmpeg-devel at ffmpeg.org
> > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>