Re: [9front] patch: faces -c remove faces on click

[sirjofri+ml-9front@sirjofri.de]

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

hello again,

I guess this is the final patch version, this time with an updated
'usage' message.

So in total we have:

- The change itself.
- Notes in the man page.
- Updated 'usage' message.

sirjofri
.


[-- Attachment #2: Type: text/plain, Size: 1525 bytes --]

diff -r 73caf2698436 sys/man/1/faces
--- a/sys/man/1/faces	Tue Dec 01 09:23:16 2020 +0100
+++ b/sys/man/1/faces	Fri Dec 04 09:24:34 2020 +0100
@@ -4,7 +4,7 @@
 .SH SYNOPSIS
 .B faces
 [
-.B -ih
+.B -ihc
 ] [
 .B -m
 .I maildir
@@ -86,6 +86,12 @@
 .B -m
 flags may be used to watch multiple mailboxes.
 .PP
+Starting
+.I faces
+with the
+.B -c
+flag allows the user to remove faces with a click with button 1.
+.PP
 The
 .B -h
 flag causes a different, venerable behavior in which
diff -r 73caf2698436 sys/src/cmd/faces/main.c
--- a/sys/src/cmd/faces/main.c	Tue Dec 01 09:23:16 2020 +0100
+++ b/sys/src/cmd/faces/main.c	Fri Dec 04 09:24:34 2020 +0100
@@ -9,6 +9,7 @@
 
 int	history = 0;	/* use old interface, showing history of mailbox rather than current state */
 int	initload = 0;	/* initialize program with contents of mail box */
+int	clickrm = 0;	/* allows removing mail faces by left clicking */
 
 enum
 {
@@ -600,7 +601,7 @@
 		}else{
 			for(i=first; i<last; i++)	/* clear vwhois faces */
 				if(ptinrect(p, facerect(i-first)) 
-				&& strstr(faces[i]->str[Sshow], "/XXXvwhois")){
+				&& (clickrm || strstr(faces[i]->str[Sshow], "/XXXvwhois"))){
 					delface(i);
 					flushimage(display, 1);
 				}
@@ -675,7 +676,7 @@
 void
 usage(void)
 {
-	fprint(2, "usage: faces [-hi] [-m maildir]\n");
+	fprint(2, "usage: faces [-chi] [-m maildir]\n");
 	exits("usage");
 }
 
@@ -695,6 +696,9 @@
 		addmaildir(EARGF(usage()));
 		maildir = nil;
 		break;
+	case 'c':
+		clickrm++;
+		break;
 	default:
 		usage();
 	}ARGEND