[Python-checkins] python/dist/src/Modules cPickle.c,2.120,2.121

gvanrossum@users.sourceforge.net gvanrossum@users.sourceforge.net
Mon, 03 Feb 2003 17:54:52 -0800


Update of /cvsroot/python/python/dist/src/Modules
In directory sc8-pr-cvs1:/tmp/cvs-serv3141/Modules

Modified Files:
	cPickle.c 
Log Message:
Rename the extension registry variables to have leading underscores --
this clarifies that they are part of an internal API (albeit shared
between pickle.py, copy_reg.py and cPickle.c).

I'd like to do the same for copy_reg.dispatch_table, but worry that it
might be used by existing code.  This risk doesn't exist for the
extension registry.


Index: cPickle.c
===================================================================
RCS file: /cvsroot/python/python/dist/src/Modules/cPickle.c,v
retrieving revision 2.120
retrieving revision 2.121
diff -C2 -d -r2.120 -r2.121
*** cPickle.c	4 Feb 2003 00:30:46 -0000	2.120
--- cPickle.c	4 Feb 2003 01:54:49 -0000	2.121
***************
*** 103,111 ****
  
  /* For EXT[124] opcodes. */
! /* copy_reg.extension_registry, {(module_name, function_name): code} */
  static PyObject *extension_registry;
! /* copy_reg.inverted_registry, {code: (module_name, function_name)} */
  static PyObject *inverted_registry;
! /* copy_reg.extension_cache, {code: object} */
  static PyObject *extension_cache;
  
--- 103,111 ----
  
  /* For EXT[124] opcodes. */
! /* copy_reg._extension_registry, {(module_name, function_name): code} */
  static PyObject *extension_registry;
! /* copy_reg._inverted_registry, {code: (module_name, function_name)} */
  static PyObject *inverted_registry;
! /* copy_reg._extension_cache, {code: object} */
  static PyObject *extension_cache;
  
***************
*** 5093,5105 ****
  
  	extension_registry = PyObject_GetAttrString(copy_reg,
! 				"extension_registry");
  	if (!extension_registry) return -1;
  
  	inverted_registry = PyObject_GetAttrString(copy_reg,
! 				"inverted_registry");
  	if (!inverted_registry) return -1;
  
  	extension_cache = PyObject_GetAttrString(copy_reg,
! 				"extension_cache");
  	if (!extension_cache) return -1;
  
--- 5093,5105 ----
  
  	extension_registry = PyObject_GetAttrString(copy_reg,
! 				"_extension_registry");
  	if (!extension_registry) return -1;
  
  	inverted_registry = PyObject_GetAttrString(copy_reg,
! 				"_inverted_registry");
  	if (!inverted_registry) return -1;
  
  	extension_cache = PyObject_GetAttrString(copy_reg,
! 				"_extension_cache");
  	if (!extension_cache) return -1;