This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA] i386-tdep.c: fix a bug in prec i386 code
- From: Joel Brobecker <brobecker at adacore dot com>
- To: Hui Zhu <teawater at gmail dot com>
- Cc: gdb-patches ml <gdb-patches at sourceware dot org>, Mark Kettenis <mark dot kettenis at xs4all dot nl>
- Date: Thu, 4 Mar 2010 11:38:55 +0400
- Subject: Re: [RFA] i386-tdep.c: fix a bug in prec i386 code
- References: <daef60381003031837r297e9bb2s70889676afb3a51a@mail.gmail.com>
> 2010-03-04 Hui Zhu <teawater@gmail.com>
>
> * i386-tdep.c (i386_process_record): Change "addr" to "tmpu64".
OK.
As an aside, your code needs a really good thorough cleanup. I warned
you already about the use of variables with a meaningless name, and
you'll make this sort of mistake again for as long as you keep using
them. However, my main point is that the use of a giant switch statement
makes your code very hard to read and review. I really suggest that
you create a new file, precord-i386.c where you put your stuff there,
and instead of inlining the code inside each case, you define small
contained procedures for each instruction (or instruction group).
It'll be easier to find the code that handles such and such instruction,
easier to write a ChangeLog entry that tells us more about where the
change was made, and it'll make the switch block actually possible
to read.
--
Joel