mirror of
				https://github.com/python/cpython.git
				synced 2025-10-30 21:21:22 +00:00 
			
		
		
		
	gh-116417: Fix make check-c-globals for _testlimitedcapi (#116570)
* Remove unused '_testcapimodule' global in Modules/_testcapi/unicode.c. * Update c-analyzer to not use the internal C API in _testlimitedcapi.c.
This commit is contained in:
		
							parent
							
								
									729bfb3105
								
							
						
					
					
						commit
						c5fa796619
					
				
					 3 changed files with 1 additions and 5 deletions
				
			
		|  | @ -3,8 +3,6 @@ | |||
| #include "parts.h" | ||||
| #include "util.h" | ||||
| 
 | ||||
| static struct PyModuleDef *_testcapimodule = NULL;  // set at initialization
 | ||||
| 
 | ||||
| static PyObject * | ||||
| codec_incrementalencoder(PyObject *self, PyObject *args) | ||||
| { | ||||
|  | @ -2098,8 +2096,6 @@ static PyMethodDef TestMethods[] = { | |||
| 
 | ||||
| int | ||||
| _PyTestCapi_Init_Unicode(PyObject *m) { | ||||
|     _testcapimodule = PyModule_GetDef(m); | ||||
| 
 | ||||
|     if (PyModule_AddFunctions(m, TestMethods) < 0) { | ||||
|         return -1; | ||||
|     } | ||||
|  |  | |||
|  | @ -7,6 +7,7 @@ | |||
| FILES_WITHOUT_INTERNAL_CAPI = frozenset(( | ||||
|     # Modules/ | ||||
|     '_testcapimodule.c', | ||||
|     '_testlimitedcapi.c', | ||||
|     '_testclinic_limited.c', | ||||
|     'xxlimited.c', | ||||
|     'xxlimited_35.c', | ||||
|  |  | |||
|  | @ -444,7 +444,6 @@ Modules/_testcapi/heaptype.c	-	_testcapimodule	- | |||
| Modules/_testcapi/mem.c	-	FmData	- | ||||
| Modules/_testcapi/mem.c	-	FmHook	- | ||||
| Modules/_testcapi/structmember.c	-	test_structmembersType_OldAPI	- | ||||
| Modules/_testcapi/unicode.c	-	_testcapimodule	- | ||||
| Modules/_testcapi/watchers.c	-	g_dict_watch_events	- | ||||
| Modules/_testcapi/watchers.c	-	g_dict_watchers_installed	- | ||||
| Modules/_testcapi/watchers.c	-	g_type_modified_events	- | ||||
|  |  | |||
| Can't render this file because it has a wrong number of fields in line 4. | 
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue
	
	 Victor Stinner
						Victor Stinner