[Ffmpeg-devel] [PATCH 6/10] Remove "qdiff" OptionDef

Panagiotis Issaris takis.issaris
Wed Sep 20 02:09:41 CEST 2006


Hi,

The attached patch removes the "qdiff" OptionDef. There was an
inconsistency in the range checking as zero was allowed but the range
reported did not allow zero. The patch assumes the range checking was
correct, but the message wasn't.

 20 -    if (video_qdiff < 0 ||
 21 -        video_qdiff > 31) {
 22 -        fprintf(stderr, "qdiff must be >= 1 and <= 31\n");

Regression tests succeed.

 ffmpeg.c           |   13 -------------
 libavcodec/utils.c |    2 +-
 2 files changed, 1 insertion(+), 14 deletions(-)

With friendly regards,
Takis
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pi-20060920T013806-ffmpeg-remove_qdiff_optiondef.diff
Type: text/x-patch
Size: 3194 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060920/7cb02b96/attachment.bin>



More information about the ffmpeg-devel mailing list