fix declared in 2 places

Author: BobTheBuidler

mypyc/lib-rt/CPy.h

@@ -74,6 +74,10 @@ static inline PyObject *CPyTuple_LoadEmptyTupleConstant(void) {
     return __mypyc_empty_tuple__;
 }
 
+// Shared unicode objects for method names
+extern PyObject *clear_id_unicode;
+extern PyObject *copy_id_unicode;
+
 // Native object operations
 
 

mypyc/lib-rt/dict_ops.c

@@ -15,8 +15,7 @@ static PyObject *update_id_unicode = NULL;
 static PyObject *keys_id_unicode = NULL;
 static PyObject *values_id_unicode = NULL;
 static PyObject *items_id_unicode = NULL;
-static PyObject *clear_id_unicode = NULL;
-static PyObject *copy_id_unicode = NULL;
+// clear_id_unicode and copy_id_unicode are shared with list_ops, declared in misc_ops.c
 
 // Dict subclasses like defaultdict override things in interesting
 // ways, so we don't want to just directly use the dict methods. Not

mypyc/lib-rt/list_ops.c

@@ -9,8 +9,7 @@
 #define Py_TPFLAGS_SEQUENCE (1 << 5)
 #endif
 
-static PyObject *clear_id_unicode = NULL;
-static PyObject *copy_id_unicode = NULL;
+// clear_id_unicode and copy_id_unicode are shared with dict_ops, declared in misc_ops.c
 
 PyObject *CPyList_Build(Py_ssize_t len, ...) {
     Py_ssize_t i;
@@ -38,7 +37,7 @@ char CPyList_Clear(PyObject *list) {
     } else {
         if (clear_id_unicode == NULL) {
             _Py_IDENTIFIER(clear);
-            PyObject *clear_id_unicode = _PyUnicode_FromId(&PyId_clear);
+            clear_id_unicode = _PyUnicode_FromId(&PyId_clear);
             if (clear_id_unicode == NULL) {
                 return 0;
             }
@@ -57,7 +56,7 @@ PyObject *CPyList_Copy(PyObject *list) {
     }
     if (copy_id_unicode == NULL) {
         _Py_IDENTIFIER(copy);
-        PyObject *copy_id_unicode = _PyUnicode_FromId(&PyId_copy);
+        copy_id_unicode = _PyUnicode_FromId(&PyId_copy);
         if (copy_id_unicode == NULL) {
             return NULL;
         }

mypyc/lib-rt/misc_ops.c

@@ -6,6 +6,10 @@
 #include <patchlevel.h>
 #include "CPy.h"
 
+PyObject *clear_id_unicode = NULL;
+PyObject *copy_id_unicode = NULL;
+static PyObject *send_id_unicode = NULL;
+
 PyObject *CPy_GetCoro(PyObject *obj)
 {
     // If the type has an __await__ method, call it,
@@ -20,8 +24,6 @@ PyObject *CPy_GetCoro(PyObject *obj)
     }
 }
 
-static PyObject *send_id_unicode = NULL;
-
 PyObject *CPyIter_Send(PyObject *iter, PyObject *val)
 {
     // Do a send, or a next if second arg is None.
@@ -31,7 +33,7 @@ PyObject *CPyIter_Send(PyObject *iter, PyObject *val)
     } else {
         if (send_id_unicode == NULL) {
             _Py_IDENTIFIER(send);
-            PyObject *send_id_unicode = _PyUnicode_FromId(&PyId_send); /* borrowed */
+            send_id_unicode = _PyUnicode_FromId(&PyId_send); /* borrowed */
             if (send_id_unicode == NULL) {
                 return NULL;
             }