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-testcontroller.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-testcontroller.c')
| -rw-r--r-- | utilities/ovs-testcontroller.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/utilities/ovs-testcontroller.c b/utilities/ovs-testcontroller.c index a615ab49c..4ef7d4bb4 100644 --- a/utilities/ovs-testcontroller.c +++ b/utilities/ovs-testcontroller.c @@ -91,7 +91,7 @@ static char *unixctl_path = NULL; static void new_switch(struct switch_ *, struct vconn *); static void parse_options(int argc, char *argv[]); -static void usage(void) NO_RETURN; +NO_RETURN static void usage(void); int main(int argc, char *argv[]) |