This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Refactor common/target-common into meaningful bits
- From: Pedro Alves <palves at redhat dot com>
- To: lgustavo at codesourcery dot com
- Cc: "'gdb-patches at sourceware dot org'" <gdb-patches at sourceware dot org>, Tom Tromey <tromey at redhat dot com>
- Date: Mon, 19 Aug 2013 14:45:05 +0100
- Subject: Re: [PATCH] Refactor common/target-common into meaningful bits
- References: <51FA9649 dot 5060008 at codesourcery dot com> <520E3BE9 dot 9010402 at codesourcery dot com>
On 08/16/2013 03:49 PM, Luis Machado wrote:
> This is an updated version of the patch that drops the target-* naming,
> creates a new directory named "target" under src/gdb and renames the
> files target-<something>.* to <something.*.
>
> Based on previous discussions, it seems this is what it should look like
> in the future.
>
> One of the differences now is the inclusion of, for example,
> "target/resume.h" instead of "resume.h". It looks more intuitive this way.
Yes, and the corresponding '#include "wait.h"' would collide with
the system's /usr/include/wait.h.
> -target-common.o: ${srcdir}/common/target-common.c
> - $(COMPILE) $(srcdir)/common/target-common.c
> +target-waitstatus.o: ${srcdir}/target/waitstatus.c
Any reason this isn't waitstatus.o?
> @@ -550,7 +552,7 @@ filestuff.o: ../common/filestuff.c
> agent.o: ../common/agent.c
> $(COMPILE) $<
> $(POSTCOMPILE)
> -target-common.o: ../common/target-common.c
> +target-waitstatus.o: ../target/waitstatus.c
> $(COMPILE) $<
> $(POSTCOMPILE)
Here too.
Otherwise looks fine to me.
Thanks,
--
Pedro Alves