fixes #573, fixes #574, clean up conflicts with constants/globals

bhaller · bhaller · commit 499761869069 · 2025-11-08T09:14:28.000-05:00
diff --git a/EidosSLiMTests/SLiMTests.mm b/EidosSLiMTests/SLiMTests.mm
@@ -39,6 +39,7 @@ - (void)setUp {
 #ifdef _OPENMP
 	Eidos_WarmUpOpenMP(&SLIM_ERRSTREAM, changed_max_thread_count, (int)max_thread_count, true, /* max per-task thread counts */ "maxThreads");
 #endif
+	SLiM_ConfigureContext();
 	Eidos_WarmUp();
 	SLiM_WarmUp();
 	
diff --git a/QtSLiM/QtSLiMAppDelegate.cpp b/QtSLiM/QtSLiMAppDelegate.cpp
@@ -189,6 +189,7 @@ QtSLiMAppDelegate::QtSLiMAppDelegate(QObject *p_parent) : QObject(p_parent)
     QCoreApplication::setApplicationVersion(SLIM_VERSION_STRING);
     
     // Warm up our back ends before anything else happens, including our own class objects
+    SLiM_ConfigureContext();
     Eidos_WarmUp();
     SLiM_WarmUp();
     
diff --git a/QtSLiM/QtSLiMScriptTextEdit.cpp b/QtSLiM/QtSLiMScriptTextEdit.cpp
@@ -512,8 +512,13 @@ void QtSLiMTextEdit::scriptHelpOptionClick(QString searchString)
     else if (searchString == "//")              searchString = "comments";
     else if (searchString == "if")              searchString = "if and if–else statements";
     else if (searchString == "else")            searchString = "if and if–else statements";
+    else if (searchString == "do")              searchString = "do–while statements";
+    else if (searchString == "while")           searchString = "while statements";    // this brings up both while and do-while statements, correctly
     else if (searchString == "for")             searchString = "for statements";
     else if (searchString == "in")              searchString = "for statements";
+    else if (searchString == "next")            searchString = "next statements";
+    else if (searchString == "break")           searchString = "break statements";
+    else if (searchString == "return")          searchString = "return statements";
     else if (searchString == "function")        searchString = "user-defined functions";
     // and SLiM substitutions; "initialize" is deliberately omitted here so that the initialize...() methods also come up
     else if (searchString == "first")			searchString = "Eidos events";
diff --git a/QtSLiM/QtSLiMSyntaxHighlighting.cpp b/QtSLiM/QtSLiMSyntaxHighlighting.cpp
@@ -90,6 +90,7 @@ void QtSLiMOutputHighlighter::highlightBlock(const QString &text)
                         setFormat(match.capturedStart(), match.capturedLength(), genomicElementFormat);
                     else if (matchString[0] == 'm')
                         setFormat(match.capturedStart(), match.capturedLength(), mutationTypeFormat);
+					// we don't presently color sX or iX in the output
                 }
             }
         }
diff --git a/QtSLiM/QtSLiMWindow.cpp b/QtSLiM/QtSLiMWindow.cpp
@@ -2275,6 +2275,7 @@ void QtSLiMWindow::startNewSimulationFromScript(void)
         slimgui = new SLiMgui(*community, this);
 
         // set up the "slimgui" symbol for it immediately
+        // BCH 11/7/2025: note this symbol is now protected in SLiM_ConfigureContext()
         community->simulation_constants_->InitializeConstantSymbolEntry(slimgui->SymbolTableEntry());
     }
     
diff --git a/SLiMgui/AppDelegate.mm b/SLiMgui/AppDelegate.mm
@@ -230,6 +230,7 @@ - (void)applicationWillFinishLaunching:(NSNotification *)aNotification
 	Eidos_WarmUpOpenMP(&std::cout, true, 4, false, /* default per-task thread counts */ "");
 #endif
 	
+	SLiM_ConfigureContext();
 	Eidos_WarmUp();
 	SLiM_WarmUp();
 	
diff --git a/SLiMgui/SLiMWindowController.mm b/SLiMgui/SLiMWindowController.mm
@@ -595,6 +595,7 @@ - (void)startNewSimulationFromScript
 		slimgui = new SLiMgui(*community, self);
 		
 		// set up the "slimgui" symbol for it immediately
