add explicit braces to clarify which if the else goes with 78/9378/1
authorAlan Coopersmith <alan.coopersmith@oracle.com>
Sun, 13 Jan 2013 22:39:54 +0000 (14:39 -0800)
committerSung-Jin Park <sj76.park@samsung.com>
Tue, 3 Sep 2013 04:06:58 +0000 (00:06 -0400)
Suggested by clang:

xvinfo.c:125:25: warning: add explicit braces to avoid dangling else
      [-Wdangling-else]
                        else
                        ^

Change-Id: I4539a8d3115a21eef9400873d25c46088c85ab90
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
xvinfo.c

index 605af56..bdc69cd 100644 (file)
--- a/xvinfo.c
+++ b/xvinfo.c
@@ -118,12 +118,13 @@ main(int argc, char *argv[])
                             attributes[k].name,
                             attributes[k].min_value, attributes[k].max_value);
 
-                    if (attributes[k].flags & XvSettable)
+                    if (attributes[k].flags & XvSettable) {
                         if (!shortmode)
                             fprintf(stdout,
                                     "              client settable attribute\n");
                         else
                             fprintf(stdout, "              settable");
+                    }
 
                     if (attributes[k].flags & XvGettable) {
                         Atom the_atom;