update code order

shihai1991 · shihai1991 · commit d08c18facc1d · 2020-08-05T23:12:07.000+08:00
diff --git a/Python/pystate.c b/Python/pystate.c
@@ -286,9 +286,6 @@ PyInterpreterState_Clear(PyInterpreterState *interp)
     }
     HEAD_UNLOCK(runtime);
 
-    PyObject *sysdict = interp->sysdict;
-    PyObject *builtins = interp->builtins;
-
     Py_CLEAR(interp->audit_hooks);
 
     PyConfig_Clear(&interp->config);
@@ -297,6 +294,8 @@ PyInterpreterState_Clear(PyInterpreterState *interp)
     Py_CLEAR(interp->codec_error_registry);
     Py_CLEAR(interp->modules);
     Py_CLEAR(interp->modules_by_index);
+    PyDict_Clear(interp->sysdict);
+    PyDict_Clear(interp->builtins);
     Py_CLEAR(interp->sysdict);
     Py_CLEAR(interp->builtins);
     Py_CLEAR(interp->builtins_copy);
@@ -311,13 +310,6 @@ PyInterpreterState_Clear(PyInterpreterState *interp)
     if (_PyRuntimeState_GetFinalizing(runtime) == NULL) {
         _PyWarnings_Fini(interp);
     }
-
-    /* We don't clear sysdict and builtins until the end of this function.
-       Because clearing other attributes can execute arbitrary Python code
-       which requires sysdict and builtins. */
-    PyDict_Clear(sysdict);
-    PyDict_Clear(builtins);
-
     // XXX Once we have one allocator per interpreter (i.e.
     // per-interpreter GC) we must ensure that all of the interpreter's
     // objects have been cleaned up at the point.
