Mark needless return(bla) with a macro so we can shut up FlexeLint
This commit is contained in:
parent
449dc16d47
commit
db0abbb4c8
|
@ -85,6 +85,15 @@
|
|||
*/
|
||||
#define __match_proto__(xxx) /*lint -e{818} */
|
||||
|
||||
/*
|
||||
* Some functions never return, and there's nothing Turing can do about it.
|
||||
* Some compilers think you should still include a final return(bla) in
|
||||
* such functions, while other tools complain about unreachable code.
|
||||
* Wrapping in a macro means we can shut the tools up.
|
||||
*/
|
||||
|
||||
#define NEEDLESS_RETURN(foo) return(foo)
|
||||
|
||||
/**********************************************************************
|
||||
* Compiler tricks
|
||||
*/
|
||||
|
|
|
@ -79,7 +79,7 @@ getdst(enum ocx_chan chan)
|
|||
if (chan == OCX_DEBUG)
|
||||
return (stdout);
|
||||
WRONG("Wrong ocx_chan");
|
||||
return (NULL);
|
||||
NEEDLESS_RETURN(NULL);
|
||||
}
|
||||
|
||||
static void __match_proto__()
|
||||
|
|
|
@ -167,7 +167,7 @@ tb_Now(struct timestamp *storage)
|
|||
|
||||
(void)storage;
|
||||
WRONG("No TB_Now");
|
||||
return (NULL);
|
||||
NEEDLESS_RETURN(NULL);
|
||||
}
|
||||
|
||||
tb_now_f *TB_Now = tb_Now;
|
||||
|
@ -179,7 +179,7 @@ tb_Sleep(double dur)
|
|||
{
|
||||
(void)dur;
|
||||
WRONG("No TB_Sleep");
|
||||
return (-1);
|
||||
NEEDLESS_RETURN(-1);
|
||||
}
|
||||
|
||||
tb_sleep_f *TB_Sleep = tb_Sleep;
|
||||
|
|
Loading…
Reference in New Issue