Address code review

corona10 · corona10 · commit 2ac6901e4351 · 2023-09-22T08:24:13.000+09:00
diff --git a/Doc/using/cmdline.rst b/Doc/using/cmdline.rst
@@ -547,7 +547,7 @@ Miscellaneous options
      will do nothing if is not supported on the current system. The default value
      is "off". See also :envvar:`PYTHONPERFSUPPORT` and :ref:`perf_profiling`.
    * :samp:`-X cpu_count={n}` overrides :func:`os.cpu_count`.
-     And *n* must be greater than 0.
+     *n* must be greater than or equal to 1.
      This option is useful for users who need to limit CPU resources of a container system.
      See also :envvar:`PYTHONCPUCOUNT`.
 
diff --git a/Doc/whatsnew/3.13.rst b/Doc/whatsnew/3.13.rst
@@ -167,7 +167,7 @@ os
 --
 
 * :func:`os.cpu_count` can be overrided  through the new
-  environment variable :envvar:`PYTHONCPUCOUNT`, the new command-line option
+  environment variable :envvar:`PYTHONCPUCOUNT` or the new command-line option
   :option:`-X cpu_count <-X>`. This option is useful for users who need to limit
   CPU resources of a container system.(Contributed by Donghee Na in :gh:`109595`)
 
diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
@@ -14353,6 +14353,7 @@ os_cpu_count_impl(PyObject *module)
     if (config->cpu_count > 0) {
         return PyLong_FromLong(config->cpu_count);
     }
+
     int ncpu = 0;
 #ifdef MS_WINDOWS
 #ifdef MS_WINDOWS_DESKTOP
diff --git a/Python/initconfig.c b/Python/initconfig.c
@@ -1690,9 +1690,9 @@ config_read_env_vars(PyConfig *config)
 static PyStatus
 config_init_cpu_count(PyConfig *config)
 {
-    int cpu_count = -1;
     const char *env = config_get_env(config, "PYTHONCPUCOUNT");
     if (env) {
+        int cpu_count = -1;
         if (_Py_str_to_int(env, &cpu_count) != 0) {
             cpu_count = -1;
         }
@@ -1702,6 +1702,7 @@ config_init_cpu_count(PyConfig *config)
     }
     const wchar_t *xoption = config_get_xoption(config, L"cpu_count");
     if (xoption) {
+        int cpu_count = -1;
         const wchar_t *sep = wcschr(xoption, L'=');
         if (sep) {
             if (config_wstr_to_int(sep + 1, &cpu_count) < 0 || cpu_count < 1) {

Original file line number	Diff line number	Diff line change
`@@ -14353,6 +14353,7 @@ os_cpu_count_impl(PyObject *module)`
`14353`	`14353`	`if (config->cpu_count > 0) {`
`14354`	`14354`	`return PyLong_FromLong(config->cpu_count);`
`14355`	`14355`	`}`
	`14356`	`+`
`14356`	`14357`	`int ncpu = 0;`
`14357`	`14358`	`#ifdef MS_WINDOWS`
`14358`	`14359`	`#ifdef MS_WINDOWS_DESKTOP`
Original file line number	Diff line number	Diff line change
`@@ -1690,9 +1690,9 @@ config_read_env_vars(PyConfig *config)`
`1690`	`1690`	`static PyStatus`
`1691`	`1691`	`config_init_cpu_count(PyConfig *config)`
`1692`	`1692`	`{`
`1693`		`- int cpu_count = -1;`
`1694`	`1693`	`const char *env = config_get_env(config, "PYTHONCPUCOUNT");`
`1695`	`1694`	`if (env) {`
	`1695`	`+ int cpu_count = -1;`
`1696`	`1696`	`if (_Py_str_to_int(env, &cpu_count) != 0) {`
`1697`	`1697`	`cpu_count = -1;`
`1698`	`1698`	`}`
`@@ -1702,6 +1702,7 @@ config_init_cpu_count(PyConfig *config)`
`1702`	`1702`	`}`
`1703`	`1703`	`const wchar_t *xoption = config_get_xoption(config, L"cpu_count");`
`1704`	`1704`	`if (xoption) {`
	`1705`	`+ int cpu_count = -1;`
`1705`	`1706`	`const wchar_t *sep = wcschr(xoption, L'=');`
`1706`	`1707`	`if (sep) {`
`1707`	`1708`	`if (config_wstr_to_int(sep + 1, &cpu_count) < 0 \|\| cpu_count < 1) {`