Restore the 'U' command back to "User Edit" and commented-out the

experimental 'U' (View Unread-Only toggle) command. The 'U' command from
the e-mail menu (e.g. email_sec.js) works better than the toggle approach and
is probably all that is needed.
Also, the user number lookup in the User Edit bit is better at finding the
proper user account (e.g. for received netmail).

src/sbbs3/readmail.cpp

@@ -406,9 +406,9 @@ void sbbs_t::readmail(uint usernumber, int which, long lm_mode)
 			bprintf(text[ReadingAllMail],smb.curmsg+1,smb.msgs);
 		else
 			bprintf(text[ReadingMail],smb.curmsg+1,smb.msgs);
-		sprintf(str,"ADFLNQRTU?<>[]{}-+/");
+		sprintf(str,"ADFLNQRT?<>[]{}-+/");
 		if(SYSOP)
-			strcat(str,"C!SPH");
+			strcat(str,"CUSPH");
 		if(which == MAIL_YOUR)
 			strcat(str,"K");	// kill all (visible)
 		else
@@ -781,17 +781,21 @@ void sbbs_t::readmail(uint usernumber, int which, long lm_mode)
 				}
 				smb.curmsg=(u-1);
 				break;
-			case '!':   /* user edit */
+			case 'U':   /* user edit */
 				msg.hdr.number=msg.idx.number;
 				smb_getmsgidx(&smb,&msg);
-				if((unum=(which==MAIL_SENT ? msg.idx.to : msg.idx.from)) == 0)
-					unum=(which==MAIL_SENT ? msg.idx.from : msg.idx.to);
+				unum = msg.idx.from;
+				if(unum == 0)
+					unum = matchuser(&cfg, msg.from, /*sysop_alias: */FALSE);
+				if(unum == 0 && which != MAIL_YOUR)
+					unum = msg.idx.to;
 				if(unum == 0 || unum > lastuser(&cfg)) {
 					bputs(text[UnknownUser]);
 					domsg=false;
 				} else
 					useredit(unum);
 				break;
+#if 0
 			case 'U':	/* View Unread-Only (toggle) */
 			{
 				domsg = false;
@@ -809,6 +813,7 @@ void sbbs_t::readmail(uint usernumber, int which, long lm_mode)
 				CRLF;
 				break;
 			}
+#endif
 			case 'V':	/* View SPAM (toggle) */
 			{
 				domsg = false;