From 6774c92b5809b80c767ef8094b2f26d06556e0fd Mon Sep 17 00:00:00 2001 From: Stephen Kelly Date: Wed, 11 Feb 2015 20:25:44 +0100 Subject: cmMacroCommand: Declare tmps in the scope that it's used. We don't particularly need to reuse the string memory here, and this pattern is not common in CMake. --- Source/cmMacroCommand.cxx | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/Source/cmMacroCommand.cxx b/Source/cmMacroCommand.cxx index bd53b1e..6635fee 100644 --- a/Source/cmMacroCommand.cxx +++ b/Source/cmMacroCommand.cxx @@ -84,8 +84,6 @@ bool cmMacroHelperCommand::InvokeInitialPass std::vector expandedArgs; this->Makefile->ExpandArguments(args, expandedArgs); - std::string tmps; - // make sure the number of arguments passed is at least the number // required by the signature if (expandedArgs.size() < this->Args.size() - 1) @@ -151,7 +149,7 @@ bool cmMacroHelperCommand::InvokeInitialPass } else { - tmps = k->Value; + std::string tmps = k->Value; // replace formal arguments for (unsigned int j = 0; j < variables.size(); ++j) { -- cgit v0.12