diff options
author | Paul Tagliamonte <paultag@fluxbox.org> | 2011-11-11 14:29:57 (GMT) |
---|---|---|
committer | Mathias Gumz <akira at fluxbox dot org> | 2012-01-04 06:43:21 (GMT) |
commit | 0f994dbb28e2a348f7c254e051622acf88e08226 (patch) | |
tree | 09d698eada6e44a3426336ce90fddf82c5411e17 | |
parent | a2cf6fff7b94cea7be87bb4e175c662c8c50a117 (diff) | |
download | fluxbox-0f994dbb28e2a348f7c254e051622acf88e08226.zip fluxbox-0f994dbb28e2a348f7c254e051622acf88e08226.tar.bz2 |
No need to check if a pointer is null before de-allocating it.
5.3.5/2 of the C++ standard:
"In either alternative, if the value of the operand of delete is the null
pointer the operation has no effect."
-rw-r--r-- | src/FbTk/MultiButtonMenuItem.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/FbTk/MultiButtonMenuItem.cc b/src/FbTk/MultiButtonMenuItem.cc index 1b61a99..f5b9990 100644 --- a/src/FbTk/MultiButtonMenuItem.cc +++ b/src/FbTk/MultiButtonMenuItem.cc | |||
@@ -41,8 +41,7 @@ MultiButtonMenuItem::MultiButtonMenuItem(int buttons, const FbTk::BiDiString &la | |||
41 | } | 41 | } |
42 | 42 | ||
43 | MultiButtonMenuItem::~MultiButtonMenuItem() { | 43 | MultiButtonMenuItem::~MultiButtonMenuItem() { |
44 | if (m_button_exe != 0) | 44 | delete [] m_button_exe; |
45 | delete [] m_button_exe; | ||
46 | } | 45 | } |
47 | 46 | ||
48 | void MultiButtonMenuItem::setCommand(int button, FbTk::RefCount<FbTk::Command<void> > &cmd) { | 47 | void MultiButtonMenuItem::setCommand(int button, FbTk::RefCount<FbTk::Command<void> > &cmd) { |