[FFmpeg-devel] [PATCH]Fix linesize for pix_fmt pal8

Stefano Sabatini stefasab at gmail.com
Sat Jun 30 23:41:39 CEST 2012

On date Wednesday 2012-06-20 07:33:04 +0000, Carl Eugen Hoyos encoded:
> Stefano Sabatini <stefasab <at> gmail.com> writes:
> > Check attached patch, it should fix the problem when coupled 
> > with the new imgutils functions.
> The patch breaks decoding some of the samples from ticket #823.

New try (I still don't like very much this approach), but at least I
realized that changing the pal8 output alignment is just wrong. And we
could use a better heuristic for detecting the linesize (for example
iterating the loop:
aligned_size(A) <= buf_size && aligned_size(A+1) >= buf_size
=> choose A

and may be also useful to be able to override the alignment in case of
need (working on a patch for that).

But apart from this I see more failures with other formats:
ffmpeg -i tests/lena.pnm -s 257x257 -vcodec rawvideo -pix_fmt PIX_FMT -y out.avi

in particular with formats monob, rgb565, rgb24, still investigating
(hints are welcome).
FFmpeg = Frenzy & Fascinating Monstrous Prodigious Elastic Ghost
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0004-lavc-rawdec-fix-palette-alignment-setting.patch
Type: text/x-diff
Size: 1965 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120630/c843a2b5/attachment.bin>

More information about the ffmpeg-devel mailing list