From 24b9144d4ba83d37751786b08b48ad62fb7aef26 Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Tue, 13 Feb 2018 12:13:59 -0500 Subject: [PATCH] Use enum flags for flags passed to openp gdb/ChangeLog: * defs.h (enum openp_flags): New enum. (OPF_TRY_CWD_FIRST, OPF_SEARCH_IN_PATH, OPF_RETURN_REALPATH): Move to enum openp_flags. (openp_flags): New enum flags. (openp): Change parameter type to openp_flags. * source.c (openp): Change parameter type to openp_flags. * cli/cli-cmds.c (find_and_open_script): Use openp_flags. * dwarf2read.c (try_open_dwop_file): Use openp_flags. --- gdb/ChangeLog | 11 +++++++++++ gdb/cli/cli-cmds.c | 2 +- gdb/defs.h | 14 ++++++++++---- gdb/dwarf2read.c | 4 ++-- gdb/source.c | 2 +- 5 files changed, 25 insertions(+), 8 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 6ec60ae929..a09935f97e 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,14 @@ +2018-02-13 Simon Marchi + + * defs.h (enum openp_flags): New enum. + (OPF_TRY_CWD_FIRST, OPF_SEARCH_IN_PATH, OPF_RETURN_REALPATH): + Move to enum openp_flags. + (openp_flags): New enum flags. + (openp): Change parameter type to openp_flags. + * source.c (openp): Change parameter type to openp_flags. + * cli/cli-cmds.c (find_and_open_script): Use openp_flags. + * dwarf2read.c (try_open_dwop_file): Use openp_flags. + 2018-02-13 Simon Marchi * maint.c (_initialize_maint_cmds): Fix prefix of maint set/show diff --git a/gdb/cli/cli-cmds.c b/gdb/cli/cli-cmds.c index 18d1d4b383..c3962e939e 100644 --- a/gdb/cli/cli-cmds.c +++ b/gdb/cli/cli-cmds.c @@ -496,7 +496,7 @@ gdb::optional find_and_open_script (const char *script_file, int search_path) { int fd; - int search_flags = OPF_TRY_CWD_FIRST | OPF_RETURN_REALPATH; + openp_flags search_flags = OPF_TRY_CWD_FIRST | OPF_RETURN_REALPATH; gdb::optional opened; gdb::unique_xmalloc_ptr file (tilde_expand (script_file)); diff --git a/gdb/defs.h b/gdb/defs.h index 4fb2129b30..70389ad745 100644 --- a/gdb/defs.h +++ b/gdb/defs.h @@ -342,11 +342,17 @@ extern const char *pc_prefix (CORE_ADDR); /* From source.c */ /* See openp function definition for their description. */ -#define OPF_TRY_CWD_FIRST 0x01 -#define OPF_SEARCH_IN_PATH 0x02 -#define OPF_RETURN_REALPATH 0x04 -extern int openp (const char *, int, const char *, int, char **); +enum openp_flag +{ + OPF_TRY_CWD_FIRST = 0x01, + OPF_SEARCH_IN_PATH = 0x02, + OPF_RETURN_REALPATH = 0x04, +}; + +DEF_ENUM_FLAGS_TYPE(openp_flag, openp_flags); + +extern int openp (const char *, openp_flags, const char *, int, char **); extern int source_full_path_of (const char *, char **); diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c index cbfd34154a..93453431c1 100644 --- a/gdb/dwarf2read.c +++ b/gdb/dwarf2read.c @@ -12880,7 +12880,7 @@ static gdb_bfd_ref_ptr try_open_dwop_file (struct dwarf2_per_objfile *dwarf2_per_objfile, const char *file_name, int is_dwp, int search_cwd) { - int desc, flags; + int desc; char *absolute_name; /* Blech. OPF_TRY_CWD_FIRST also disables searching the path list if FILE_NAME contains a '/'. So we can't use it. Instead prepend "." @@ -12899,7 +12899,7 @@ try_open_dwop_file (struct dwarf2_per_objfile *dwarf2_per_objfile, else search_path = xstrdup (debug_file_directory); - flags = OPF_RETURN_REALPATH; + openp_flags flags = OPF_RETURN_REALPATH; if (is_dwp) flags |= OPF_SEARCH_IN_PATH; desc = openp (search_path, flags, file_name, diff --git a/gdb/source.c b/gdb/source.c index 009bec5285..eba06f6e6b 100644 --- a/gdb/source.c +++ b/gdb/source.c @@ -736,7 +736,7 @@ is_regular_file (const char *name, int *errno_ptr) /* >>>> This should only allow files of certain types, >>>> eg executable, non-directory. */ int -openp (const char *path, int opts, const char *string, +openp (const char *path, openp_flags opts, const char *string, int mode, char **filename_opened) { int fd; -- 2.34.1