[FFmpeg-devel] [PATCH] Add FITS Decoder
Paras Chadha
paraschadha18 at gmail.com
Tue Jul 4 19:22:29 EEST 2017
On Tue, Jul 4, 2017 at 2:57 PM, Nicolas George <george at nsup.org> wrote:
> Le quartidi 14 messidor, an CCXXV, Paras Chadha a écrit :
> > Made all the changes suggested
> > Added a new function which reads a header FITS line safely. It also
> makes it more modular
> > Added an option for the user to enter the value to be used in place of
> BLANK pixels
> > Refactored code using macros to make it short
>
> Note: this is the commit message, it is there to stay, so it must
> contain information about the whole commit, not its differences with a
> previous draft of it. These, and any other kind of comment, belong
> between the first triple-dash line and the beginning of the actual diff,
> where Git places the shortstats of the diff.
>
> Also, lines should be split way below 80 characters.
>
Okay, will remember this the next time
>
> >
> > Signed-off-by: Paras Chadha <paraschadha18 at gmail.com>
> > ---
> > Changelog | 1 +
> > doc/general.texi | 2 +
> > libavcodec/Makefile | 1 +
> > libavcodec/allcodecs.c | 1 +
> > libavcodec/avcodec.h | 1 +
> > libavcodec/codec_desc.c | 8 +
> > libavcodec/fitsdec.c | 505 ++++++++++++++++++++++++++++++
> ++++++++++++++++++
> > libavcodec/version.h | 4 +-
> > libavformat/img2.c | 1 +
> > 9 files changed, 522 insertions(+), 2 deletions(-)
> > create mode 100644 libavcodec/fitsdec.c
>
> Note: before reviewing the code, I think the boundary between the
> demuxer and the decoder needs to be properly defined. See my comments in
> this mail, near the end:
> https://ffmpeg.org/pipermail/ffmpeg-devel/2017-July/213157.html
okay
>
>
> Regards,
>
> --
> Nicolas George
>
--
Have a Nice Day !
More information about the ffmpeg-devel
mailing list