public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [PATCH 1/2] Add objfile.username: improve docs for Objfile.filename
@ 2015-01-20 23:03 Doug Evans
  2015-01-21  3:41 ` Eli Zaretskii
  0 siblings, 1 reply; 3+ messages in thread
From: Doug Evans @ 2015-01-20 23:03 UTC (permalink / raw)
  To: gdb-patches, eliz

Hi.

I have a need to expose objfile->original_name to python.
In order to do that I need to tweak the docs for Objfile.filename
to distinguish how it's different.

2015-01-20  Doug Evans  <dje@google.com>

	* python.texi (Objfiles In Python) <Objfile.filename>: Improve docs.
	* guile.texi (Objfiles In Guile) <objfile-filename>: Improve docs.

diff --git a/gdb/doc/guile.texi b/gdb/doc/guile.texi
index 8355d9f..53e69f2 100644
--- a/gdb/doc/guile.texi
+++ b/gdb/doc/guile.texi
@@ -2246,7 +2246,8 @@ if it is invalid at the time the procedure is called.
 @end deffn
 
 @deffn {Scheme Procedure} objfile-filename objfile
-Return the file name of @var{objfile} as a string.
+Return the file name of @var{objfile} as a string,
+with symbolic links resolved.
 @end deffn
 
 @deffn {Scheme Procedure} objfile-pretty-printers objfile
diff --git a/gdb/doc/python.texi b/gdb/doc/python.texi
index b9a50d0..ba03841 100644
--- a/gdb/doc/python.texi
+++ b/gdb/doc/python.texi
@@ -3513,7 +3513,10 @@ Each objfile is represented by an instance of the @code{gdb.Objfile}
 class.
 
 @defvar Objfile.filename
-The file name of the objfile as a string.
+The file name of the objfile as a string, with symbolic links resolved.
+
+The value is @code{None} if the objfile is no longer valid.
+See the @code{gdb.Objfile.is_valid} method, described below.
 @end defvar
 
 @defvar Objfile.owner

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/2] Add objfile.username: improve docs for Objfile.filename
  2015-01-20 23:03 [PATCH 1/2] Add objfile.username: improve docs for Objfile.filename Doug Evans
@ 2015-01-21  3:41 ` Eli Zaretskii
  2015-01-27 18:33   ` Doug Evans
  0 siblings, 1 reply; 3+ messages in thread
From: Eli Zaretskii @ 2015-01-21  3:41 UTC (permalink / raw)
  To: Doug Evans; +Cc: gdb-patches

> From: Doug Evans <dje@google.com>
> Date: Tue, 20 Jan 2015 15:03:06 -0800
> 
> I have a need to expose objfile->original_name to python.
> In order to do that I need to tweak the docs for Objfile.filename
> to distinguish how it's different.
> 
> 2015-01-20  Doug Evans  <dje@google.com>
> 
> 	* python.texi (Objfiles In Python) <Objfile.filename>: Improve docs.
> 	* guile.texi (Objfiles In Guile) <objfile-filename>: Improve docs.

This is OK, thanks.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/2] Add objfile.username: improve docs for Objfile.filename
  2015-01-21  3:41 ` Eli Zaretskii
@ 2015-01-27 18:33   ` Doug Evans
  0 siblings, 0 replies; 3+ messages in thread
From: Doug Evans @ 2015-01-27 18:33 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: gdb-patches

On Tue, Jan 20, 2015 at 7:41 PM, Eli Zaretskii <eliz@gnu.org> wrote:
>> From: Doug Evans <dje@google.com>
>> Date: Tue, 20 Jan 2015 15:03:06 -0800
>>
>> I have a need to expose objfile->original_name to python.
>> In order to do that I need to tweak the docs for Objfile.filename
>> to distinguish how it's different.
>>
>> 2015-01-20  Doug Evans  <dje@google.com>
>>
>>       * python.texi (Objfiles In Python) <Objfile.filename>: Improve docs.
>>       * guile.texi (Objfiles In Guile) <objfile-filename>: Improve docs.
>
> This is OK, thanks.

Committed, thanks.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-01-27 18:05 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-20 23:03 [PATCH 1/2] Add objfile.username: improve docs for Objfile.filename Doug Evans
2015-01-21  3:41 ` Eli Zaretskii
2015-01-27 18:33   ` Doug Evans

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).