This is an automatic generated email to let you know that the following patch 
were queued at the 
http://git.linuxtv.org/cgit.cgi/v4l-utils.git tree:

Subject: v4l2-ctl: fix compile warning
Author:  Hans Verkuil <hverkuil-ci...@xs4all.nl>
Date:    Mon Sep 2 14:35:05 2019 +0200

  CXX      v4l2_ctl-v4l2-ctl-common.o
In function ‘bool parse_next_subopt(char**, char**)’,
    inlined from ‘bool parse_next_subopt(char**, char**)’ at 
v4l2-ctl-common.cpp:783:13:
v4l2-ctl-common.cpp:792:9: warning: ‘%s’ directive argument is null 
[-Wformat-overflow=]
  792 |  fprintf(stderr, "No value given to suboption <%s>\n",
      |  ~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  793 |    subopts[opt]);
      |    ~~~~~~~~~~~~~

The <%s> format is bogus, drop it.

Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 utils/v4l2-ctl/v4l2-ctl-common.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

---

http://git.linuxtv.org/cgit.cgi/v4l-utils.git/commit/?id=119c5e16d1cf9b0bcd5c1c29f19eac11fdfa57d6
diff --git a/utils/v4l2-ctl/v4l2-ctl-common.cpp 
b/utils/v4l2-ctl/v4l2-ctl-common.cpp
index 388c57b90ed3..651917e7c4a0 100644
--- a/utils/v4l2-ctl/v4l2-ctl-common.cpp
+++ b/utils/v4l2-ctl/v4l2-ctl-common.cpp
@@ -789,8 +789,7 @@ static bool parse_next_subopt(char **subs, char **value)
 
        if (opt < 0 || *value)
                return false;
-       fprintf(stderr, "No value given to suboption <%s>\n",
-                       subopts[opt]);
+       fprintf(stderr, "Missing suboption value\n");
        return true;
 }
 

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to