[tiny] missing close bracket in texi for add-symbol option

Message ID ffe004e0-e3fe-ba9a-2edb-7228b893c1cf@atom.grundel
State New
Headers show
Series
  • [tiny] missing close bracket in texi for add-symbol option
Related show

Commit Message

Ronald Hoogenboom Feb. 17, 2018, 12:30 p.m.
Now that my --add-symbol feature has made it in the mainstream objcopy, 
this bothers me a bit. I put 1 close bracket ']' too few in the texi 
file. See the patch below (against current master).

This is almost not worth mentioning, but still...

Comments

Nick Clifton Feb. 20, 2018, 1:35 p.m. | #1
Hi Ronald,

> Now that my --add-symbol feature has made it in the mainstream objcopy, this bothers me a bit. I put 1 close bracket ']' too few in the texi file. See the patch below (against current master).

> 

> This is almost not worth mentioning, but still...


No, it is good to make sure that the documentation is accurate.

I have applied your patch and checked it in to the repository.

Cheers
  Nick

Patch

diff --git a/binutils/doc/binutils.texi b/binutils/doc/binutils.texi
index 35f672f..caf7f9d 100644
--- a/binutils/doc/binutils.texi
+++ b/binutils/doc/binutils.texi
@@ -1170,7 +1170,7 @@  objcopy [@option{-F} @var{bfdname}|@option{--target=}@var{bfdname}]
         [@option{--localize-symbols=}@var{filename}]
         [@option{--globalize-symbols=}@var{filename}]
         [@option{--weaken-symbols=}@var{filename}]
-        [@option{--add-symbol} @var{name}=[@var{section}:]@var{value}[,@var{flags}]
+        [@option{--add-symbol} @var{name}=[@var{section}:]@var{value}[,@var{flags}]]
         [@option{--alt-machine-code=}@var{index}]
         [@option{--prefix-symbols=}@var{string}]
         [@option{--prefix-sections=}@var{string}]