diff --git a/3rdparty/python-console/modified/pyinterpreter.cc b/3rdparty/python-console/modified/pyinterpreter.cc index 89c9b88caa..cbb6322f49 100644 --- a/3rdparty/python-console/modified/pyinterpreter.cc +++ b/3rdparty/python-console/modified/pyinterpreter.cc @@ -82,19 +82,29 @@ const std::list &pyinterpreter_suggest(const std::string &hint) PyEval_AcquireThread(m_threadState); m_suggestions.clear(); int i = 0; - std::string command = string_format("sys.completer.complete('%s', %d)\n", hint.c_str(), i); + std::string escaped; + for (char c : hint) { + if (c == '\'' || c == '\\') + escaped += '\\'; + escaped += c; + } + std::string command = string_format("sys.completer.complete('%s', %d)\n", escaped.c_str(), i); std::string res; do { PyObject *py_result; PyObject *dum; py_result = Py_CompileString(command.c_str(), "", Py_single_input); + if (py_result == nullptr) + break; dum = PyEval_EvalCode(py_result, glb, loc); + if (dum == nullptr) + break; Py_XDECREF(dum); Py_XDECREF(py_result); res = redirector_take_output(m_threadState); ++i; - command = string_format("sys.completer.complete('%s', %d)\n", hint.c_str(), i); + command = string_format("sys.completer.complete('%s', %d)\n", escaped.c_str(), i); if (res.size()) { // throw away the newline res = res.substr(1, res.size() - 3); diff --git a/common/pycontainers.h b/common/pycontainers.h index 04c670ccf6..2b9ee20820 100644 --- a/common/pycontainers.h +++ b/common/pycontainers.h @@ -285,7 +285,9 @@ template struct map_pair_wrapper { if ((i >= 2) || (i < 0)) KeyError(); - return (i == 1) ? object(value_conv()(x.ctx, x.base.second)) : object(x.base.first); + return (i == 1) ? object(value_conv()(x.ctx, x.base.second)) + : object(PythonConversion::string_converter().to_str(x.ctx, + x.base.first)); } static int len(wrapped_pair &x) { return 2; } @@ -410,7 +412,8 @@ template struct map_pair_wrapper_uptr if ((i >= 2) || (i < 0)) KeyError(); return (i == 1) ? object(PythonConversion::ContextualWrapper(x.ctx, *x.base.second.get())) - : object(x.base.first); + : object(PythonConversion::string_converter().to_str(x.ctx, + x.base.first)); } static int len(wrapped_pair &x) { return 2; }