diff options
author | Jeremy Huddleston <jeremyhu@apple.com> | 2011-05-08 16:52:55 -0700 |
---|---|---|
committer | Jeremy Huddleston <jeremyhu@apple.com> | 2011-05-08 16:52:55 -0700 |
commit | cde30320d98b95f7c0cec5bed906b7107124f8fa (patch) | |
tree | c9bf36794037b3fe66d7ac3b4c337ba3ca35d331 | |
parent | 289e8d27a8417ca94fd2063a44b84338ff499a3e (diff) |
Properly fallback on CWBackPixmap = None when failing to parse or allocate a solid color
xrefresh.c:319:3: warning: Value stored to 'action' is never read
action = doNone;
^ ~~~~~~
Found-by: clang static analyzer
Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>
-rw-r--r-- | xrefresh.c | 3 |
1 files changed, 2 insertions, 1 deletions
@@ -316,7 +316,8 @@ main(int argc, char *argv[]) } else { fprintf (stderr,"%s: unable to allocate color '%s'.\n", ProgramName, solidcolor); - action = doNone; + xswa.background_pixmap = None; + mask |= CWBackPixmap; } break; case doDefault: |