free_bar: make this function reusable

* set last_msg to NULL after freeing it
* no need to test for last_msg being NULL before freeing it
  (likewise in update_last_message)

1 files changed, 3 insertions, 4 deletions

diff --git a/src/bar.c b/src/bar.c
index ef72a87..8af3308 100644
--- a/src/bar.c
+++ b/src/bar.c
@@ -531,8 +531,7 @@ draw_mark (rp_screen *s, char *msg, int mark_start, int mark_end)
 static void
 update_last_message (char *msg, int mark_start, int mark_end)
 {
-  if (last_msg)
-    free (last_msg);
+  free (last_msg);
   last_msg = xstrdup (msg);
   last_mark_start = mark_start;
   last_mark_end = mark_end;
@@ -603,6 +602,6 @@ show_last_message (void)
 void
 free_bar (void)
 {
-  if (last_msg)
-    free (last_msg);
+  free (last_msg);
+  last_msg = NULL;
 }