Fix a FIXME in mi-out.c
authorTom Tromey <tromey@adacore.com>
Tue, 2 Jul 2019 15:13:53 +0000 (09:13 -0600)
committerTom Tromey <tromey@adacore.com>
Mon, 15 Jul 2019 15:31:19 +0000 (09:31 -0600)
This removes a FIXME comment from mi_ui_out::do_field_int, by
replacing a printf with a use of plongest.

2019-07-15  Tom Tromey  <tromey@adacore.com>

* mi/mi-out.c (mi_ui_out::do_field_int): Use plongest.

gdb/ChangeLog
gdb/mi/mi-out.c

index 1b085957682ef31a296c9801af6a1eb0dcd8c1f4..bb2e8d133f10e1713c9f331cba5b422564804f46 100644 (file)
@@ -1,3 +1,7 @@
+2019-07-15  Tom Tromey  <tromey@adacore.com>
+
+       * mi/mi-out.c (mi_ui_out::do_field_int): Use plongest.
+
 2019-07-15  Tom Tromey  <tromey@adacore.com>
 
        * mi/mi-out.h (class mi_ui_out) <do_field_unsigned>: Declare.
index dd99a2b7fb1863bc8471780f0a6aa88fada30263..75e3f0313d89ad05602969559e77f62fb1b351b6 100644 (file)
@@ -98,10 +98,7 @@ void
 mi_ui_out::do_field_int (int fldno, int width, ui_align alignment,
                         const char *fldname, int value)
 {
-  char buffer[20];     /* FIXME: how many chars long a %d can become? */
-
-  xsnprintf (buffer, sizeof (buffer), "%d", value);
-  do_field_string (fldno, width, alignment, fldname, buffer,
+  do_field_string (fldno, width, alignment, fldname, plongest (value),
                   ui_out_style_kind::DEFAULT);
 }