[FFmpeg-devel] [PATCH] lavf/img2dec: rename find_image_range() max_start param to start_index

Stefano Sabatini stefasab at gmail.com
Sat Aug 4 18:38:28 CEST 2012


The new name seems more consistent with the assumed logic.

"start_index" represents the minimum accepted value as first index, and
not the maximum value as implicitely assumed by the parameter name.
---
 libavformat/img2dec.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavformat/img2dec.c b/libavformat/img2dec.c
index 24006a8..402caa8 100644
--- a/libavformat/img2dec.c
+++ b/libavformat/img2dec.c
@@ -110,13 +110,13 @@ static int is_glob(const char *path)
 
 /* return -1 if no image found */
 static int find_image_range(int *pfirst_index, int *plast_index,
-                            const char *path, int max_start)
+                            const char *path, int start_index)
 {
     char buf[1024];
     int range, last_index, range1, first_index;
 
     /* find the first image */
-    for (first_index = max_start; first_index < max_start + 5; first_index++) {
+    for (first_index = start_index; first_index < start_index + 5; first_index++) {
         if (av_get_frame_filename(buf, sizeof(buf), path, first_index) < 0){
             *pfirst_index =
             *plast_index = 1;
-- 
1.7.5.4



More information about the ffmpeg-devel mailing list