From 5bf08422449f7eba9f8912db06ad1c0e29f70e60 Mon Sep 17 00:00:00 2001
From: Ulf Lilleengen <lulf@FreeBSD.org>
Date: Thu, 29 Jul 2010 16:06:40 +0000
Subject: [PATCH] - Avoid calling the copy constructor when it is not
 necessary.

---
 sbin/devd/devd.cc | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/sbin/devd/devd.cc b/sbin/devd/devd.cc
index c2ff49c6695f..7af6f4e3053e 100644
--- a/sbin/devd/devd.cc
+++ b/sbin/devd/devd.cc
@@ -264,7 +264,7 @@ match::~match()
 bool
 match::do_match(config &c)
 {
-	string value = c.get_variable(_var);
+	const string &value = c.get_variable(_var);
 	bool retval;
 
 	if (Dflag)
@@ -577,7 +577,7 @@ void
 config::expand_one(const char *&src, string &dst)
 {
 	int count;
-	string buffer, varstr;
+	string buffer;
 
 	src++;
 	// $$ -> $
@@ -615,8 +615,7 @@ config::expand_one(const char *&src, string &dst)
 		buffer.append(src++, 1);
 	} while (is_id_char(*src));
 	buffer.append("", 1);
-	varstr = get_variable(buffer.c_str());
-	dst.append(varstr);
+	dst.append(get_variable(buffer.c_str()));
 }
 
 const string