../src/xf86Wacom.c:511:16: warning: variable 'i' may be uninitialized when used 
here [-Wconditional-uninitialized]
                pInfo->name, i + 1, wait);

xsetwacom.c:2308:36: warning: variable 'x' may be uninitialized when used here 
[-Wconditional-uninitialized]
                return set_output_area(dpy, dev, x, y, width, height);
                                                 ^
xsetwacom.c:2308:39: warning: variable 'y' may be uninitialized when used here 
[-Wconditional-uninitialized]
                return set_output_area(dpy, dev, x, y, width, height);
                                                    ^
xsetwacom.c:2308:42: warning: variable 'width' may be uninitialized when used 
here [-Wconditional-uninitialized]
                return set_output_area(dpy, dev, x, y, width, height);
                                                       ^~~~~
xsetwacom.c:2308:49: warning: variable 'height' may be uninitialized when used 
here [-Wconditional-uninitialized]
                return set_output_area(dpy, dev, x, y, width, height);
                                                              ^~~~~~
isdv4-serial-inputattach.c:107:6: warning: variable 'attr_id' may be 
uninitialized when used here [-Wconditional-uninitialized]
        if (attr_id && sscanf(attr_id, "WACf%x", &id) == 1 && id >= 0x8)
            ^~~~~~~
isdv4-serial-inputattach.c:179:22: warning: variable 'baudrate' may be 
uninitialized when used here [-Wconditional-uninitialized]
        set_serial_attr(fd, baudrate);
                            ^~~~~~~~

Signed-off-by: Jason Gerecke <killert...@gmail.com>
---
 src/xf86Wacom.c                  |  2 +-
 tools/isdv4-serial-inputattach.c |  6 +++---
 tools/xsetwacom.c                | 18 ++++++------------
 3 files changed, 10 insertions(+), 16 deletions(-)

diff --git a/src/xf86Wacom.c b/src/xf86Wacom.c
index de0438f..a04f936 100644
--- a/src/xf86Wacom.c
+++ b/src/xf86Wacom.c
@@ -480,7 +480,7 @@ Bool wcmIsWacomDevice (char* fname)
 char *wcmEventAutoDevProbe (InputInfoPtr pInfo)
 {
        /* We are trying to find the right eventX device */
-       int i, wait = 0;
+       int i = 0, wait = 0;
        const int max_wait = 2000;
 
        /* If device is not available after Resume, wait some ms */
diff --git a/tools/isdv4-serial-inputattach.c b/tools/isdv4-serial-inputattach.c
index 1bddaf8..b8575e2 100644
--- a/tools/isdv4-serial-inputattach.c
+++ b/tools/isdv4-serial-inputattach.c
@@ -85,7 +85,7 @@ int get_baud_rate(int fd)
        int id;
        struct udev *udev;
        struct udev_device *device, *parent;
-       const char *attr_id;
+       const char *attr_id = NULL;
 
        fstat(fd, &st);
 
@@ -126,7 +126,7 @@ int main(int argc, char **argv)
         int sensor_id;
        char *filename;
        int fd, rc = 1;
-       int baudrate;
+       int baudrate = 0;
        int have_baudrate = 0;
 
        int c, optidx = 0;
@@ -149,7 +149,7 @@ int main(int argc, char **argv)
                        case 'b':
                                have_baudrate = 1;
                                baudrate = atoi(optarg);
-                               if (baudrate == 0) {
+                               if (baudrate <= 0) {
                                        usage();
                                        return 1;
                                }
diff --git a/tools/xsetwacom.c b/tools/xsetwacom.c
index 304f9fa..1f6148f 100644
--- a/tools/xsetwacom.c
+++ b/tools/xsetwacom.c
@@ -2267,14 +2267,14 @@ static Bool set_output_area(Display *dpy, XDevice *dev,
  */
 static Bool set_output_xrandr(Display *dpy, XDevice *dev, char *output_name)
 {
-       int i, found = 0;
+       int i, success = 0;
        int x, y, width, height;
        XRRScreenResources *res;
        XRROutputInfo *output_info;
        XRRCrtcInfo *crtc_info;
 
        res = XRRGetScreenResources(dpy, DefaultRootWindow(dpy));
-       for (i = 0; i < res->noutput && !found; i++)
+       for (i = 0; i < res->noutput; i++)
        {
                output_info = XRRGetOutputInfo(dpy, res, res->outputs[i]);
 
@@ -2295,24 +2295,18 @@ static Bool set_output_xrandr(Display *dpy, XDevice 
*dev, char *output_name)
 
                if (strcmp(output_info->name, output_name) == 0)
                {
-                       found = 1;
+                       TRACE("Setting CRTC %s\n", output_name);
+                       success = set_output_area(dpy, dev, x, y, width, 
height);
                        break;
                }
        }
        XRRFreeScreenResources(res);
 
-       /* crtc holds our screen info, need to compare to actual screen size */
-       if (found)
-       {
-               TRACE("Setting CRTC %s\n", output_name);
-               return set_output_area(dpy, dev, x, y, width, height);
-       } else
-       {
+       if (!success)
                printf("Unable to find output '%s'. "
                        "Output may not be connected.\n", output_name);
 
-               return False;
-       }
+       return success;
 }
 
 /**
-- 
2.1.0


------------------------------------------------------------------------------
Want excitement?
Manually upgrade your production database.
When you want reliability, choose Perforce
Perforce version control. Predictably reliable.
http://pubads.g.doubleclick.net/gampad/clk?id=157508191&iu=/4140/ostg.clktrk
_______________________________________________
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to