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: MI failures related to string printing


 > > If your just talking about the one FAIL in mi-var-child.exp, why not
 > > just mark it as an XFAIL?  I see that the other XFAIL actually
 > > passes (for me, at least).
 > 
 > The very fact that we're even having this discussion means that you
 > did something wrong Nick; you either didn't check for regressions or
 > ignored them.

Neither, it doesn't fail for me.  Presumably this has something to do with
the compiler/OS and where the variable values are stored.

 > Turning a regression test from a PASS into a FAIL, means you've
 > changed behaviour.  Now that change could be intentional, but then you
 > should have said so when you submitted the patch, and you should have
 > adjusted the test.

The nature of the patch was discussed at length and if there was no change in
behaviour then the patch would be pointless.  If the test had failed for me, I
would have tried to adjust it.  When is it appropriate to use an XFAIL, if
not here?

 > I'm still not convinced the change is ok.  Having 'char *' point to a
 > buffer that's not null-terminated is not uncommon.  We have a lot of
 > those in gdb itself.

There's still time to revert it but I think we should identify a real situation
i.e when GDB is used with a frontend, where it causes a problem first.

-- 
Nick                                           http://www.inet.net.nz/~nickrob


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