From 16e9019ef738c251ecfbed512c6835f0f0a424f7 Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Tue, 12 Jan 2021 14:19:49 -0500 Subject: [PATCH] gdb: move baud_rate and serial_parity declarations to serial.h They are currently in target.h, it would make more sense to have them in serial.h, since they are defined in serial.c. gdb/ChangeLog: * target.h (baud_rate, serial_parity): Move declarations... * serial.h: ... here. * main.c: Include serial.h. * serial.c (baud_rate, serial_parity): Update doc. Change-Id: Idc983c154c80ccc29b07ce68df3483cefe03fb71 --- gdb/ChangeLog | 7 +++++++ gdb/main.c | 1 + gdb/serial.c | 7 ++----- gdb/serial.h | 8 ++++++++ gdb/target.h | 6 ------ 5 files changed, 18 insertions(+), 11 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 88d605fa1c..42ef34a3b2 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,10 @@ +2021-01-12 Simon Marchi + + * target.h (baud_rate, serial_parity): Move declarations... + * serial.h: ... here. + * main.c: Include serial.h. + * serial.c (baud_rate, serial_parity): Update doc. + 2021-01-12 Simon Marchi * top.c (pre_init_ui_hook): Remove. diff --git a/gdb/main.c b/gdb/main.c index 9e52c41536..1e1fbf2f35 100644 --- a/gdb/main.c +++ b/gdb/main.c @@ -54,6 +54,7 @@ #endif #include "gdbsupport/alt-stack.h" #include "observable.h" +#include "serial.h" /* The selected interpreter. This will be used as a set command variable, so it should always be malloc'ed - since diff --git a/gdb/serial.c b/gdb/serial.c index 4b75178021..1854721d7e 100644 --- a/gdb/serial.c +++ b/gdb/serial.c @@ -623,10 +623,7 @@ serial_pipe (struct serial *scbs[2]) static struct cmd_list_element *serial_set_cmdlist; static struct cmd_list_element *serial_show_cmdlist; -/* Baud rate specified for talking to serial target systems. Default - is left as -1, so targets can choose their own defaults. */ -/* FIXME: This means that "show serial baud" and gr_files_info can - print -1 or (unsigned int)-1. This is a Bad User Interface. */ +/* See serial.h. */ int baud_rate = -1; @@ -638,7 +635,7 @@ serial_baud_show_cmd (struct ui_file *file, int from_tty, value); } -/* Parity for serial port. */ +/* See serial.h. */ int serial_parity = GDBPARITY_NONE; diff --git a/gdb/serial.h b/gdb/serial.h index d9f5d4ba5e..9a800491af 100644 --- a/gdb/serial.h +++ b/gdb/serial.h @@ -36,6 +36,14 @@ typedef void *serial_ttystate; struct serial; struct serial_ops; +/* Speed in bits per second, or -1 which means don't mess with the speed. */ + +extern int baud_rate; + +/* Parity for serial port */ + +extern int serial_parity; + /* Create a new serial for OPS. The new serial is not opened. */ /* Try to open NAME. Returns a new `struct serial *' on success, NULL diff --git a/gdb/target.h b/gdb/target.h index e1a1d7a922..40bf416cd6 100644 --- a/gdb/target.h +++ b/gdb/target.h @@ -2477,12 +2477,6 @@ extern gdb::optional target_get_osdata (const char *type); information (higher values, more information). */ extern int remote_debug; -/* Speed in bits per second, or -1 which means don't mess with the speed. */ -extern int baud_rate; - -/* Parity for serial port */ -extern int serial_parity; - /* Timeout limit for response from target. */ extern int remote_timeout; -- 2.34.1