Use mvwaddstr() instead of mvwprintw()

When we only want to display a string at a specific place of the
screen, there's no need to use the more complex mvwprintw(), use
mvwaddstr() instead.

This should be slightly more efficient, and, above all, it prevents
weird things to happen if our string contains a '%', being interpreted
as an unwanted format string.

Signed-off-by: Baptiste Jonglez <baptiste--git@jonglez.org>
Signed-off-by: Lukas Fleischer <calcurse@cryptocrack.de>

1 files changed, 2 insertions, 2 deletions

diff --git a/src/help.c b/src/help.c
index 6dbac96..69bb79d 100644
--- a/src/help.c
+++ b/src/help.c
@@ -103,7 +103,7 @@ help_write_pad(struct window *win, char *title, char *text, enum key action)
   rownum = 0;
   erase_window_part(win->p, rownum, colnum, BUFSIZ, win->w);
   custom_apply_attr(win->p, ATTR_HIGHEST);
-  mvwprintw(win->p, rownum, colnum, "%s", title);
+  mvwaddstr(win->p, rownum, colnum, title);
   if ((int)action != KEY_RESIZE && action < NBKEYS) {
     switch (action) {
     case KEY_END_OF_WEEK:
@@ -137,7 +137,7 @@ help_write_pad(struct window *win, char *title, char *text, enum key action)
   colnum = 0;
   rownum += get_help_lines(title);
   custom_remove_attr(win->p, ATTR_HIGHEST);
-  mvwprintw(win->p, rownum, colnum, "%s", text);
+  mvwaddstr(win->p, rownum, colnum, text);
   rownum += get_help_lines(text);
   return rownum;
 }