diff options
| author | Ben Pfaff <blp@nicira.com> | 2013-04-23 16:40:56 -0700 |
|---|---|---|
| committer | Ben Pfaff <blp@nicira.com> | 2013-05-03 13:29:46 -0700 |
| commit | 07fc4ed3410006950012aef8d78f017c5b833e98 (patch) | |
| tree | 549ced1c9f9e3fd9caeee9782aece83b2b1f69d7 /ovsdb/ovsdb-tool.c | |
| parent | d64e176c3521c5132c0d9afa5718d52b7465453a (diff) | |
| download | openvswitch-07fc4ed3410006950012aef8d78f017c5b833e98.tar.gz | |
Make most "struct option" instances "const".
Reducing non-const static data makes code more obviously thread-safe.
Although option parsing does not normally need to be thread-safe, I
don't know of a drawback to making its data const.
Signed-off-by: Ben Pfaff <blp@nicira.com>
Diffstat (limited to 'ovsdb/ovsdb-tool.c')
| -rw-r--r-- | ovsdb/ovsdb-tool.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ovsdb/ovsdb-tool.c b/ovsdb/ovsdb-tool.c index fe6ab423f..2ef4061aa 100644 --- a/ovsdb/ovsdb-tool.c +++ b/ovsdb/ovsdb-tool.c @@ -66,7 +66,7 @@ main(int argc, char *argv[]) static void parse_options(int argc, char *argv[]) { - static struct option long_options[] = { + static const struct option long_options[] = { {"more", no_argument, NULL, 'm'}, {"verbose", optional_argument, NULL, 'v'}, {"help", no_argument, NULL, 'h'}, |