[Python-Dev] Adding insint() function

Andrew Kuchling akuchlin@mems-exchange.org
Fri, 18 Aug 2000 11:00:37 -0400


Four modules define insint() functions to insert an integer into a
dictionary in order to initialize constants in their module
dictionaries:

kronos Modules>grep -l insint *.c
pcremodule.c
shamodule.c
socketmodule.c
zlibmodule.c
kronos Modules>          

(Hm... I was involved with 3 of them...)  Other modules don't use a
helper function, but just do PyDict_SetItemString(d, "foo",
PyInt_FromLong(...)) directly.  

This duplication bugs me.  Shall I submit a patch to add an API
convenience function to do this, and change the modules to use it?
Suggested prototype and name: PyDict_InsertInteger( dict *, string,
long)

--amk