On 7/18/21 1:21 AM, Peter Maydell wrote: > Currently the linux-user qemu.h pulls in gdbstub.h. There's no real reason > why it should do this; include it directly from the C files which require > it, and drop the include line in qemu.h. > > (Note that several of the C files previously relying on this indirect > include were going out of their way to only include gdbstub.h conditionally > on not CONFIG_USER_ONLY!) > > Signed-off-by: Peter Maydell <peter.mayd...@linaro.org> > --- > linux-user/qemu.h | 1 - > gdbstub.c | 2 +- > linux-user/exit.c | 1 + > linux-user/main.c | 1 + > linux-user/signal.c | 2 ++ > semihosting/arm-compat-semi.c | 2 +- > target/m68k/m68k-semi.c | 2 +- > target/nios2/nios2-semi.c | 2 +- > 8 files changed, 8 insertions(+), 5 deletions(-)
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>