Cleanup: Pass std::string arguments by reference (this avoids making
unnecessary copies) and label them const, per the
Google style guide.
Tested:
$ ./bootstrap_test.sh all
--
Change-Id: I17636703cf43283f71b9b913c130a51065bd896b
Reviewed-on: https://bazel-review.googlesource.com/1350
MOS_MIGRATED_REVID=93876332
diff --git a/src/main/cpp/blaze.cc b/src/main/cpp/blaze.cc
index edc9f2c..42dcda0 100644
--- a/src/main/cpp/blaze.cc
+++ b/src/main/cpp/blaze.cc
@@ -1325,7 +1325,7 @@
// Figure out the base directories based on embedded data, username, cwd, etc.
// Sets globals->options.install_base, globals->options.output_base,
// globals->lock_file, globals->jvm_log_file.
-static void ComputeBaseDirectories(const string self_path) {
+static void ComputeBaseDirectories(const string &self_path) {
// Only start a server when in a workspace because otherwise we won't do more
// than emit a help message.
if (!BlazeStartupOptions::InWorkspace(globals->workspace)) {
