[PATCH] Monitor.c (trivial) Remove bedside manner in email text.

[PATCH] Monitor.c (trivial) Remove bedside manner in email text.

[Phil Kauffman]

The words removed are superfluous and add no value to the message.
In addition, they imply that the message is from a person,
which is obviously not the case.
---
 Monitor.c |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/Monitor.c b/Monitor.c
index 5cb24fa..74a88e1 100644
--- a/Monitor.c
+++ b/Monitor.c
@@ -402,14 +402,12 @@ static void alert(char *event, char *dev, char *disc, struct alert_info *info)
 			if (disc && disc[0] == ' ')
 				fprintf(mp, "Extra information:%s.\n\n", disc);
 
-			fprintf(mp, "Faithfully yours, etc.\n");
-
 			mdstat = fopen("/proc/mdstat", "r");
 			if (mdstat) {
 				char buf[8192];
 				int n;
 				fprintf(mp,
-					"\nP.S. The /proc/mdstat file"
+					"\nThe /proc/mdstat file"
 					" currently contains the following:\n\n");
 				while ( (n=fread(buf, 1, sizeof(buf), mdstat)) > 0)
 					n=fwrite(buf, 1, n, mp);
-- 
1.7.9.5

Re: [PATCH] Monitor.c (trivial) Remove bedside manner in email text.

[NeilBrown]

[-- Attachment #1: Type: text/plain, Size: 1447 bytes --]

On Tue,  2 Sep 2014 14:27:48 -0500 Phil Kauffman <kauffman@cs.uchicago.edu>
wrote:

> The words removed are superfluous and add no value to the message.
> In addition, they imply that the message is from a person,
> which is obviously not the case.
> ---
>  Monitor.c |    4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/Monitor.c b/Monitor.c
> index 5cb24fa..74a88e1 100644
> --- a/Monitor.c
> +++ b/Monitor.c
> @@ -402,14 +402,12 @@ static void alert(char *event, char *dev, char *disc, struct alert_info *info)
>  			if (disc && disc[0] == ' ')
>  				fprintf(mp, "Extra information:%s.\n\n", disc);
>  
> -			fprintf(mp, "Faithfully yours, etc.\n");
> -
>  			mdstat = fopen("/proc/mdstat", "r");
>  			if (mdstat) {
>  				char buf[8192];
>  				int n;
>  				fprintf(mp,
> -					"\nP.S. The /proc/mdstat file"
> +					"\nThe /proc/mdstat file"
>  					" currently contains the following:\n\n");
>  				while ( (n=fread(buf, 1, sizeof(buf), mdstat)) > 0)
>  					n=fwrite(buf, 1, n, mp);

"superfluous" - probably.
"no value" - I did receive a comment from someone saying that the "Faithfully
your" line gave them a chuckle when they needed it.

Did/do the words you would like removed actually cause a problem?
I'm inclined to not apply the patch but if you have a case which is stronger
than the one you have made so far, I will certainly listen to it.

Thanks,
NeilBrown

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 828 bytes --]