[FFmpeg-devel] [PATCH] Add FITS Demuxer

[Paras Chadha]

On Tue, Jul 4, 2017 at 10:20 PM, Nicolas George <george at nsup.org> wrote:

> Le sextidi 16 messidor, an CCXXV, Paras Chadha a écrit :
> > There is no global header.
> >
> > Basically FITS files can have multiple images.
> <snip>
>
> Thanks for all the details. When there are several images, they are all
> one after the other?
>

No, it is not mandatory that they are after one another. There can be other
extensions between image extensions too.


>
> If so, then I really think you should stop the demuxer and integrate
> FITS into the image2 system. See libavformat/img2dec.c: you only need a
> parser and a probe function, which is basically a subset of what you
> have written already.
>
> One of the benefits is that the parser is in libavcodec, so it is much
> easier to have it use directly the function fits_read_header() from the
> decoder.


So, now should i do this ? Even if i am able to call read_header() from
there, i will have to modify read_header() a lot and it will become very
complex and long.


>
> Regards,
>
> --
>   Nicolas George
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>


-- 
Have a Nice Day !