aboutsummaryrefslogtreecommitdiff
path: root/gdb/auto-load.c
diff options
context:
space:
mode:
authorSimon Marchi <simon.marchi@polymtl.ca>2020-10-05 09:02:42 -0400
committerSimon Marchi <simon.marchi@polymtl.ca>2020-10-05 09:03:09 -0400
commit7d1441178ac6d5b31e3ebf312012c2d5bfaeec91 (patch)
tree752eb8287e0296cf2f176b3fbf4659a91345b49a /gdb/auto-load.c
parent6d96a5946d1ec76fea79bc36aa95d7d3dd304200 (diff)
downloadgdb-7d1441178ac6d5b31e3ebf312012c2d5bfaeec91.zip
gdb-7d1441178ac6d5b31e3ebf312012c2d5bfaeec91.tar.gz
gdb-7d1441178ac6d5b31e3ebf312012c2d5bfaeec91.tar.bz2
gdb: avoid unnecessary string copy in auto_load_objfile_script_1
Assigning the result of STRIP_DRIVE_SPEC to an std::string creates an unnecessary copy of the string. STRIP_DRIVE_SPEC is defined as: #define STRIP_DRIVE_SPEC(f) ((f) + 2) So if it is passed a "const char *", it returns a "const char *". We could use a "const char *" intermediary variable instead of an std::string, or (as implemented in this patch) just use it directly in the concatenation right after. gdb/ChangeLog: * auto-load.c (auto_load_objfile_script_1): Don't use debugfile_holder as temporary variable when stripping drive letter. Change-Id: If2ccc7a156b22100754d9cdf6778ac7eeb93da4c
Diffstat (limited to 'gdb/auto-load.c')
-rw-r--r--gdb/auto-load.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/gdb/auto-load.c b/gdb/auto-load.c
index 9a51d2f..43d007c 100644
--- a/gdb/auto-load.c
+++ b/gdb/auto-load.c
@@ -777,10 +777,8 @@ auto_load_objfile_script_1 (struct objfile *objfile, const char *realname,
/* Convert Windows file name from c:/dir/file to /c/dir/file. */
if (HAS_DRIVE_SPEC (debugfile))
- {
- debugfile_holder = STRIP_DRIVE_SPEC (debugfile);
- filename = std::string("\\") + debugfile[0] + debugfile_holder;
- }
+ filename = (std::string("\\") + debugfile[0]
+ + STRIP_DRIVE_SPEC (debugfile));
for (const gdb::unique_xmalloc_ptr<char> &dir : vec)
{