[binutils-gdb] Use enum flags for flags passed to openp

[Simon Marchi <simark at sourceware dot org>]

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=24b9144d4ba83d37751786b08b48ad62fb7aef26

commit 24b9144d4ba83d37751786b08b48ad62fb7aef26
Author: Simon Marchi <simon.marchi@polymtl.ca>
Date:   Tue Feb 13 12:13:59 2018 -0500

    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.

Diff:
---
 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 6ec60ae..a09935f 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,16 @@
 2018-02-13  Simon Marchi  <simon.marchi@polymtl.ca>
 
+	* 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  <simon.marchi@polymtl.ca>
+
 	* maint.c (_initialize_maint_cmds): Fix prefix of maint set/show
 	per-command.
 
diff --git a/gdb/cli/cli-cmds.c b/gdb/cli/cli-cmds.c
index 18d1d4b..c3962e9 100644
--- a/gdb/cli/cli-cmds.c
+++ b/gdb/cli/cli-cmds.c
@@ -496,7 +496,7 @@ gdb::optional<open_script>
 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<open_script> opened;
 
   gdb::unique_xmalloc_ptr<char> file (tilde_expand (script_file));
diff --git a/gdb/defs.h b/gdb/defs.h
index 4fb2129..70389ad 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 cbfd341..9345343 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 009bec5..eba06f6 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;