diff options
author | Lukas Fleischer <calcurse@cryptocrack.de> | 2011-10-19 23:31:56 +0200 |
---|---|---|
committer | Lukas Fleischer <calcurse@cryptocrack.de> | 2011-11-02 18:32:17 +0100 |
commit | 7cc6305588acea9c7960abaacf823d62f798f5ba (patch) | |
tree | 415648b8c723eba9a34fc3fc76056f798e170477 /src/todo.c | |
parent | 44bc9605d6d3af9162dc917c43b7a466a24a230b (diff) | |
download | calcurse-7cc6305588acea9c7960abaacf823d62f798f5ba.zip |
Do not cast unused return values to void
A small style fix that removes all remaining "(void)" casts. Using these
isn't encouraged in GNU coding guidelines and doesn't serve a certain
purpose, except for satisfying a few static code analysis tools. We
already nuked some of these in previous patches, but this semantic patch
should fix what's left:
@@
identifier func;
@@
- (void)func (
+ func (
...);
Long lines were re-formatted manually.
Signed-off-by: Lukas Fleischer <calcurse@cryptocrack.de>
Diffstat (limited to 'src/todo.c')
-rw-r--r-- | src/todo.c | 8 |
1 files changed, 4 insertions, 4 deletions
@@ -187,9 +187,9 @@ void todo_write (struct todo *todo, FILE *f) { if (todo->note) - (void)fprintf (f, "[%d]>%s %s\n", todo->id, todo->note, todo->mesg); + fprintf (f, "[%d]>%s %s\n", todo->id, todo->note, todo->mesg); else - (void)fprintf (f, "[%d] %s\n", todo->id, todo->mesg); + fprintf (f, "[%d] %s\n", todo->id, todo->mesg); } /* Delete a note previously attached to a todo item. */ @@ -338,10 +338,10 @@ todo_chg_priority (int action) int do_chg = 1; backup = todo_get_item (hilt); - (void)strncpy (backup_mesg, backup->mesg, strlen (backup->mesg) + 1); + strncpy (backup_mesg, backup->mesg, strlen (backup->mesg) + 1); backup_id = backup->id; if (backup->note) - (void)strncpy (backup_note, backup->note, MAX_NOTESIZ + 1); + strncpy (backup_note, backup->note, MAX_NOTESIZ + 1); else backup_note[0] = '\0'; switch (action) |