Fix REPL interrupt instrumentation conditional (#24837)

lihaoyi · web-flow · commit e73ff2c23b0c · 2025-12-27T10:36:46.000+01:00
Fixes a regression in #24514, which occurred in 3.8.0-RC3. We mixed up the double-negative when porting the conditional from a string-equality-check to the `setting.is`-check
diff --git a/repl/src/dotty/tools/repl/AbstractFileClassLoader.scala b/repl/src/dotty/tools/repl/AbstractFileClassLoader.scala
@@ -59,7 +59,8 @@ class AbstractFileClassLoader(root: AbstractFile, parent: ClassLoader, interrupt
 
     val bytes = file.toByteArray
 
-    if !interruptInstrumentation.is(InterruptInstrumentation.Enabled) then defineClassInstrumented(name, bytes)
+
+    if interruptInstrumentation.is(InterruptInstrumentation.Enabled) then defineClassInstrumented(name, bytes)
     else defineClass(name, bytes, 0, bytes.length)
   }
 

Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,8 @@ class AbstractFileClassLoader(root: AbstractFile, parent: ClassLoader, interrupt`
`59`	`59`
`60`	`60`	`val bytes = file.toByteArray`
`61`	`61`
`62`		`- if !interruptInstrumentation.is(InterruptInstrumentation.Enabled) then defineClassInstrumented(name, bytes)`
	`62`	`+`
	`63`	`+ if interruptInstrumentation.is(InterruptInstrumentation.Enabled) then defineClassInstrumented(name, bytes)`
`63`	`64`	`else defineClass(name, bytes, 0, bytes.length)`
`64`	`65`	`}`
`65`	`66`