git://git.openafs.org
/
openafs.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
(parent:
3bbff16
)
gtx: Remove pointless assignments from gtxtest
author
Simon Wilkinson
<sxw@your-file-system.com>
Thu, 21 Feb 2013 20:12:35 +0000 (20:12 +0000)
committer
Jeffrey Altman
<jaltman@your-file-system.com>
Sun, 24 Feb 2013 19:27:28 +0000 (11:27 -0800)
Remove assignments to 'code' when we're not going to pay attention
to them.
Caught by clang-analyzer
Change-Id: Ib2f8c20c0bc741d0bb1b8ed4fdb46ea61511fc95
Reviewed-on: http://gerrit.openafs.org/9204
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
src/gtx/gtxtest.c
patch
|
blob
|
history
diff --git
a/src/gtx/gtxtest.c
b/src/gtx/gtxtest.c
index
fef3703
..
3626a36
100644
(file)
--- a/
src/gtx/gtxtest.c
+++ b/
src/gtx/gtxtest.c
@@
-45,7
+45,7
@@
ChangeMenuCmd(void *param1, void *param2)
if (code)
gtxframe_DisplayString(tf, "Can't delete menu!");
} else if (arock == 3) {
- code = gtxframe_ClearMenus(tf);
+ gtxframe_ClearMenus(tf);
gtxframe_AddMenu(frameA, "FrameB", "b");
gtxframe_AddMenu(frameA, "NewLabel", "c");
}
@@
-72,7
+72,7
@@
ChangeListCmd(void *param1, void *param2)
if (code)
gtxframe_DisplayString(tf, "Can't delete light!");
} else if (arock == 3) {
- code = gtxframe_ClearList(tf);
+ gtxframe_ClearList(tf);
gtxframe_AddToList(frameA, objectA);
gtxframe_AddToList(frameA, lightA);
}