cmNinjaTargetGenerator: Drop unnecessary mutation

GetPreprocessScanRule's caller always has `vars.Source` set to `$in`.
This commit is contained in:
Brad King
2020-11-05 17:36:13 -05:00
parent fcf3fc4447
commit 170cfc764d

View File

@@ -543,7 +543,8 @@ std::string GetScanCommand(const std::string& cmakeCmd, const std::string& tdi,
// Helper function to create dependency scanning rule, with optional
// explicit preprocessing step if preprocessCommand is non-empty
cmNinjaRule GetPreprocessScanRule(
const std::string& ruleName, cmRulePlaceholderExpander::RuleVariables& vars,
const std::string& ruleName,
cmRulePlaceholderExpander::RuleVariables const& vars,
const std::string& responseFlag, const std::string& flags,
const std::string& launcher,
cmRulePlaceholderExpander* const rulePlaceholderExpander,
@@ -566,7 +567,6 @@ cmNinjaRule GetPreprocessScanRule(
// Preprocessing uses the original source, compilation uses
// preprocessed output or original source
ppVars.Source = vars.Source;
vars.Source = "$in";
// Copy preprocessor definitions to the preprocessor rule.
ppVars.Defines = vars.Defines;