Arbitrary code cleanups

Just some stuff that piles up while "looking" at the code, which
eventually gets annoying to ignore while staging real changes.

* Replaced a few NULL occurrences with 0
* Rely on default parameter for std::vector::resize.
* Replaced a few "" with std::string()
* Prefer .empty() to == ""
* Removed a few comparisons with NULL
* Don't check pointers before deleting them
* Removed a bunch of redundant semicolons
* Made some global variables static (local to their compilation unit)
* Prefer prefix ++/-- operators to postfix versions when possible
* Corrected location of a comment

1 files changed, 2 insertions, 4 deletions

diff --git a/src/gui/ministatus.cpp b/src/gui/ministatus.cpp
index 0fdf59b1..f3a8b807 100644
--- a/src/gui/ministatus.cpp
+++ b/src/gui/ministatus.cpp
@@ -92,11 +92,9 @@ MiniStatusWindow::MiniStatusWindow():
 void MiniStatusWindow::setIcon(int index, AnimatedSprite *sprite)
 {
     if (index >= (int) mIcons.size())
-        mIcons.resize(index + 1, NULL);
-
-    if (mIcons[index])
-        delete mIcons[index];
+        mIcons.resize(index + 1);
 
+    delete mIcons[index];
     mIcons[index] = sprite;
 }