diff options
author | Gurucharan Shetty <gshetty@nicira.com> | 2014-09-15 12:58:09 -0700 |
---|---|---|
committer | Gurucharan Shetty <gshetty@nicira.com> | 2014-09-15 15:15:35 -0700 |
commit | 270f328621bfd8f80659150d5d3eec51754b5bfb (patch) | |
tree | c068c18ce82dcee21750e55f61ee2b3e4e157064 /utilities/ovs-dpctl.c | |
parent | d72eff6cec01187d9b6b0f8befdbdec0943e6012 (diff) | |
download | openvswitch-270f328621bfd8f80659150d5d3eec51754b5bfb.tar.gz |
compiler: Define NO_RETURN for MSVC.
To prevent warnings such as "Not all control paths return a value",
we should define NO_RETURN for MSVC.
Currently for gcc, we add NO_RETURN at the end of function declaration.
But for MSVC, "__declspec(noreturn)" is needed at the beginning of function
declaration. So this commit moves NO_RETURN to the beginning of the function
declaration as it works with gcc and clang too.
Signed-off-by: Gurucharan Shetty <gshetty@nicira.com>
Acked-by: Ben Pfaff <blp@nicira.com>
Diffstat (limited to 'utilities/ovs-dpctl.c')
-rw-r--r-- | utilities/ovs-dpctl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/utilities/ovs-dpctl.c b/utilities/ovs-dpctl.c index 6c25bfdbf..94a6b90f2 100644 --- a/utilities/ovs-dpctl.c +++ b/utilities/ovs-dpctl.c @@ -44,7 +44,7 @@ static struct dpctl_params dpctl_p; -static void usage(void *userdata OVS_UNUSED) NO_RETURN; +NO_RETURN static void usage(void *userdata OVS_UNUSED); static void parse_options(int argc, char *argv[]); static void |