Path: utzoo!utgpu!jarvis.csri.toronto.edu!mailrus!ncar!ames!lll-winken!uunet!ispi!jbayer From: jbayer@ispi.UUCP (Jonathan Bayer) Newsgroups: comp.mail.elm Subject: Re: Visual Impact Message-ID: <597@ispi.UUCP> Date: 24 Apr 89 14:57:46 GMT References: <5191@nis.mn.org> <5192@nis.mn.org> <5006@pbhyf.PacBell.COM> <645@maxim.ERBE.SE> <970@pkmab.se> <1331@lokkur.UUCP> <1397@sialis.mn.org> Reply-To: jbayer@ispi.UUCP (Jonathan Bayer) Organization: Intelligent Software Products, Inc. Lines: 110 In article <1397@sialis.mn.org> rjg@sialis.mn.org (Robert J. Granvin) writes: >>The brackets mean "what's inside here is a message, what's outside isn't." >>They are a delimiter, not punctuation. So why not break the mold of what >>us computer folks expect and still retain by punctuation by changing >> [Deleting your entire mailbox.] >>to >> [ Deleting your entire mailbox. ] > >>A very small change, but suddenly the message looks like what the >>developers intended -- a tiny box holding a grammaticly correct >>phrase. It no longer looks like a double-punctuated phrase. Note >>the double-space at front, single at back. This is a short patch to modify the bracketed messages. It adds two spaces before the left bracket, and one space before the right bracket. This is an unofficial patch. I have forwarded a copy to Syd Weinstein. JB *** builtin.c- Thu Mar 30 10:35:11 1989 --- builtin.c Mon Apr 24 10:19:44 1989 *************** *** 280,286 **** headers[current-1]->to); sprintf(title1, "%s %d/%d ", ! headers[current-1]->status & DELETED ? "[deleted]" : headers[current-1]->status & FORM_LETTER ? "Form": "Message", current, message_count); t1_len = strlen(title1); --- 280,286 ---- headers[current-1]->to); sprintf(title1, "%s %d/%d ", ! headers[current-1]->status & DELETED ? "[ deleted ]" : headers[current-1]->status & FORM_LETTER ? "Form": "Message", current, message_count); t1_len = strlen(title1); *** leavembox.c- Thu Mar 30 11:10:43 1989 --- leavembox.c Mon Apr 24 10:17:13 1989 *************** *** 306,335 **** if (to_store > 0) { if (to_delete > 0) sprintf(buffer, ! "[Keeping %d message%s, storing %d, and deleting %d.]", to_keep, plural(to_keep), to_store, to_delete); else ! sprintf(buffer, "[Keeping %d message%s and storing %d.]", to_keep, plural(to_keep), to_store); } else { if (to_delete > 0) ! sprintf(buffer, "[Keeping %d message%s and deleting %d.]", to_keep, plural(to_keep), to_delete); else ! sprintf(buffer, "[Keeping %s.]", to_keep > 1 ? "all messages" : "message"); } } else if (to_store > 0) { if (to_delete > 0) ! sprintf(buffer, "[Storing %d message%s and deleting %d.]", to_store, plural(to_store), to_delete); else ! sprintf(buffer, "[Storing %s.]", to_store > 1? "all messages" : "message"); } else { if (to_delete > 0) ! sprintf(buffer, "[Deleting all messages.]"); else buffer[0] = '\0'; } --- 306,335 ---- if (to_store > 0) { if (to_delete > 0) sprintf(buffer, ! "[ Keeping %d message%s, storing %d, and deleting %d. ]", to_keep, plural(to_keep), to_store, to_delete); else ! sprintf(buffer, "[ Keeping %d message%s and storing %d. ]", to_keep, plural(to_keep), to_store); } else { if (to_delete > 0) ! sprintf(buffer, "[ Keeping %d message%s and deleting %d. ]", to_keep, plural(to_keep), to_delete); else ! sprintf(buffer, "[ Keeping %s. ]", to_keep > 1 ? "all messages" : "message"); } } else if (to_store > 0) { if (to_delete > 0) ! sprintf(buffer, "[ Storing %d message%s and deleting %d. ]", to_store, plural(to_store), to_delete); else ! sprintf(buffer, "[ Storing %s. ]", to_store > 1? "all messages" : "message"); } else { if (to_delete > 0) ! sprintf(buffer, "[ Deleting all messages. ]"); else buffer[0] = '\0'; } -- Jonathan Bayer Beware: The light at the end of the Intelligent Software Products, Inc. tunnel may be an oncoming dragon 19 Virginia Ave. ...uunet!ispi!jbayer Rockville Centre, NY 11570 (516) 766-2867 jbayer@ispi.UUCP