Re: [FFmpeg-devel] [PATCH]lavc/psd: Support 1bpp images

2017-01-10 Thread Carl Eugen Hoyos
2017-01-11 0:20 GMT+01:00 Martin Vignali : >> Please move the uncompressed calculation >> below the color_mode switch and commit. (... to fix possible rle compression.) > In attach new patch. Patch applied. Thank you, Carl Eugen ___ ffmpeg-devel mail

Re: [FFmpeg-devel] [PATCH]lavc/psd: Support 1bpp images

2017-01-10 Thread Martin Vignali
> > Please move the uncompressed calculation > below the color_mode switch and commit. > > In attach new patch. Martin From db1ca3645624fe8fac29f09cef3ef89dd013fef6 Mon Sep 17 00:00:00 2001 From: Martin Vignali Date: Wed, 11 Jan 2017 00:18:29 +0100 Subject: [PATCH] libavcodec/psd : add support fo

Re: [FFmpeg-devel] [PATCH]lavc/psd: Support 1bpp images

2017-01-10 Thread Carl Eugen Hoyos
2017-01-11 0:07 GMT+01:00 Martin Vignali : > 2017-01-10 23:56 GMT+01:00 Carl Eugen Hoyos : > >> 2017-01-10 23:53 GMT+01:00 Martin Vignali : >> >> > But maybe you can put the Store data code inside the planar part >> >> > (it's very similar) >> >> >> >> That means I have to remove the pixel_size con

Re: [FFmpeg-devel] [PATCH]lavc/psd: Support 1bpp images

2017-01-10 Thread Martin Vignali
2017-01-10 23:56 GMT+01:00 Carl Eugen Hoyos : > 2017-01-10 23:53 GMT+01:00 Martin Vignali : > >> > But maybe you can put the Store data code inside the planar part > >> > (it's very similar) > >> > >> That means I have to remove the pixel_size context variable: > >> Do you want me to do that? > >

Re: [FFmpeg-devel] [PATCH]lavc/psd: Support 1bpp images

2017-01-10 Thread Carl Eugen Hoyos
2017-01-10 23:53 GMT+01:00 Martin Vignali : >> > But maybe you can put the Store data code inside the planar part >> > (it's very similar) >> >> That means I have to remove the pixel_size context variable: >> Do you want me to do that? > > No i don't think you need to remove it. Then I don't know

Re: [FFmpeg-devel] [PATCH]lavc/psd: Support 1bpp images

2017-01-10 Thread Martin Vignali
> > But maybe you can put the Store data code inside the planar part (it's > very > > similar) > > That means I have to remove the pixel_size context variable: > Do you want me to do that? > > No i don't think you need to remove it. You can override for this case, the s->linesize inside the switch

Re: [FFmpeg-devel] [PATCH]lavc/psd: Support 1bpp images

2017-01-10 Thread Carl Eugen Hoyos
2017-01-10 23:42 GMT+01:00 Martin Vignali : > 2017-01-10 20:00 GMT+01:00 Carl Eugen Hoyos : > >> 2017-01-10 19:45 GMT+01:00 Gonzalo Garramuño : >> > >> > >> > El 10/01/2017 a las 10:21, Carl Eugen Hoyos escribió: >> >> >> >> } >> >> +} else if (avctx->pix_fmt == AV_PIX_FMT_MONOWHITE)

Re: [FFmpeg-devel] [PATCH]lavc/psd: Support 1bpp images

2017-01-10 Thread Martin Vignali
2017-01-10 20:00 GMT+01:00 Carl Eugen Hoyos : > 2017-01-10 19:45 GMT+01:00 Gonzalo Garramuño : > > > > > > El 10/01/2017 a las 10:21, Carl Eugen Hoyos escribió: > >> > >> } > >> +} else if (avctx->pix_fmt == AV_PIX_FMT_MONOWHITE) { > >> +ptr = picture->data[0]; > >> +

Re: [FFmpeg-devel] [PATCH]lavc/psd: Support 1bpp images

2017-01-10 Thread Carl Eugen Hoyos
2017-01-10 19:45 GMT+01:00 Gonzalo Garramuño : > > > El 10/01/2017 a las 10:21, Carl Eugen Hoyos escribió: >> >> } >> +} else if (avctx->pix_fmt == AV_PIX_FMT_MONOWHITE) { >> +ptr = picture->data[0]; >> +for (y = 0; y < s->height; y++) { >> +memcpy(ptr, ptr

Re: [FFmpeg-devel] [PATCH]lavc/psd: Support 1bpp images

2017-01-10 Thread Gonzalo Garramuño
El 10/01/2017 a las 10:21, Carl Eugen Hoyos escribió: } +} else if (avctx->pix_fmt == AV_PIX_FMT_MONOWHITE) { +ptr = picture->data[0]; +for (y = 0; y < s->height; y++) { +memcpy(ptr, ptr_data, s->width + 7 >> 3); +ptr_data += s->width + 7 >>

[FFmpeg-devel] [PATCH]lavc/psd: Support 1bpp images

2017-01-10 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #6044 for me. Please comment, Carl Eugen From 13fd23ed9dcb5e3affbb3456c64c3bb821b80613 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Tue, 10 Jan 2017 14:19:31 +0100 Subject: [PATCH] lavc/psd: Support 1bpp images. Fixes ticket #6044. --- libavcodec/psd.c