[FFmpeg-devel] [PATCH] Add FITS Demuxer

[Nicolas George]

Le sextidi 16 messidor, an CCXXV, Paras Chadha a écrit :
> So, now should i do this ?

Based on what you explained here, FITS seems like just an image format,
without provisions for animations like GIF or PNG. Therefore, it should
have been integrated in the img2 framework in the first place and
writing a dedicated demuxer was a mistake.

>			     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.

I do not see why you would need to modify fits_read_header() a lot:
fits_read_header() populates the FITSContext structure, and the parser
uses the fields to compute the size. Done.

You may need to make minimal changes to fits_read_header() to prevent it
from doing extra works, for example building metadata that will not be
used, but that is very minor.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170708/ab0e1ecf/attachment.sig>