[FFmpeg-devel] Brainfart cropping
Michael Niedermayer
michaelni
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>>
> !h->sps.frame_mbs_only_flag)){
> 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
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080910/3c56de59/attachment.pgp>
More information about the ffmpeg-devel
mailing list