3743

One way to ensure we always set old_name is to create a method to
initialize names as opposed to just assigning them.

Still not ideal because we still assign directly most of the time, so
it's easy to forget.

1 files changed, 1 insertions, 1 deletions

diff --git a/053recipe_header.cc b/053recipe_header.cc
index c8904ba3..7101d7d6 100644
--- a/053recipe_header.cc
+++ b/053recipe_header.cc
@@ -481,7 +481,7 @@ void fill_in_return_ingredients(const recipe_ordinal r) {
   const instruction& final_instruction = caller_recipe.steps.at(SIZE(caller_recipe.steps)-1);
   if (final_instruction.name != "reply" && final_instruction.name != "return") {
     instruction inst;
-    inst.name = "return";
+    inst.initialize_name("return");
     add_header_products(inst, caller_recipe);
     caller_recipe.steps.push_back(inst);
   }