Remove unused InitializationOptions in #initializeBytecodeTable

Only set the binding value for COGMTVM if it was actually found (this line was not removed since it is in a Pharo6-VM compatibility package).
hernanmd · Mar 9, 2023 · 4e547b7 · 4e547b7
1 parent 4bfceaa
commit 4e547b7
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/smalltalksrc/VMMaker/StackInterpreter.class.st b/smalltalksrc/VMMaker/StackInterpreter.class.st
@@ -626,9 +626,6 @@ StackInterpreter class >> initializeBytecodeTable [
 	VMBytecodeConstants falsifyBytecodeSetFlags: InitializationOptions.
 	BytecodeSetHasDirectedSuperSend := false.
 
-	(InitializationOptions at: #bytecodeTableInitializer ifAbsent: nil) 
-		ifNotNil: [ :initalizer | ^ self perform: initalizer ].
-
 	^ self initializeBytecodeTableForSistaV1
 ]
 

diff --git a/smalltalksrc/VMMakerCompatibilityForPharo6/PharoVMMaker.class.st b/smalltalksrc/VMMakerCompatibilityForPharo6/PharoVMMaker.class.st
@@ -211,15 +211,17 @@ PharoVMMaker >> vmMakerWith: interpreterClass memoryManager: memoryManager [
 
 	| platformDirectory |
 	VMMakerConfiguration initializeForPharo.
-	(interpreterClass bindingOf: #COGMTVM) value: false.
+
+	(interpreterClass bindingOf: #COGMTVM)
+		ifNotNil: [ : binding | binding value: false ].
 
 	platformDirectory := self platformDirectoryFor: memoryManager.
 
 
 	^ (VMMaker
 		   makerFor: interpreterClass
 		   and: StackToRegisterMappingCogit
-		   with: { 
+		   with: {
 				   #COGMTVM.
 				   false.
 				   #ObjectMemory.