[FFmpeg-devel] [PATCH]Be more verbose about H264 cropping errors

Carl Eugen Hoyos cehoyos at ag.or.at
Mon Aug 22 09:12:17 CEST 2011


Hi!

An alternative is to print all cropping values in both cases, but that makes 
the lines even longer.

Please comment, Carl Eugen
-------------- next part --------------
diff --git a/libavcodec/h264_ps.c b/libavcodec/h264_ps.c
index 3c7b04a..fa99372 100644
--- a/libavcodec/h264_ps.c
+++ b/libavcodec/h264_ps.c
@@ -403,10 +403,10 @@ int ff_h264_decode_seq_parameter_set(H264Context *h){
         sps->crop_top   = get_ue_golomb(&s->gb);
         sps->crop_bottom= get_ue_golomb(&s->gb);
         if(sps->crop_left || sps->crop_top){
-            av_log(h->s.avctx, AV_LOG_ERROR, "insane cropping not completely supported, this could look slightly wrong ...\n");
+            av_log(h->s.avctx, AV_LOG_ERROR, "insane cropping not completely supported, this could look slightly wrong ... (left: %d, top: %d)\n", sps->crop_left, sps->crop_top);
         }
         if(sps->crop_right >= crop_horizontal_limit || sps->crop_bottom >= crop_vertical_limit){
-            av_log(h->s.avctx, AV_LOG_ERROR, "brainfart cropping not supported, this could look slightly wrong ...\n");
+            av_log(h->s.avctx, AV_LOG_ERROR, "brainfart cropping not supported, this could look slightly wrong ... (right: %d, bottom: %d)\n", sps->crop_right, sps->crop_bottom);
         }
     }else{
         sps->crop_left  =


More information about the ffmpeg-devel mailing list