From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1879) id 68B903858D28; Mon, 18 Apr 2022 20:12:39 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 68B903858D28 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Simon Marchi To: gdb-cvs@sourceware.org Subject: [binutils-gdb] gdbsupport: make gdb_abspath return an std::string X-Act-Checkin: binutils-gdb X-Git-Author: Simon Marchi X-Git-Refname: refs/heads/master X-Git-Oldrev: e0c34637019b0a070780b57b50d9026c0aca16f4 X-Git-Newrev: 7ab2607f97e5deaeae91018edf3ef5b4255a842c Message-Id: <20220418201239.68B903858D28@sourceware.org> Date: Mon, 18 Apr 2022 20:12:39 +0000 (GMT) X-BeenThere: gdb-cvs@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-cvs mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 18 Apr 2022 20:12:39 -0000 https://sourceware.org/git/gitweb.cgi?p=3Dbinutils-gdb.git;h=3D7ab2607f97e5= deaeae91018edf3ef5b4255a842c commit 7ab2607f97e5deaeae91018edf3ef5b4255a842c Author: Simon Marchi Date: Wed Apr 13 17:31:02 2022 -0400 gdbsupport: make gdb_abspath return an std::string =20 I'm trying to switch these functions to use std::string instead of char arrays, as much as possible. Some callers benefit from it (can avoid doing a copy of the result), while others suffer (have to make one more copy). =20 Change-Id: Iced49b8ee2f189744c5072a3b217aab5af17a993 Diff: --- gdb/compile/compile.c | 4 ++-- gdb/corelow.c | 2 +- gdb/dwarf2/index-cache.c | 4 +--- gdb/main.c | 7 +------ gdb/objfiles.c | 4 ++-- gdb/source.c | 19 +++++++++++-------- gdb/top.c | 10 ++-------- gdb/tracefile-tfile.c | 2 +- gdbserver/server.cc | 20 ++++++++++---------- gdbsupport/pathstuff.cc | 41 ++++++++++++++++++++--------------------- gdbsupport/pathstuff.h | 2 +- 11 files changed, 52 insertions(+), 63 deletions(-) diff --git a/gdb/compile/compile.c b/gdb/compile/compile.c index 2a1f0f1bd6b..5cbb341b383 100644 --- a/gdb/compile/compile.c +++ b/gdb/compile/compile.c @@ -297,8 +297,8 @@ compile_file_command (const char *args, int from_tty) error (_("You must provide a filename for this command.")); =20 args =3D skip_spaces (args); - gdb::unique_xmalloc_ptr abspath =3D gdb_abspath (args); - std::string buffer =3D string_printf ("#include \"%s\"\n", abspath.get (= )); + std::string abspath =3D gdb_abspath (args); + std::string buffer =3D string_printf ("#include \"%s\"\n", abspath.c_str= ()); eval_compile_command (NULL, buffer.c_str (), scope, NULL); } =20 diff --git a/gdb/corelow.c b/gdb/corelow.c index a23dc81c5af..8c33fb7ebb2 100644 --- a/gdb/corelow.c +++ b/gdb/corelow.c @@ -469,7 +469,7 @@ core_target_open (const char *arg, int from_tty) gdb::unique_xmalloc_ptr filename (tilde_expand (arg)); if (strlen (filename.get ()) !=3D 0 && !IS_ABSOLUTE_PATH (filename.get ())) - filename =3D gdb_abspath (filename.get ()); + filename =3D make_unique_xstrdup (gdb_abspath (filename.get ()).c_str = ()); =20 flags =3D O_BINARY | O_LARGEFILE; if (write_files) diff --git a/gdb/dwarf2/index-cache.c b/gdb/dwarf2/index-cache.c index 18e60cbd78c..fb827e04e59 100644 --- a/gdb/dwarf2/index-cache.c +++ b/gdb/dwarf2/index-cache.c @@ -298,9 +298,7 @@ set_index_cache_directory_command (const char *arg, int= from_tty, cmd_list_element *element) { /* Make sure the index cache directory is absolute and tilde-expanded. = */ - gdb::unique_xmalloc_ptr abs - =3D gdb_abspath (index_cache_directory.c_str ()); - index_cache_directory =3D abs.get (); + index_cache_directory =3D gdb_abspath (index_cache_directory.c_str ()); global_index_cache.set_directory (index_cache_directory); } =20 diff --git a/gdb/main.c b/gdb/main.c index 8c0807ff83b..ec2b7b01752 100644 --- a/gdb/main.c +++ b/gdb/main.c @@ -133,12 +133,7 @@ set_gdb_data_directory (const char *new_datadir) "../foo" and "../foo" doesn't exist then we'll record $(pwd)/../foo w= hich isn't canonical, but that's ok. */ if (!IS_ABSOLUTE_PATH (gdb_datadir.c_str ())) - { - gdb::unique_xmalloc_ptr abs_datadir - =3D gdb_abspath (gdb_datadir.c_str ()); - - gdb_datadir =3D abs_datadir.get (); - } + gdb_datadir =3D gdb_abspath (gdb_datadir.c_str ()); } =20 /* Relocate a file or directory. PROGNAME is the name by which gdb diff --git a/gdb/objfiles.c b/gdb/objfiles.c index 0c71e0bd6a9..849c6d73cab 100644 --- a/gdb/objfiles.c +++ b/gdb/objfiles.c @@ -331,7 +331,7 @@ objfile::objfile (bfd *abfd, const char *name, objfile_= flags flags_) =20 objfile_alloc_data (this); =20 - gdb::unique_xmalloc_ptr name_holder; + std::string name_holder; if (name =3D=3D NULL) { gdb_assert (abfd =3D=3D NULL); @@ -344,7 +344,7 @@ objfile::objfile (bfd *abfd, const char *name, objfile_= flags flags_) else { name_holder =3D gdb_abspath (name); - expanded_name =3D name_holder.get (); + expanded_name =3D name_holder.c_str (); } original_name =3D obstack_strdup (&objfile_obstack, expanded_name); =20 diff --git a/gdb/source.c b/gdb/source.c index e9016573333..9d9ff4bbc3e 100644 --- a/gdb/source.c +++ b/gdb/source.c @@ -537,15 +537,15 @@ add_path (const char *dirname, char **which_path, int= parse_separators) =20 for (const gdb::unique_xmalloc_ptr &name_up : dir_vec) { - char *name =3D name_up.get (); + const char *name =3D name_up.get (); char *p; struct stat st; - gdb::unique_xmalloc_ptr new_name_holder; + std::string new_name_holder; =20 /* Spaces and tabs will have been removed by buildargv(). NAME is the start of the directory. P is the '\0' following the end. */ - p =3D name + strlen (name); + p =3D name_up.get () + strlen (name); =20 while (!(IS_DIR_SEPARATOR (*name) && p <=3D name + 1) /* "/" */ #ifdef HAVE_DOS_BASED_FILE_SYSTEM @@ -589,16 +589,18 @@ add_path (const char *dirname, char **which_path, int= parse_separators) if (name[0] =3D=3D '\0') goto skip_dup; if (name[0] =3D=3D '~') - new_name_holder.reset (tilde_expand (name)); + new_name_holder + =3D gdb::unique_xmalloc_ptr (tilde_expand (name)).get (); #ifdef HAVE_DOS_BASED_FILE_SYSTEM else if (IS_ABSOLUTE_PATH (name) && p =3D=3D name + 2) /* "d:" =3D> = "d:." */ - new_name_holder.reset (concat (name, ".", (char *) NULL)); + new_name_holder =3D std::string (name) + "."; #endif else if (!IS_ABSOLUTE_PATH (name) && name[0] !=3D '$') new_name_holder =3D gdb_abspath (name); else - new_name_holder.reset (savestring (name, p - name)); - name =3D new_name_holder.get (); + new_name_holder =3D std::string (name, p - name); + + name =3D new_name_holder.c_str (); =20 /* Unless it's a variable, check existence. */ if (name[0] !=3D '$') @@ -950,7 +952,8 @@ done: else if ((opts & OPF_RETURN_REALPATH) !=3D 0) *filename_opened =3D gdb_realpath (filename); else - *filename_opened =3D gdb_abspath (filename); + *filename_opened + =3D make_unique_xstrdup (gdb_abspath (filename).c_str ()); } =20 errno =3D last_errno; diff --git a/gdb/top.c b/gdb/top.c index e776ac2d70e..1cfffbeee7e 100644 --- a/gdb/top.c +++ b/gdb/top.c @@ -2167,12 +2167,7 @@ set_history_filename (const char *args, that was read. */ if (!history_filename.empty () && !IS_ABSOLUTE_PATH (history_filename.c_str ())) - { - gdb::unique_xmalloc_ptr temp - (gdb_abspath (history_filename.c_str ())); - - history_filename =3D temp.get (); - } + history_filename =3D gdb_abspath (history_filename.c_str ()); } =20 /* Whether we're in quiet startup mode. */ @@ -2444,7 +2439,6 @@ _initialize_top () const char *fname =3D ".gdb_history"; #endif =20 - gdb::unique_xmalloc_ptr temp (gdb_abspath (fname)); - history_filename =3D temp.get (); + history_filename =3D gdb_abspath (fname); } } diff --git a/gdb/tracefile-tfile.c b/gdb/tracefile-tfile.c index 107236330d1..f84ad2f4a3f 100644 --- a/gdb/tracefile-tfile.c +++ b/gdb/tracefile-tfile.c @@ -471,7 +471,7 @@ tfile_target_open (const char *arg, int from_tty) =20 gdb::unique_xmalloc_ptr filename (tilde_expand (arg)); if (!IS_ABSOLUTE_PATH (filename.get ())) - filename =3D gdb_abspath (filename.get ()); + filename =3D make_unique_xstrdup (gdb_abspath (filename.get ()).c_str = ()); =20 flags =3D O_BINARY | O_LARGEFILE; flags |=3D O_RDONLY; diff --git a/gdbserver/server.cc b/gdbserver/server.cc index 24925cbbb5f..33c42714e72 100644 --- a/gdbserver/server.cc +++ b/gdbserver/server.cc @@ -90,31 +90,31 @@ bool non_stop; static struct { /* Set the PROGRAM_PATH. Here we adjust the path of the provided binary if needed. */ - void set (gdb::unique_xmalloc_ptr &&path) + void set (const char *path) { - m_path =3D std::move (path); + m_path =3D path; =20 /* Make sure we're using the absolute path of the inferior when creating it. */ - if (!contains_dir_separator (m_path.get ())) + if (!contains_dir_separator (m_path.c_str ())) { int reg_file_errno; =20 /* Check if the file is in our CWD. If it is, then we prefix its name with CURRENT_DIRECTORY. Otherwise, we leave the name as-is because we'll try searching for it in $PATH. */ - if (is_regular_file (m_path.get (), ®_file_errno)) - m_path =3D gdb_abspath (m_path.get ()); + if (is_regular_file (m_path.c_str (), ®_file_errno)) + m_path =3D gdb_abspath (m_path.c_str ()); } } =20 /* Return the PROGRAM_PATH. */ - char *get () - { return m_path.get (); } + const char *get () + { return m_path.empty () ? nullptr : m_path.c_str (); } =20 private: /* The program name, adjusted if needed. */ - gdb::unique_xmalloc_ptr m_path; + std::string m_path; } program_path; static std::vector program_args; static std::string wrapper_argv; @@ -3076,7 +3076,7 @@ handle_v_run (char *own_buf) } } else - program_path.set (gdb::unique_xmalloc_ptr (new_program_name)); + program_path.set (new_program_name); =20 /* Free the old argv and install the new one. */ free_vector_argv (program_args); @@ -3933,7 +3933,7 @@ captured_main (int argc, char *argv[]) int i, n; =20 n =3D argc - (next_arg - argv); - program_path.set (make_unique_xstrdup (next_arg[0])); + program_path.set (next_arg[0]); for (i =3D 1; i < n; i++) program_args.push_back (xstrdup (next_arg[i])); =20 diff --git a/gdbsupport/pathstuff.cc b/gdbsupport/pathstuff.cc index a347123cf2b..dd6ffa49fb4 100644 --- a/gdbsupport/pathstuff.cc +++ b/gdbsupport/pathstuff.cc @@ -130,23 +130,23 @@ gdb_realpath_keepfile (const char *filename) =20 /* See gdbsupport/pathstuff.h. */ =20 -gdb::unique_xmalloc_ptr +std::string gdb_abspath (const char *path) { gdb_assert (path !=3D NULL && path[0] !=3D '\0'); =20 if (path[0] =3D=3D '~') - return gdb_tilde_expand_up (path); + return gdb_tilde_expand (path); =20 if (IS_ABSOLUTE_PATH (path) || current_directory =3D=3D NULL) - return make_unique_xstrdup (path); + return path; =20 /* Beware the // my son, the Emacs barfs, the botch that catch... */ - return gdb::unique_xmalloc_ptr - (concat (current_directory, - IS_DIR_SEPARATOR (current_directory[strlen (current_directory) - 1]) - ? "" : SLASH_STRING, - path, (char *) NULL)); + return string_printf + ("%s%s%s", current_directory, + (IS_DIR_SEPARATOR (current_directory[strlen (current_directory) - 1]) + ? "" : SLASH_STRING), + path); } =20 /* See gdbsupport/pathstuff.h. */ @@ -229,8 +229,8 @@ get_standard_cache_dir () if (xdg_cache_home !=3D NULL && xdg_cache_home[0] !=3D '\0') { /* Make sure the path is absolute and tilde-expanded. */ - gdb::unique_xmalloc_ptr abs (gdb_abspath (xdg_cache_home)); - return string_printf ("%s/gdb", abs.get ()); + std::string abs =3D gdb_abspath (xdg_cache_home); + return string_printf ("%s/gdb", abs.c_str ()); } #endif =20 @@ -238,8 +238,8 @@ get_standard_cache_dir () if (home !=3D NULL && home[0] !=3D '\0') { /* Make sure the path is absolute and tilde-expanded. */ - gdb::unique_xmalloc_ptr abs (gdb_abspath (home)); - return string_printf ("%s/" HOME_CACHE_DIR "/gdb", abs.get ()); + std::string abs =3D gdb_abspath (home); + return string_printf ("%s/" HOME_CACHE_DIR "/gdb", abs.c_str ()); } =20 #ifdef WIN32 @@ -247,8 +247,8 @@ get_standard_cache_dir () if (win_home !=3D NULL && win_home[0] !=3D '\0') { /* Make sure the path is absolute and tilde-expanded. */ - gdb::unique_xmalloc_ptr abs (gdb_abspath (win_home)); - return string_printf ("%s/gdb", abs.get ()); + std::string abs =3D gdb_abspath (win_home); + return string_printf ("%s/gdb", abs.c_str ()); } #endif =20 @@ -296,8 +296,8 @@ get_standard_config_dir () if (xdg_config_home !=3D NULL && xdg_config_home[0] !=3D '\0') { /* Make sure the path is absolute and tilde-expanded. */ - gdb::unique_xmalloc_ptr abs (gdb_abspath (xdg_config_home)); - return string_printf ("%s/gdb", abs.get ()); + std::string abs =3D gdb_abspath (xdg_config_home); + return string_printf ("%s/gdb", abs.c_str ()); } #endif =20 @@ -305,8 +305,8 @@ get_standard_config_dir () if (home !=3D NULL && home[0] !=3D '\0') { /* Make sure the path is absolute and tilde-expanded. */ - gdb::unique_xmalloc_ptr abs (gdb_abspath (home)); - return string_printf ("%s/" HOME_CONFIG_DIR "/gdb", abs.get ()); + std::string abs =3D gdb_abspath (home); + return string_printf ("%s/" HOME_CONFIG_DIR "/gdb", abs.c_str ()); } =20 return {}; @@ -347,9 +347,8 @@ find_gdb_home_config_file (const char *name, struct sta= t *buf) if (homedir !=3D nullptr && homedir[0] !=3D '\0') { /* Make sure the path is absolute and tilde-expanded. */ - gdb::unique_xmalloc_ptr abs (gdb_abspath (homedir)); - std::string path =3D (std::string (abs.get ()) + SLASH_STRING - + std::string (name)); + std::string abs =3D gdb_abspath (homedir); + std::string path =3D string_printf ("%s/%s", abs.c_str (), name); if (stat (path.c_str (), buf) =3D=3D 0) return path; } diff --git a/gdbsupport/pathstuff.h b/gdbsupport/pathstuff.h index 50e388aad04..04d9bf9029b 100644 --- a/gdbsupport/pathstuff.h +++ b/gdbsupport/pathstuff.h @@ -53,7 +53,7 @@ extern gdb::unique_xmalloc_ptr If CURRENT_DIRECTORY is NULL, this function returns a copy of PATH. */ =20 -extern gdb::unique_xmalloc_ptr gdb_abspath (const char *path); +extern std::string gdb_abspath (const char *path); =20 /* If the path in CHILD is a child of the path in PARENT, return a pointer to the first component in the CHILD's pathname below the