[Ffmpeg-devel] [PATCH] H.263: More descriptive error message

[Tom Joseph]

I think it would be nice if ffmpeg had a more friendly and less  
confusing error message when h263 is told to deal with an invalid  
picture size.  The patch below is a one-liner.

Thanks,
--Tom Joseph

--- mpegvideo.c.old     2005-12-13 22:56:55.000000000 -0500
+++ mpegvideo.c 2005-12-13 23:00:02.000000000 -0500
@@ -1138,7 +1138,7 @@
          break;
      case CODEC_ID_H263:
          if (h263_get_picture_format(s->width, s->height) == 7) {
-            av_log(avctx, AV_LOG_INFO, "Input picture size isn't  
suitable for h263 codec! try h263+\n");
+            av_log(avctx, AV_LOG_INFO, "The specified picture size  
of %dx%d is not valid for the H.263 codec.  Valid sizes are 128x96,  
176x144, 352x288, 704x576, and 1408x1152.\n", s->width, s->height);
              return -1;
          }
          s->out_format = FMT_H263;