[FFmpeg-devel] [PATCH] Fix audiomatch complie warning

Steven Liu lingjiujianke at gmail.com
Wed Aug 24 17:20:11 EEST 2016


Making sure bestpos is initialized to zero to prevent gcc from kvetching.
It's harmless (although it's not obvious that it's harmless)
from code inspection:
tests/audiomatch.c: In function ‘main’:
tests/audiomatch.c:40: warning: ‘bestpos’ may be used uninitialized in this
function

Thanks to Moritz Barsnick for first bringing this to the attention.

Signed-off-by: LiuQi <liuqi at gosun.com>
---
 tests/audiomatch.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/tests/audiomatch.c b/tests/audiomatch.c
index 9752f68..9fb511f 100644
--- a/tests/audiomatch.c
+++ b/tests/audiomatch.c
@@ -37,11 +37,11 @@ int main(int argc, char **argv){
     FILE *f[2];
     int i, pos;
     int siglen, datlen;
-    int bestpos;
-    double bestc=0;
-    double sigamp= 0;
+    int bestpos = 0;
+    double bestc = 0;
+    double sigamp = 0;
     int16_t *signal, *data;
-    int maxshift= 16384;
+    int maxshift = 16384;

     if (argc < 3) {
         printf("audiomatch <testfile> <reffile>\n");
--
1.7.1
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Fix-audiomatch-complie-warning.patch
Type: application/octet-stream
Size: 1237 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160824/24f8b064/attachment.obj>


More information about the ffmpeg-devel mailing list