Couple of fixes from cppcheck via Tiago Cunha.
authornicm <nicm@openbsd.org>
Wed, 15 Jan 2014 11:44:18 +0000 (11:44 +0000)
committernicm <nicm@openbsd.org>
Wed, 15 Jan 2014 11:44:18 +0000 (11:44 +0000)
usr.bin/tmux/arguments.c
usr.bin/tmux/grid.c

index 46c012f..5d3fa33 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: arguments.c,v 1.7 2014/01/09 13:51:57 nicm Exp $ */
+/* $OpenBSD: arguments.c,v 1.8 2014/01/15 11:44:18 nicm Exp $ */
 
 /*
  * Copyright (c) 2010 Nicholas Marriott <nicm@users.sourceforge.net>
@@ -78,7 +78,6 @@ struct args *
 args_parse(const char *template, int argc, char **argv)
 {
        struct args     *args;
-       char            *ptr;
        int              opt;
 
        args = xcalloc(1, sizeof *args);
@@ -89,7 +88,7 @@ args_parse(const char *template, int argc, char **argv)
        while ((opt = getopt(argc, argv, template)) != -1) {
                if (opt < 0)
                        continue;
-               if (opt == '?' || (ptr = strchr(template, opt)) == NULL) {
+               if (opt == '?' || strchr(template, opt) == NULL) {
                        args_free(args);
                        return (NULL);
                }
index fd856f4..a123579 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: grid.c,v 1.32 2014/01/09 13:58:06 nicm Exp $ */
+/* $OpenBSD: grid.c,v 1.33 2014/01/15 11:44:18 nicm Exp $ */
 
 /*
  * Copyright (c) 2008 Nicholas Marriott <nicm@users.sourceforge.net>
@@ -124,7 +124,7 @@ grid_compare(struct grid *ga, struct grid *gb)
        struct grid_cell        *gca, *gcb;
        u_int                    xx, yy;
 
-       if (ga->sx != gb->sx || ga->sy != ga->sy)
+       if (ga->sx != gb->sx || ga->sy != gb->sy)
                return (1);
 
        for (yy = 0; yy < ga->sy; yy++) {