This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]