Apply suggestions from code review

Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
python · Eclips4 · Oct 31, 2024 · Oct 31, 2024 · Oct 31, 2024 · Oct 31, 2024
commit 49336ab89cde97502ea2deb4503f8b5f6e346b3b
diff --git a/Misc/NEWS.d/next/Core_and_Builtins/2024-11-10-04-59-43.gh-issue-126204.ASKDL1.rst b/Misc/NEWS.d/next/Core_and_Builtins/2024-11-10-04-59-43.gh-issue-126204.ASKDL1.rst
@@ -1 +1 @@
-Add a :func:`!sys._jit_enabled` helper to determine in runtime if JIT is enabled or not.
+Allow to determine whether the JIT is enabled at runtime via :func:`!sys._jit_enabled`.
@@ -1303,14 +1303,13 @@ init_interp_main(PyThreadState *tstate)
 // perf profiler works fine with tier 2 interpreter, so
 // only checking for a "real JIT".
 #if _Py_TIER % 2 != 0
-        int enabled = _PySys_JITEnabled();
-        printf("enabled = %d\n", enabled);
-        if (enabled && config->perf_profiling > 0) {
+        if (_PySys_JITEnabled() && config->perf_profiling > 0) {
             (void)PyErr_WarnEx(
-            PyExc_RuntimeWarning,
-            "JIT deactivated as perf profiling support is active",
-            0);
-        } else
+                PyExc_RuntimeWarning,
+	            "JIT deactivated as perf profiling support is active",
+	            0);
+        }
+        else
 #endif
     {
         PyObject *opt = _PyOptimizer_NewUOpOptimizer();

@@ -2541,7 +2541,7 @@ PyAPI_FUNC(int) PyUnstable_CopyPerfMapFile(const char* parent_filename) {
 int
 _PySys_JITEnabled(void)
 {
-    char *env = Py_GETENV("PYTHON_JIT");
+    const char *env = Py_GETENV("PYTHON_JIT");
     if (_Py_TIER2 == 1) {
         // Interpreter was built with enabled jit, but it can be disabled via PYTHON_JIT=0
         if (env && *env != '\0') {
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		Add a :func:`!sys._jit_enabled` helper to determine in runtime if JIT is enabled or not.
		Allow to determine whether the JIT is enabled at runtime via :func:`!sys._jit_enabled`.