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

[Panagiotis Issaris]

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>