In preparation for https://github.com/llvm/llvm-project/pull/187031 The `SanitizedScriptingModuleName` will be re-used from `Platform` (in addition to `PlatformDarwin` where it currently lives). To do that we'll need to move it to a common place. `ScriptInterpreter` seems like the most natural place for this to live. I introduced a new virtual `GetSanitizedScriptingModuleName` that `ScriptInterpreter`s can override in the future if they have their own sanitization logic. I made the default implementation the one we've been using for Python because that way the unit-tests that currently mock the `ScriptInterpreter` don't need to copy the implementation. Also made `WarnIfInvalidUnsanitizedScriptExists` a protected static function on `Platform` because it didn't seem right for that to also live in `ScriptInterpreter`.
12 KiB
12 KiB