+		// BCH 11/7/2025: note this symbol is now protected in SLiM_ConfigureContext()
 		community->simulation_constants_->InitializeConstantSymbolEntry(slimgui->SymbolTableEntry());
 	}
 }
diff --git a/VERSIONS b/VERSIONS
@@ -16,6 +16,7 @@ development head (in the master branch):
 	add mtext() call to Plot, for drawing text in the margins outside the plot area
 	add rowSums() and colSums() functions to Eidos, for use with matrices as a faster alternative to apply()
 	add the PCG random number generator, switch to pcg32_fast and pcg64_fast, remove all use of the old taus2 and MT19937-64 generators; note this completely breaks backward reproducibility
+	fix various bugs involving conflicts between defined constants and other symbols, including #573 and #574; this sets new definition rules that could break some existing scripts (but is unlikely to)
 
 
 version 5.1 (Eidos version 4.1):
diff --git a/core/chromosome.cpp b/core/chromosome.cpp
@@ -1129,6 +1129,7 @@ Mutation *Chromosome::ApplyMutationCallbacks(Mutation *p_mut, Haplosome *p_haplo
 					// We can use that method because we know the lifetime of the symbol table is shorter than that of
 					// the value objects, and we know that the values we are setting here will not change (the objects
 					// referred to by the values may change, but the values themselves will not change).
+					// BCH 11/7/2025: note these symbols are now protected in SLiM_ConfigureContext()
 					if (mutation_callback->contains_mut_)
 					{
 						local_mut.StackAllocated();			// prevent Eidos_intrusive_ptr from trying to delete this
diff --git a/core/community.cpp b/core/community.cpp
@@ -2310,7 +2310,19 @@ void Community::AllSpecies_RunInitializeCallbacks(void)
 	
 	// set up global symbols for all species, and for ourselves
 	for (Species *species : all_species_)
-		simulation_constants_->InitializeConstantSymbolEntry(species->SymbolTableEntry());
+	{
+		EidosSymbolTableEntry &symbol_entry = species->SymbolTableEntry();
+		EidosGlobalStringID symbol_id = symbol_entry.first;
+		std::string symbol_string = EidosStringRegistry::StringForGlobalStringID(symbol_id);
+		
+		if (simulation_constants_->ContainsSymbol(symbol_id))
+			EIDOS_TERMINATION << "ERROR (Community::AllSpecies_RunInitializeCallbacks): A species with name '" << symbol_string << "' cannot be defined because that name is already in use." << EidosTerminate();
+		if (!Eidos_GoodSymbolForDefine(symbol_string) && (symbol_string != "sim"))
+			EIDOS_TERMINATION << "ERROR (Community::AllSpecies_RunInitializeCallbacks): A species with name '" << symbol_string << "' cannot be defined because the symbol '" << symbol_string << "' is reserved." << EidosTerminate();
+		
+		simulation_constants_->InitializeConstantSymbolEntry(symbol_entry);
+	}
+	
 	simulation_constants_->InitializeConstantSymbolEntry(SymbolTableEntry());
 	
 	// we're done with the initialization tick, so remove the zero-tick functions
diff --git a/core/interaction_type.cpp b/core/interaction_type.cpp
@@ -1356,6 +1356,7 @@ double InteractionType::ApplyInteractionCallbacks(Individual *p_receiver, Indivi
 					// We can use that method because we know the lifetime of the symbol table is shorter than that of
 					// the value objects, and we know that the values we are setting here will not change (the objects
 					// referred to by the values may change, but the values themselves will not change).
+					// BCH 11/7/2025: note these symbols are now protected in SLiM_ConfigureContext()
 					if (interaction_callback->contains_distance_)
 					{
 						local_distance.StackAllocated();		// prevent Eidos_intrusive_ptr from trying to delete this
diff --git a/core/log_file.cpp b/core/log_file.cpp
@@ -213,6 +213,7 @@ EidosValue_SP LogFile::_GeneratedValue_CustomScript(const LogFileGeneratorInfo &
 #endif
 			);
 		
+		// BCH 11/7/2025: note this symbol is now protected in SLiM_ConfigureContext()
 		callback_symbols.InitializeConstantSymbolEntry(gID_context, p_generator_info.context_);
 		
 		result_SP = interpreter.EvaluateInterpreterBlock(false, true);	// do not print output, return the last statement value
@@ -265,6 +266,7 @@ void LogFile::_GeneratedValues_CustomMeanAndSD(const LogFileGeneratorInfo &p_gen
 #endif
 			);
 		
+		// BCH 11/7/2025: note this symbol is now protected in SLiM_ConfigureContext()
 		callback_symbols.InitializeConstantSymbolEntry(gID_context, p_generator_info.context_);
 		
 		result_SP = interpreter.EvaluateInterpreterBlock(false, true);	// do not print output, return the last statement value
diff --git a/core/main.cpp b/core/main.cpp
@@ -374,6 +374,7 @@ int main(int argc, char *argv[])	// FIXME: clang-tidy flags this with bugprone-e
 #ifdef _OPENMP
 			Eidos_WarmUpOpenMP(&SLIM_ERRSTREAM, changed_max_thread_count, (int)max_thread_count, true, /* max per-task thread counts */ "maxThreads");
 #endif
+			SLiM_ConfigureContext();
 			Eidos_WarmUp();
 			SLiM_WarmUp();
 			
@@ -601,6 +602,7 @@ int main(int argc, char *argv[])	// FIXME: clang-tidy flags this with bugprone-e
 	}
 	
 	// run the simulation
+	SLiM_ConfigureContext();
 	Eidos_WarmUp();
 	SLiM_WarmUp();
 	
diff --git a/core/population.cpp b/core/population.cpp
@@ -786,6 +786,7 @@ slim_popsize_t Population::ApplyMateChoiceCallbacks(slim_popsize_t p_parent1_ind
 				// We can use that method because we know the lifetime of the symbol table is shorter than that of
 				// the value objects, and we know that the values we are setting here will not change (the objects
 				// referred to by the values may change, but the values themselves will not change).
+				// BCH 11/7/2025: note these symbols are now protected in SLiM_ConfigureContext()
 				if (mate_choice_callback->contains_individual_)
 				{
 					Individual *parent1 = p_source_subpop->parent_individuals_[p_parent1_index];
@@ -1123,6 +1124,7 @@ bool Population::ApplyModifyChildCallbacks(Individual *p_child, Individual *p_pa
 			// We can use that method because we know the lifetime of the symbol table is shorter than that of
 			// the value objects, and we know that the values we are setting here will not change (the objects
 			// referred to by the values may change, but the values themselves will not change).
+			// BCH 11/7/2025: note these symbols are now protected in SLiM_ConfigureContext()
 			if (modify_child_callback->contains_child_)
 				callback_symbols.InitializeConstantSymbolEntry(gID_child, p_child->CachedEidosValue());
 			
@@ -2734,6 +2736,7 @@ bool Population::ApplyRecombinationCallbacks(Individual *p_parent, Haplosome *p_
 			// We can use that method because we know the lifetime of the symbol table is shorter than that of
 			// the value objects, and we know that the values we are setting here will not change (the objects
 			// referred to by the values may change, but the values themselves will not change).
+			// BCH 11/7/2025: note these symbols are now protected in SLiM_ConfigureContext()
 			if (recombination_callback->contains_individual_)
 				callback_symbols.InitializeConstantSymbolEntry(gID_individual, p_parent->CachedEidosValue());
 			if (recombination_callback->contains_haplosome1_)
diff --git a/core/slim_functions.cpp b/core/slim_functions.cpp
@@ -2146,6 +2146,9 @@ EidosValue_SP SLiM_ExecuteFunction_summarizeIndividuals(const std::vector<EidosV
 	EidosValue *perUnitArea_value = p_arguments[5].get();
 	EidosValue *spatiality_value = p_arguments[6].get();
 	
+	if (p_interpreter.SymbolTable().SymbolDefinedAnywhere(gID_individuals))
+		EIDOS_TERMINATION << "ERROR (SLiM_ExecuteFunction_summarizeIndividuals): summarizeIndividuals() can't set up a local variable named 'individuals' because a symbol with that name is already defined." << EidosTerminate();
+	
 	// Get individuals vector; complicated as usual by singleton vs. vector
 	int individuals_count = individuals_value->Count();
 	
@@ -2419,6 +2422,7 @@ EidosValue_SP SLiM_ExecuteFunction_summarizeIndividuals(const std::vector<EidosV
 			
 			// We set up a "constant" value for `individuals` that refers to the stack-allocated object vector made above
 			// For each grid cell we will munge the contents of that vector, without having to touch the symbol table again
+			// BCH 11/7/2025: Note that we now check for conflicts with this symbol up at the top
 			constants.InitializeConstantSymbolEntry(gID_individuals, EidosValue_SP(&individuals_vec));
 			
 			// go through the individuals and tally them
@@ -2465,7 +2469,7 @@ EidosValue_SP SLiM_ExecuteFunction_summarizeIndividuals(const std::vector<EidosV
 		catch (...)
 		{
 			// If exceptions throw, then we want to set up the error information to highlight the
-			// sapply() that failed, since we can't highlight the actual error.  (If exceptions
+			// function call that failed, since we can't highlight the actual error.  (If exceptions
 			// don't throw, this catch block will never be hit; exit() will already have been called
 			// and the error will have been reported from the context of the lambda script string.)
 			if (gEidosTerminateThrows)
diff --git a/core/slim_globals.cpp b/core/slim_globals.cpp
@@ -100,8 +100,10 @@ void SLiM_WarmUp(void)
 		// Set up our shared pool for Mutation objects
 		SLiM_CreateMutationBlock();
 		
-		// Configure the Eidos context information
-		SLiM_ConfigureContext();
+		// Make sure the Eidos context information has already been configured; this has to be done first thing,
+		// so that any customizations to Eidos that SLiM introduces take effect before anything else happens
+		if (gEidosContextVersion == 0.0)
+			EIDOS_TERMINATION << "ERROR (SLiM_WarmUp): (internal error) SLiM_ConfigureContext() was not called before SLiMWarmUp()." << EidosTerminate();
 		
 		// Allocate global permanents
 		gStaticEidosValue_StringA = EidosValue_String_SP(new (gEidosValuePool->AllocateChunk()) EidosValue_String(gStr_A));
@@ -1655,6 +1657,39 @@ void SLiM_ConfigureContext(void)
 		gEidosContextVersionString = std::string("SLiM version ") + std::string(SLIM_VERSION_STRING);
 		gEidosContextLicense = "SLiM is free software: you can redistribute it and/or\nmodify it under the terms of the GNU General Public\nLicense as published by the Free Software Foundation,\neither version 3 of the License, or (at your option)\nany later version.\n\nSLiM is distributed in the hope that it will be\nuseful, but WITHOUT ANY WARRANTY; without even the\nimplied warranty of MERCHANTABILITY or FITNESS FOR\nA PARTICULAR PURPOSE.  See the GNU General Public\nLicense for more details.\n\nYou should have received a copy of the GNU General\nPublic License along with SLiM.  If not, see\n<http://www.gnu.org/licenses/>.\n";
 		gEidosContextCitation = "To cite SLiM in publications please use:\n\nHaller, B.C., and Messer, P.W. (2023). SLiM 4:\nMultispecies eco-evolutionary modeling. The American\nNaturalist 201(5). DOI: https://doi.org/10.1086/723601\n\nFor papers using tree-sequence recording, please cite:\n\nHaller, B.C., Galloway, J., Kelleher, J., Messer, P.W.,\n& Ralph, P.L. (2019). Tree‐sequence recording in SLiM\nopens new horizons for forward‐time simulation of whole\ngenomes. Molecular Ecology Resources 19(2), 552-566.\nDOI: https://doi.org/10.1111/1755-0998.12968\n";
+		
+		// Make a list of symbols that are not allowed as global constants or variables, because they would
+		// potentially conflict with pseudo-parameters or other facilities downstream.
+		gEidosContextReservedSymbols.push_back("self");
+		gEidosContextReservedSymbols.push_back("community");
+		gEidosContextReservedSymbols.push_back("sim");
+		gEidosContextReservedSymbols.push_back("slimgui");
+		gEidosContextReservedSymbols.push_back("context");		// defined by LogFile
+		gEidosContextReservedSymbols.push_back("distance");		// defined in ApplyInteractionCallbacks()
+		gEidosContextReservedSymbols.push_back("strength");		// defined in ApplyInteractionCallbacks()
+		gEidosContextReservedSymbols.push_back("receiver");		// defined in ApplyInteractionCallbacks()
+		gEidosContextReservedSymbols.push_back("exerter");		// defined in ApplyInteractionCallbacks()
+		gEidosContextReservedSymbols.push_back("mut");			// defined in ApplyMutationEffectCallbacks() etc.
+		gEidosContextReservedSymbols.push_back("effect");		// defined in ApplyMutationEffectCallbacks()
+		gEidosContextReservedSymbols.push_back("individual");	// defined in ApplyMutationEffectCallbacks() etc.
+		gEidosContextReservedSymbols.push_back("subpop");		// defined in ApplyMutationEffectCallbacks() etc.
+		gEidosContextReservedSymbols.push_back("homozygous");	// defined in ApplyMutationEffectCallbacks()
+		gEidosContextReservedSymbols.push_back("fitness");		// defined in ApplySurvivalCallbacks()
+		gEidosContextReservedSymbols.push_back("draw");			// defined in ApplySurvivalCallbacks()
+		gEidosContextReservedSymbols.push_back("surviving");	// defined in ApplySurvivalCallbacks()
+		gEidosContextReservedSymbols.push_back("sourceSubpop");	// defined in ApplyMateChoiceCallbacks() etc.
+		gEidosContextReservedSymbols.push_back("weights");		// defined in ApplyMateChoiceCallbacks()
+		gEidosContextReservedSymbols.push_back("child");		// defined in ApplyModifyChildCallbacks()
+		gEidosContextReservedSymbols.push_back("parent1");		// defined in ApplyModifyChildCallbacks()
+		gEidosContextReservedSymbols.push_back("isSelfing");	// defined in ApplyModifyChildCallbacks()
+		gEidosContextReservedSymbols.push_back("isCloning");	// defined in ApplyModifyChildCallbacks()
+		gEidosContextReservedSymbols.push_back("parent2");		// defined in ApplyModifyChildCallbacks()
+		gEidosContextReservedSymbols.push_back("haplosome1");	// defined in ApplyRecombinationCallbacks()
+		gEidosContextReservedSymbols.push_back("haplosome2");	// defined in ApplyRecombinationCallbacks()
+		gEidosContextReservedSymbols.push_back("parent");		// defined in ApplyMutationCallbacks()
+		gEidosContextReservedSymbols.push_back("haplosome");	// defined in ApplyMutationCallbacks()
+		gEidosContextReservedSymbols.push_back("element");		// defined in ApplyMutationCallbacks()
+		gEidosContextReservedSymbols.push_back("originalNuc");	// defined in ApplyMutationCallbacks()
 	}
 }
 
diff --git a/core/slim_test.cpp b/core/slim_test.cpp
@@ -662,6 +662,23 @@ void _RunBasicTests(void)
 							)V0G0N");
 	
 	SLiMAssertScriptRaise(ltb3_script, "long-term reference has been kept", 0, false);	// sim.getValue('x') is not scoped, so there is a long-term reference
+	
+	// Test protection against conflicts and shadowing involving global constants and variables
+	SLiMAssertScriptSuccess("initialize() { defineConstant('foo', 17); } 1 early() { }");
+	SLiMAssertScriptRaise("initialize() { defineConstant('if', 17); } 1 early() { }", "identifier 'if' is reserved", __LINE__);
+	SLiMAssertScriptRaise("initialize() { defineConstant('PI', 17); } 1 early() { }", "identifier 'PI' is already defined", __LINE__);
+	SLiMAssertScriptRaise("initialize() { defineConstant('subpop', 17); } 1 early() { }", "identifier 'subpop' is reserved", __LINE__);
+	
+	SLiMAssertScriptSuccess("species foo initialize() { } ticks all 1 early() { }");
+	SLiMAssertScriptSuccess("species sim initialize() { } ticks all 1 early() { }");
+	SLiMAssertScriptRaise("species if initialize() { } ticks all 1 early() { }", "unexpected token '<if>' in species specifier", __LINE__);
+	SLiMAssertScriptRaise("species PI initialize() { } ticks all 1 early() { }", "that name is already in use", __LINE__, /* p_expect_error_position */ false);
+	SLiMAssertScriptRaise("species subpop initialize() { } ticks all 1 early() { }", "the symbol 'subpop' is reserved", __LINE__, /* p_expect_error_position */ false);
+	SLiMAssertScriptRaise("species all initialize() { defineConstant('foo', 17); } species foo initialize() { } ticks all 1 early() { }", "that name is already in use", __LINE__, /* p_expect_error_position */ false);
+	SLiMAssertScriptRaise("species all initialize() { defineConstant('sim', 17); } species sim initialize() { } ticks all 1 early() { }", "identifier 'sim' is reserved", __LINE__);
+	
+	SLiMAssertScriptSuccess("initialize() { initializeSLiMOptions(dimensionality='xy'); } 1 late() { sim.addSubpop('p1', 100); p1.individuals.setSpatialPosition(p1.pointUniform(100)); } late() { summarizeIndividuals(p1.individuals, c(10, 10), p1.spatialBounds, operation='individuals.size();', empty=0.0, perUnitArea=T); }");
+	SLiMAssertScriptRaise("initialize() { initializeSLiMOptions(dimensionality='xy'); defineConstant('individuals', 17); } 1 late() { sim.addSubpop('p1', 100); p1.individuals.setSpatialPosition(p1.pointUniform(100)); } late() { summarizeIndividuals(p1.individuals, c(10, 10), p1.spatialBounds, operation='individuals.size();', empty=0.0, perUnitArea=T); }", "can't set up a local variable named 'individuals'", __LINE__);
 }
 
 #pragma mark error position tracking
diff --git a/core/subpopulation.cpp b/core/subpopulation.cpp
diff --git a/eidos/eidos_functions_matrices.cpp b/eidos/eidos_functions_matrices.cpp
diff --git a/eidos/eidos_globals.cpp b/eidos/eidos_globals.cpp
diff --git a/eidos/eidos_globals.h b/eidos/eidos_globals.h
diff --git a/eidos/eidos_symbol_table.cpp b/eidos/eidos_symbol_table.cpp
diff --git a/eidos/eidos_symbol_table.h b/eidos/eidos_symbol_table.h
diff --git a/eidos/eidos_test_functions_other.cpp b/eidos/eidos_test_functions_other.cpp

Original file line number	Diff line number	Diff line change
`@@ -90,6 +90,7 @@ void QtSLiMOutputHighlighter::highlightBlock(const QString &text)`
`90`	`90`	`setFormat(match.capturedStart(), match.capturedLength(), genomicElementFormat);`
`91`	`91`	`else if (matchString[0] == 'm')`
`92`	`92`	`setFormat(match.capturedStart(), match.capturedLength(), mutationTypeFormat);`
	`93`	`+ // we don't presently color sX or iX in the output`
`93`	`94`	`}`
`94`	`95`	`}`
`95`	`96`	`}`
Original file line number	Diff line number	Diff line change
`@@ -2275,6 +2275,7 @@ void QtSLiMWindow::startNewSimulationFromScript(void)`
`2275`	`2275`	`slimgui = new SLiMgui(*community, this);`
`2276`	`2276`
`2277`	`2277`	`// set up the "slimgui" symbol for it immediately`
	`2278`	`+ // BCH 11/7/2025: note this symbol is now protected in SLiM_ConfigureContext()`
`2278`	`2279`	`community->simulation_constants_->InitializeConstantSymbolEntry(slimgui->SymbolTableEntry());`
`2279`	`2280`	`}`
`2280`	`2281`
Original file line number	Diff line number	Diff line change
`@@ -595,6 +595,7 @@ - (void)startNewSimulationFromScript`
`595`	`595`	`slimgui = new SLiMgui(*community, self);`
`596`	`596`
`597`	`597`	`// set up the "slimgui" symbol for it immediately`
	`598`	`+ // BCH 11/7/2025: note this symbol is now protected in SLiM_ConfigureContext()`
`598`	`599`	`community->simulation_constants_->InitializeConstantSymbolEntry(slimgui->SymbolTableEntry());`
`599`	`600`	`}`
`600`	`601`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1129,6 +1129,7 @@ Mutation Chromosome::ApplyMutationCallbacks(Mutation p_mut, Haplosome *p_haplo`
`1129`	`1129`	`// We can use that method because we know the lifetime of the symbol table is shorter than that of`
`1130`	`1130`	`// the value objects, and we know that the values we are setting here will not change (the objects`
`1131`	`1131`	`// referred to by the values may change, but the values themselves will not change).`
	`1132`	`+ // BCH 11/7/2025: note these symbols are now protected in SLiM_ConfigureContext()`
`1132`	`1133`	`if (mutation_callback->contains_mut_)`
`1133`	`1134`	`{`
`1134`	`1135`	`local_mut.StackAllocated(); // prevent Eidos_intrusive_ptr from trying to delete this`