[FFmpeg-devel] [PATCH] Add FITS Encoder

Paras Chadha paraschadha18 at gmail.com
Fri Jul 21 17:26:16 EEST 2017


On Fri, Jul 21, 2017 at 3:27 AM, Carl Eugen Hoyos <ceffmpeg at gmail.com>
wrote:

> 2017-07-20 21:46 GMT+02:00 Paras Chadha <paraschadha18 at gmail.com>:
>
> > +            case AV_PIX_FMT_RGB24:
> > +            case AV_PIX_FMT_RGBA:
> > +                for (k = 0; k < naxis3; k++) {
> > +                    for (i = 0; i < avctx->height; i++) {
> > +                        ptr = p->data[0] + (avctx->height - i - 1) *
> p->linesize[0] + k;
> > +                        for (j = 0; j < avctx->width; j++) {
> > +                            bytestream_put_byte(&bytestream, ptr[0]);
> > +                            ptr += naxis3;
>
> (Sorry if this is nonsense:)
> Shouldn't you be using PIX_FMT_GBRP and PIX_FMT_GBRAP?
>

No, the current pixel formats are working fine. I have tested using GIMP
too. It is showing all the RGB images (and others too) correctly.


>
> Carl Eugen
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list