[patch, doc] add-symbol-file

[patch, doc] add-symbol-file

[Yao Qi]

In GDB, when using add-symbol-file with `-s' option, `address' is still
needed, but `address' is missing corresponding documentation.  Beside
this, there should be a space after `-s'.  This patch is to address
them.  OK for mainline?

-- 
Yao (齐尧) 

gdb/doc:

2011-09-28  Yao Qi  <yao@codesourcery.com>

        * gdb.texinfo (Files): Update options for `add-symbol-file'.
	Add one space after option `-s'.
---
 gdb/doc/gdb.texinfo |    4 ++--

diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo
index 051377d..75ac160 100644
--- a/gdb/doc/gdb.texinfo
+++ b/gdb/doc/gdb.texinfo
@@ -14817,14 +14817,14 @@ the program is running.  To do this, use the @code{kill} command
 @cindex dynamic linking
 @item add-symbol-file @var{filename} @var{address}
 @itemx add-symbol-file @var{filename} @var{address} @r{[} -readnow @r{]}
-@itemx add-symbol-file @var{filename} @r{-s}@var{section} @var{address} @dots{}
+@itemx add-symbol-file @var{filename} @var{address} @r{-s} @var{section} @var{address} @dots{}
 The @code{add-symbol-file} command reads additional symbol table
 information from the file @var{filename}.  You would use this command
 when @var{filename} has been dynamically loaded (by some other means)
 into the program that is running.  @var{address} should be the memory
 address at which the file has been loaded; @value{GDBN} cannot figure
 this out for itself.  You can additionally specify an arbitrary number
-of @samp{@r{-s}@var{section} @var{address}} pairs, to give an explicit
+of @samp{@r{-s} @var{section} @var{address}} pairs, to give an explicit
 section name and base address for that section.  You can specify any
 @var{address} as an expression.
 

Re: [patch, doc] add-symbol-file

[Eli Zaretskii]

> Date: Wed, 28 Sep 2011 15:48:43 +0800
> From: Yao Qi <yao@codesourcery.com>
> 
> In GDB, when using add-symbol-file with `-s' option, `address' is still
> needed, but `address' is missing corresponding documentation.  Beside
> this, there should be a space after `-s'.  This patch is to address
> them.  OK for mainline?

Yes, but please lose all the @r{} markup in the lines you modify, it
is not needed there.

Thanks.

[committed]: [patch, doc] add-symbol-file

[Yao Qi]

On 09/28/2011 04:26 PM, Eli Zaretskii wrote:
> Yes, but please lose all the @r{} markup in the lines you modify, it
> is not needed there.

@r{} markups are moved, and committed.

http://sourceware.org/ml/gdb-cvs/2011-09/msg00160.html

-- 
Yao (齐尧)