[PATCH 0/7] Refactor shared code in {i386,amd64}-linux-nat.c
Pedro Alves
palves@redhat.com
Fri Jun 27 10:54:00 GMT 2014
On 06/27/2014 11:39 AM, Pedro Alves wrote:
> The resulting #ifdefery quite reasonable, IMNSHO -- it'd be bad if there
> were lots of different combinations of symbols we'd have to #ifdef on,
> but there's really only a few #ifdef __x86_64__. So if it's
> spaghetti, it's still dry, raw, straight and neatly lined up
> in the package. :-)
Ah, I found one bit where there's more #ifdefery than necessary.
So the spaghetti did catch some moisture. :-)
Gary, please compare your resulting x86_linux_read_description
with gdbserver's x86_linux_read_description. Notice how it
has fewer #ifdef blocks. Could you rework your version of the
function in that direction please?
Thanks,
--
Pedro Alves
More information about the Gdb-patches
mailing list