[Ffmpeg-devel] [PATCH] fix error croping

Luca Abeni lucabe72
Fri Apr 6 10:15:14 CEST 2007


Hi Michael,

On Thu, 2007-04-05 at 20:28 +0200, Michael Niedermayer wrote:

[...]
> > > OK, I add a field for bytes_per_pixel to fix the issue. Please review the
> > > updated patch again. 
> > [...]
> > 
> > I think the patch now looks much better. But since it adds a field to
> > the PixFmtInfo structure I think someone else will have to approve it (I
> > do not feel comfortable with the fields is such structure).
> 
> cant the value of the new field be calculated from the existing ones
> ans so the new field be avoided?
This is what I was hoping too...
But I did not mentioned this possibility because I do not really
understand how to compute the value based on existing fields, since some
of them confuse me very much. For example, what is "nb_channels" -
number of channels (including alpha)? For planar formats, it is the
number of planes, but for packed formats... Why is "nb_channels = 1" for
PIX_FMT_YUYV422, and "nb_channels = 3" for PIX_FMT_RGB24?
Also, what's the meaning of *_chroma_shift for packed formats? (maybe
this is the field that could help in this case?)


			Thanks,
				Luca





More information about the ffmpeg-devel mailing list