From de7de58736745534152260713233604a18fa1a08 Mon Sep 17 00:00:00 2001 From: "Ilya.Kazakevich" Date: Fri, 1 Aug 2025 03:30:31 +0200 Subject: [PATCH] Python: remove unsued arg from `PythonFlavorProvider`, close a couple if implementations GitOrigin-RevId: 671271e156e0da7a71d6b62cdb1df7d5f6c65e97 --- .../jetbrains/python/sdk/flavors/PythonSdkFlavor.java | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/python/python-sdk/src/com/jetbrains/python/sdk/flavors/PythonSdkFlavor.java b/python/python-sdk/src/com/jetbrains/python/sdk/flavors/PythonSdkFlavor.java index 3451ccf6b41a..2a407f518d08 100644 --- a/python/python-sdk/src/com/jetbrains/python/sdk/flavors/PythonSdkFlavor.java +++ b/python/python-sdk/src/com/jetbrains/python/sdk/flavors/PythonSdkFlavor.java @@ -225,7 +225,7 @@ public abstract class PythonSdkFlavor { } } - result.addAll(getPlatformFlavorsFromExtensions(addPlatformIndependent)); + result.addAll(getPlatformFlavorsFromExtensions()); // Sort flavors to make venv go before unix/windows, see method doc if (addPlatformIndependent) { @@ -234,13 +234,11 @@ public abstract class PythonSdkFlavor { return result; } - public static @NotNull List> getPlatformFlavorsFromExtensions(boolean isIndependent) { + private static @NotNull List> getPlatformFlavorsFromExtensions() { List> result = new ArrayList<>(); for (PythonFlavorProvider provider : PythonFlavorProvider.EP_NAME.getExtensionList()) { PythonSdkFlavor flavor = provider.getFlavor(); - if (flavor.isPlatformIndependent() == isIndependent) { - result.add(flavor); - } + result.add(flavor); } return result; } @@ -309,7 +307,7 @@ public abstract class PythonSdkFlavor { } } - for (PythonSdkFlavor flavor : getPlatformFlavorsFromExtensions(true)) { + for (PythonSdkFlavor flavor : getPlatformFlavorsFromExtensions()) { if (flavor.isValidSdkPath(sdkPath)) { return flavor; }