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/calcurse.h | |
| parent | 44bc9605d6d3af9162dc917c43b7a466a24a230b (diff) | |
| download | calcurse-7cc6305588acea9c7960abaacf823d62f798f5ba.tar.gz 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/calcurse.h')
| -rw-r--r-- | src/calcurse.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/calcurse.h b/src/calcurse.h index fb2be66..8b6d492 100644 --- a/src/calcurse.h +++ b/src/calcurse.h @@ -161,21 +161,21 @@ int len; \ \ len = snprintf (msg, BUFSIZ, "%s: %d: ", __FILE__, __LINE__); \ - (void)snprintf (msg + len, BUFSIZ - len, __VA_ARGS__); \ + snprintf (msg + len, BUFSIZ - len, __VA_ARGS__); \ if (ui_mode == UI_CURSES) \ fatalbox (msg); \ else \ - (void)fprintf (stderr, "%s\n", msg); \ + fprintf (stderr, "%s\n", msg); \ } while (0) #define WARN_MSG(...) do { \ char msg[BUFSIZ]; \ \ - (void)snprintf (msg, BUFSIZ, __VA_ARGS__); \ + snprintf (msg, BUFSIZ, __VA_ARGS__); \ if (ui_mode == UI_CURSES) \ warnbox (msg); \ else \ - (void)fprintf (stderr, "%s\n", msg); \ + fprintf (stderr, "%s\n", msg); \ } while (0) #define EXIT(...) do { \ |