Issue #17289: The readline module now plays nicer with external modules or applications changing the rl_completer_word_break_characters global variable.

Initial patch by Bradley Froehle.
This commit is contained in:
Antoine Pitrou 2013-05-06 21:51:03 +02:00
parent 002a77d2f7
commit a7f7deb6ed
2 changed files with 22 additions and 7 deletions

View File

@ -47,6 +47,10 @@ Core and Builtins
Library
-------
- Issue #17289: The readline module now plays nicer with external modules
or applications changing the rl_completer_word_break_characters global
variable. Initial patch by Bradley Froehle.
- Issue #12181: select module: Fix struct kevent definition on OpenBSD 64-bit
platforms. Patch by Federico Schwindt.

View File

@ -70,6 +70,10 @@ on_completion_display_matches_hook(char **matches,
int num_matches, int max_length);
#endif
/* Memory allocated for rl_completer_word_break_characters
(see issue #17289 for the motivation). */
static char *completer_word_break_characters;
/* Exported function to send one line to readline's init file parser */
static PyObject *
@ -368,12 +372,20 @@ set_completer_delims(PyObject *self, PyObject *args)
{
char *break_chars;
if(!PyArg_ParseTuple(args, "s:set_completer_delims", &break_chars)) {
if (!PyArg_ParseTuple(args, "s:set_completer_delims", &break_chars)) {
return NULL;
}
free((void*)rl_completer_word_break_characters);
rl_completer_word_break_characters = strdup(break_chars);
Py_RETURN_NONE;
/* Keep a reference to the allocated memory in the module state in case
some other module modifies rl_completer_word_break_characters
(see issue #17289). */
free(completer_word_break_characters);
completer_word_break_characters = strdup(break_chars);
if (completer_word_break_characters) {
rl_completer_word_break_characters = completer_word_break_characters;
Py_RETURN_NONE;
}
else
return PyErr_NoMemory();
}
PyDoc_STRVAR(doc_set_completer_delims,
@ -918,7 +930,8 @@ setup_readline(void)
/* Set our completion function */
rl_attempted_completion_function = (CPPFunction *)flex_complete;
/* Set Python word break characters */
rl_completer_word_break_characters =
completer_word_break_characters =
rl_completer_word_break_characters =
strdup(" \t\n`~!@#$%^&*()-=+[{]}\\|;:'\",<>/?");
/* All nonalphanums except '.' */
@ -1174,8 +1187,6 @@ PyInit_readline(void)
if (m == NULL)
return NULL;
PyOS_ReadlineFunctionPointer = call_readline;
setup_readline();
return m;