[FFmpeg-devel] Brainfart cropping
Wed Sep 10 21:31:51 CEST 2008
On Wed, Sep 10, 2008 at 11:45:27AM -0700, Jason Garrett-Glaser wrote:
> For a while, I've dealt with tons of people complaining about the
> message "brainfart cropping not supported." I figured this implied
> that the crop was >= the normal crop amount (>= one MB in size,
> 16x16), but the code itself puts the threshold at >=8. This makes no
> sense, since this means that the vast majority of encoding resolutions
> will result in such a message being printed by ffmpeg!
> if(sps->crop_right >= 8 || sps->crop_bottom >= (8>>
> av_log(h->s.avctx, AV_LOG_ERROR, "brainfart cropping not
> supported, this could look slightly wrong ...\n");
> Are we sure that this is right?
the crop values are AFAIK in chroma samples, so 8 should be correct for 4:2:0
But it should be easy to check with the reference sw how much is cropped.
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Its not that you shouldnt use gotos but rather that you should write
readable code and code with gotos often but not always is less readable
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel