diff options
author | Pedro Alves <palves@redhat.com> | 2016-10-17 14:11:26 +0100 |
---|---|---|
committer | Pedro Alves <palves@redhat.com> | 2016-10-17 14:39:38 +0100 |
commit | 0b21e0ea7cf8c61cdd5915560aca2037b30ee03a (patch) | |
tree | 90feadd58d39d6d10a852802f9d022eba5839cc9 | |
parent | 74870bba694d434e4ddce8671b39a3bf3935c182 (diff) | |
download | binutils-gdb-0b21e0ea7cf8c61cdd5915560aca2037b30ee03a.tar.gz |
Use ui_file_as_string in gdb/python/
gdb/ChangeLog:
yyyy-mm-yy Pedro Alves <palves@redhat.com>
* python/py-arch.c (archpy_disassemble): Use ui_file_as_string and
std::string.
* python/py-breakpoint.c (bppy_get_commands): Use
ui_file_as_string and std::string.
* python/py-frame.c (frapy_str): Likewise.
* python/py-type.c (typy_str): Likewise.
* python/py-unwind.c (unwind_infopy_str): Likewise.
* python/py-value.c (valpy_str): Likewise.
-rw-r--r-- | gdb/python/py-arch.c | 10 | ||||
-rw-r--r-- | gdb/python/py-breakpoint.c | 8 | ||||
-rw-r--r-- | gdb/python/py-frame.c | 8 | ||||
-rw-r--r-- | gdb/python/py-type.c | 10 | ||||
-rw-r--r-- | gdb/python/py-unwind.c | 7 | ||||
-rw-r--r-- | gdb/python/py-value.c | 7 |
6 files changed, 19 insertions, 31 deletions
diff --git a/gdb/python/py-arch.c b/gdb/python/py-arch.c index 4a2dcbf82ee..60cc5a9dd26 100644 --- a/gdb/python/py-arch.c +++ b/gdb/python/py-arch.c @@ -198,7 +198,6 @@ archpy_disassemble (PyObject *self, PyObject *args, PyObject *kw) || (end_obj == NULL && count_obj == NULL && pc == start);) { int insn_len = 0; - char *as = NULL; struct ui_file *memfile = mem_fileopen (); PyObject *insn_dict = PyDict_New (); @@ -232,18 +231,20 @@ archpy_disassemble (PyObject *self, PyObject *args, PyObject *kw) } END_CATCH - as = ui_file_xstrdup (memfile, NULL); + std::string as = ui_file_as_string (memfile); + if (PyDict_SetItemString (insn_dict, "addr", gdb_py_long_from_ulongest (pc)) || PyDict_SetItemString (insn_dict, "asm", - PyString_FromString (*as ? as : "<unknown>")) + PyString_FromString (!as.empty () + ? as.c_str () + : "<unknown>")) || PyDict_SetItemString (insn_dict, "length", PyInt_FromLong (insn_len))) { Py_DECREF (result_list); ui_file_delete (memfile); - xfree (as); return NULL; } @@ -251,7 +252,6 @@ archpy_disassemble (PyObject *self, PyObject *args, PyObject *kw) pc += insn_len; i++; ui_file_delete (memfile); - xfree (as); } return result_list; diff --git a/gdb/python/py-breakpoint.c b/gdb/python/py-breakpoint.c index 80f5d1f8da0..e61cbcddaf9 100644 --- a/gdb/python/py-breakpoint.c +++ b/gdb/python/py-breakpoint.c @@ -487,9 +487,8 @@ bppy_get_commands (PyObject *self, void *closure) struct breakpoint *bp = self_bp->bp; long length; struct ui_file *string_file; - struct cleanup *chain; PyObject *result; - char *cmdstr; + struct cleanup *chain; BPPY_REQUIRE_VALID (self_bp); @@ -514,9 +513,8 @@ bppy_get_commands (PyObject *self, void *closure) END_CATCH ui_out_redirect (current_uiout, NULL); - cmdstr = ui_file_xstrdup (string_file, &length); - make_cleanup (xfree, cmdstr); - result = host_string_to_python_string (cmdstr); + std::string cmdstr = ui_file_as_string (string_file); + result = host_string_to_python_string (cmdstr.c_str ()); do_cleanups (chain); return result; } diff --git a/gdb/python/py-frame.c b/gdb/python/py-frame.c index 6bdac08579f..a66f88555ff 100644 --- a/gdb/python/py-frame.c +++ b/gdb/python/py-frame.c @@ -80,17 +80,13 @@ frame_object_to_frame_info (PyObject *obj) static PyObject * frapy_str (PyObject *self) { - char *s; PyObject *result; struct ui_file *strfile; strfile = mem_fileopen (); fprint_frame_id (strfile, ((frame_object *) self)->frame_id); - s = ui_file_xstrdup (strfile, NULL); - result = PyString_FromString (s); - xfree (s); - - return result; + std::string s = ui_file_as_string (strfile); + return PyString_FromString (s.c_str ()); } /* Implementation of gdb.Frame.is_valid (self) -> Boolean. diff --git a/gdb/python/py-type.c b/gdb/python/py-type.c index 03cc8d9ecad..da9dadd2fb0 100644 --- a/gdb/python/py-type.c +++ b/gdb/python/py-type.c @@ -1010,8 +1010,7 @@ typy_template_argument (PyObject *self, PyObject *args) static PyObject * typy_str (PyObject *self) { - char *thetype = NULL; - long length = 0; + std::string thetype; PyObject *result; TRY @@ -1025,18 +1024,17 @@ typy_str (PyObject *self) LA_PRINT_TYPE (type_object_to_type (self), "", stb, -1, 0, &type_print_raw_options); - thetype = ui_file_xstrdup (stb, &length); + thetype = ui_file_as_string (stb); do_cleanups (old_chain); } CATCH (except, RETURN_MASK_ALL) { - xfree (thetype); GDB_PY_HANDLE_EXCEPTION (except); } END_CATCH - result = PyUnicode_Decode (thetype, length, host_charset (), NULL); - xfree (thetype); + result = PyUnicode_Decode (thetype.c_str (), thetype.length (), + host_charset (), NULL); return result; } diff --git a/gdb/python/py-unwind.c b/gdb/python/py-unwind.c index cc685aeff30..3963cd07b0f 100644 --- a/gdb/python/py-unwind.c +++ b/gdb/python/py-unwind.c @@ -238,12 +238,9 @@ unwind_infopy_str (PyObject *self) } fprintf_unfiltered (strfile, ")"); } - { - char *s = ui_file_xstrdup (strfile, NULL); - result = PyString_FromString (s); - xfree (s); - } + std::string s = ui_file_as_string (strfile); + result = PyString_FromString (s.c_str ()); ui_file_delete (strfile); return result; } diff --git a/gdb/python/py-value.c b/gdb/python/py-value.c index 46683b818d3..e326005519b 100644 --- a/gdb/python/py-value.c +++ b/gdb/python/py-value.c @@ -895,7 +895,7 @@ valpy_call (PyObject *self, PyObject *args, PyObject *keywords) static PyObject * valpy_str (PyObject *self) { - char *s = NULL; + std::string s; PyObject *result; struct value_print_options opts; @@ -909,7 +909,7 @@ valpy_str (PyObject *self) common_val_print (((value_object *) self)->value, stb, 0, &opts, python_language); - s = ui_file_xstrdup (stb, NULL); + s = ui_file_as_string (stb); do_cleanups (old_chain); } @@ -919,8 +919,7 @@ valpy_str (PyObject *self) } END_CATCH - result = PyUnicode_Decode (s, strlen (s), host_charset (), NULL); - xfree (s); + result = PyUnicode_Decode (s.c_str (), s.length (), host_charset (), NULL); return result; } |