This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [rfa] NEWS additions for flash
On Thu, 2006-09-21 at 15:14 -0400, Daniel Jacobowitz wrote:
> On Thu, Sep 21, 2006 at 12:10:56PM -0700, Michael Snyder wrote:
> > On Thu, 2006-09-21 at 10:18 -0400, Daniel Jacobowitz wrote:
> > > This patch adds news entries for the flash patches I've just checked
> > > in. Is this OK?
> > >
> > > The next release is shaping up to be quite an improvement.
> >
> > On a quick scan of your (today's) patch, I don't see any tests.
> > Is this testable?
>
> In theory, you could write some tests for it using 'gdbreplay'. But
> getting gdbreplay to work in the testsuite harness would be quite
> tricky, and it would be complicated by the fact that which registers
> GDB asks for at connection is target dependent; I don't know how to
> write generic remote protocol tests.
>
> I'd love to unit test this stuff. If you have any suggestions, I'm
> all ears.
Let's see -- there's asm-source.exp, which is implemented on an
arch-by-arch basis. There's gdb.base/regs.exp, which was intended
to be done the same way (but apparently was only ever implemented
for one architecture). There's gdb.arch...
Those are just examples -- I know it would be a hassle.
I think asm-source is a very useful test, and I wish there
were others (such as regs.exp) that were done in a similar
arch-by-arch way.