This is the mail archive of the gdb-patches@sources.redhat.com 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]

[RFA] xfullpath and new regression test xfullpath.exp - take 2


Hello,

based on Eli's comments, I reworked the patch I posted in the
following message:

     http://sources.redhat.com/ml/gdb-patches/2002-04/msg00000.html

I reran the regression test on x86-linux and observed no new regression.
I am not reattaching the xfullpath.exp testcase, since Andrew already
approved it, but it can be retrieved at the URL above. I also took note
that this test will be located in the new gdb.gdb directory.

The patch is attached. I used the isalpha function, which is part of
ANSI C. I hope this is fine?

ChangeLog entry for gdb (unchanged except for the date):

2002-04-05  J. Brobecker  <brobecker@gnat.com>

        * utils.c (xfullpath): New function.
        * defs.h (xfullpath): Add declaration.
        * source.c (openp): Use xfullpath in place of gdb_realpath to
        avoid resolving the basename part of filenames when the
        associated file is a symbolic link. This fixes a potential
        inconsistency between the filenames known to GDB and the
        filenames it prints in the annotations.
        * symtab.c (lookup_symtab): Use the new xfullpath function, in order
        to be able to match a filename with either the real filename, or
        the name of any symbolic link to this file.
        (lookup_partial_symtab): Ditto.

The ChangeLog for gdb/testsuite:

2002-04-05  J. Brobecker  <brobecker@gnat.com>

        * gdb.gdb/xfullpath.exp: New test, to exercise the new
        xfullpath () function.

OK to commit?
-- 
Joel

Attachment: xfullpath.diff
Description: Text document


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