Address review

Author: vstinner

Lib/threading.py

@@ -50,9 +50,8 @@
     _HAVE_THREAD_NATIVE_ID = False
 try:
     _set_name = _thread.set_name
-    _HAVE_SET_NAME = True
 except AttributeError:
-    _HAVE_SET_NAME = False
+    _set_name = None
 ThreadError = _thread.error
 try:
     _CRLock = _thread.RLock
@@ -1032,7 +1031,7 @@ def _bootstrap_inner(self):
             self._set_ident()
             if _HAVE_THREAD_NATIVE_ID:
                 self._set_native_id()
-            if _HAVE_SET_NAME and self._name:
+            if _set_name is not None and self._name:
                 _set_name(self._name)
             self._started.set()
             with _active_limbo_lock:

Modules/_threadmodule.c

@@ -2364,7 +2364,7 @@ Internal only. Return a non-zero integer that uniquely identifies the main threa
 of the main interpreter.");
 
 
-#ifdef HAVE_PTHREAD_SETNAME_NP
+#ifdef HAVE_PTHREAD_GETNAME_NP
 /*[clinic input]
 _thread._get_name
 
@@ -2387,7 +2387,7 @@ _thread__get_name_impl(PyObject *module)
     name[size] = 0;
     return PyUnicode_DecodeFSDefault(name);
 }
-#endif  // HAVE_PTHREAD_SETNAME_NP
+#endif  // HAVE_PTHREAD_GETNAME_NP
 
 
 #ifdef HAVE_PTHREAD_SETNAME_NP