[PATCH] [MAN] 0 and EXIT both work for exit traps.

[PATCH] [MAN] 0 and EXIT both work for exit traps.

[Adam Buchbinder]

I saw a discussion in which there was some confusion over whether or not
you can use a symbolic name, since the manpage doesn't specifically say so.

Signed-off-by: Adam Buchbinder <adam.buchbinder@gmail.com>
---
 ChangeLog  |    4 ++++
 src/dash.1 |    4 +++-
 2 files changed, 7 insertions(+), 1 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index d84d3b7..73aa14f 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2012-12-29  Adam Buchbinder <adam.buchbinder@gmail.com>
+
+	* Clarify that 0 and EXIT are both acceptable for exit traps.	
+
 2012-06-09  Christoph Mathys <eraserix@gmail.com>
 
 	* Add support for ulimit -r.
diff --git a/src/dash.1 b/src/dash.1
index a9cb491..3a1c274 100644
--- a/src/dash.1
+++ b/src/dash.1
@@ -2071,7 +2071,9 @@ The signals are specified by signal number or as the name of the signal.
 If
 .Ar signal
 is
-.Li 0 ,
+.Li 0
+or
+.Li EXIT ,
 the action is executed when the shell exits.
 .Ar action
 may be null, which cause the specified signals to be ignored.
-- 
1.7.7.3

Re: [PATCH] [MAN] 0 and EXIT both work for exit traps.

[Jonathan Nieder]

Adam Buchbinder wrote:

> I saw a discussion in which there was some confusion over whether or not
> you can use a symbolic name, since the manpage doesn't specifically say so.

Looks good to me, with one nit:

[...]
> +	* Clarify that 0 and EXIT are both acceptable for exit traps.	

A trailing tab seems to have snuck in.

How about the following on top (for application with "am --scissors")?

-- >8 --
Subject: [MAN] Clarify "trap '' signals" syntax in manual page

When the manpage states

| <action> may be null, which cause the specified signals to be ignored.

it is not immediately obvious what it means for an action to be
null.  Clarify by explicitly referring to an empty string, as
opposed to a NULL pointer or the string "null".

Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
---
 ChangeLog  | 4 ++++
 src/dash.1 | 4 +++-
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index f39999e9..d6536954 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2012-12-30  Jonathan Nieder  <jrnieder@gmail.com>
+
+	* Clarify "trap '' signals" syntax in manual page.
+
 2012-12-29  Adam Buchbinder <adam.buchbinder@gmail.com>
 
 	* Clarify that 0 and EXIT are both acceptable for exit traps.
diff --git a/src/dash.1 b/src/dash.1
index 3a1c2749..b47364d9 100644
--- a/src/dash.1
+++ b/src/dash.1
@@ -2076,7 +2076,9 @@ or
 .Li EXIT ,
 the action is executed when the shell exits.
 .Ar action
-may be null, which cause the specified signals to be ignored.
+may be empty
+.Li ( "''" ) ,
+which causes the specified signals to be ignored.
 With
 .Ar action
 omitted or set to `-' the specified signals are set to their default action.
-- 
1.8.1.rc3