diff options
| author | Ralf Baechle <ralf@linux-mips.org> | 2017-08-01 03:57:55 +0200 |
|---|---|---|
| committer | Ralf Baechle <ralf@linux-mips.org> | 2017-08-03 04:21:23 +0200 |
| commit | a9c535270d1224ff0e23fa6657e52be4659cf1ff (patch) | |
| tree | 100c6e2e7368226b1835385e6fda95cfd17116ac /kiss | |
| parent | 2b87888e8428bd55e2b30533914f903782a0c27c (diff) | |
treewide: Remove useless casts of function return value to void.
Arguably useful in documenting the return value is intentionally ignored
I think it's just cluttering the screen.
This is using the following semantic patch:
@castvoid@
expression F;
@@
- (void) F
+ F
(...)
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
Diffstat (limited to 'kiss')
| -rw-r--r-- | kiss/net2kiss.c | 19 |
1 files changed, 8 insertions, 11 deletions
diff --git a/kiss/net2kiss.c b/kiss/net2kiss.c index d9e8ff5..093a191 100644 --- a/kiss/net2kiss.c +++ b/kiss/net2kiss.c @@ -252,11 +252,10 @@ static int openpty(int *amaster, int *aslave, char *name, return -1; /* out of ptys */ } else { line[5] = 't'; - (void) chown(line, getuid(), - gr ? gr->gr_gid : -1); - (void) chmod(line, S_IRUSR|S_IWUSR|S_IWGRP); + chown(line, getuid(), gr ? gr->gr_gid : -1); + chmod(line, S_IRUSR|S_IWUSR|S_IWGRP); #if 0 - (void) revoke(line); + revoke(line); #endif slave = open(line, O_RDWR, 0); if (slave != -1) { @@ -265,16 +264,14 @@ static int openpty(int *amaster, int *aslave, char *name, if (name) strcpy(name, line); if (termp) - (void) tcsetattr(slave, - TCSAFLUSH, - termp); + tcsetattr(slave, TCSAFLUSH, + termp); if (winp) - (void) ioctl(slave, - TIOCSWINSZ, - (char *)winp); + ioctl(slave, TIOCSWINSZ, + (char *)winp); return 0; } - (void) close(master); + close(master); line[5] = 'p'; } } |