all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH] Fix misspelling in flyspell.el
@ 2015-11-06 21:16 Keshav Kini
  2015-11-06 21:46 ` Alan Mackenzie
  2015-11-09 20:55 ` [PATCH v1] Reword message " Keshav Kini
  0 siblings, 2 replies; 8+ messages in thread
From: Keshav Kini @ 2015-11-06 21:16 UTC (permalink / raw)
  To: emacs-devel; +Cc: Keshav Kini

* flyspell.el (flyspell-goto-next-error): fix misspelling

Copyright-paperwork-exempt: yes
---
 lisp/textmodes/flyspell.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lisp/textmodes/flyspell.el b/lisp/textmodes/flyspell.el
index 6c4a731..fa3fba6 100644
--- a/lisp/textmodes/flyspell.el
+++ b/lisp/textmodes/flyspell.el
@@ -1673,7 +1673,7 @@ FLYSPELL-BUFFER."
     (setq flyspell-old-buffer-error (current-buffer))
     (goto-char pos)
     (if (= pos max)
-	(message "No more miss-spelled word!"))))
+        (message "No more misspelled words!"))))
 
 ;;*---------------------------------------------------------------------*/
 ;;*    flyspell-overlay-p ...                                           */
-- 
2.6.2




^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH] Fix misspelling in flyspell.el
  2015-11-06 21:16 [PATCH] Fix misspelling in flyspell.el Keshav Kini
@ 2015-11-06 21:46 ` Alan Mackenzie
  2015-11-06 22:29   ` Ashton Kemerling
                     ` (2 more replies)
  2015-11-09 20:55 ` [PATCH v1] Reword message " Keshav Kini
  1 sibling, 3 replies; 8+ messages in thread
From: Alan Mackenzie @ 2015-11-06 21:46 UTC (permalink / raw)
  To: Keshav Kini; +Cc: emacs-devel

Hello, Keshav

On Fri, Nov 06, 2015 at 03:16:23PM -0600, Keshav Kini wrote:
> * flyspell.el (flyspell-goto-next-error): fix misspelling

> Copyright-paperwork-exempt: yes
> ---
>  lisp/textmodes/flyspell.el | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

> diff --git a/lisp/textmodes/flyspell.el b/lisp/textmodes/flyspell.el
> index 6c4a731..fa3fba6 100644
> --- a/lisp/textmodes/flyspell.el
> +++ b/lisp/textmodes/flyspell.el
> @@ -1673,7 +1673,7 @@ FLYSPELL-BUFFER."
>      (setq flyspell-old-buffer-error (current-buffer))
>      (goto-char pos)
>      (if (= pos max)
> -	(message "No more miss-spelled word!"))))
> +        (message "No more misspelled words!"))))
 
>  ;;*---------------------------------------------------------------------*/
>  ;;*    flyspell-overlay-p ...                                           */
> -- 
> 2.6.2

I think there's a good chance that that missssspelling is deliberate, a
little joke, something to irritate people slightly with.  It would be a
shame to correct it.

-- 
Alan Mackenzie (Nuremberg, Germany).



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] Fix misspelling in flyspell.el
  2015-11-06 21:46 ` Alan Mackenzie
@ 2015-11-06 22:29   ` Ashton Kemerling
  2015-11-07  8:41   ` Eli Zaretskii
  2015-11-09 20:53   ` Keshav Kini
  2 siblings, 0 replies; 8+ messages in thread
From: Ashton Kemerling @ 2015-11-06 22:29 UTC (permalink / raw)
  To: emacs-devel

Alan Mackenzie <acm@muc.de> writes:

> Hello, Keshav
>
> On Fri, Nov 06, 2015 at 03:16:23PM -0600, Keshav Kini wrote:
>> * flyspell.el (flyspell-goto-next-error): fix misspelling
>
>> Copyright-paperwork-exempt: yes
>> ---
>>  lisp/textmodes/flyspell.el | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>
>> diff --git a/lisp/textmodes/flyspell.el b/lisp/textmodes/flyspell.el
>> index 6c4a731..fa3fba6 100644
>> --- a/lisp/textmodes/flyspell.el
>> +++ b/lisp/textmodes/flyspell.el
>> @@ -1673,7 +1673,7 @@ FLYSPELL-BUFFER."
>>      (setq flyspell-old-buffer-error (current-buffer))
>>      (goto-char pos)
>>      (if (= pos max)
>> -	(message "No more miss-spelled word!"))))
>> +        (message "No more misspelled words!"))))
>  
>>  ;;*---------------------------------------------------------------------*/
>>  ;;*    flyspell-overlay-p ...                                           */
>> -- 
>> 2.6.2
>
> I think there's a good chance that that missssspelling is deliberate, a
> little joke, something to irritate people slightly with.  It would be a
> shame to correct it.


If anything, it should be left as a testament to why flyspell is
necessary.

--
Ashton



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] Fix misspelling in flyspell.el
  2015-11-06 21:46 ` Alan Mackenzie
  2015-11-06 22:29   ` Ashton Kemerling
@ 2015-11-07  8:41   ` Eli Zaretskii
  2015-11-07  9:01     ` David Kastrup
  2015-11-09 20:53   ` Keshav Kini
  2 siblings, 1 reply; 8+ messages in thread
From: Eli Zaretskii @ 2015-11-07  8:41 UTC (permalink / raw)
  To: Alan Mackenzie; +Cc: keshav.kini, emacs-devel

> Date: Fri, 6 Nov 2015 21:46:30 +0000
> From: Alan Mackenzie <acm@muc.de>
> Cc: emacs-devel@gnu.org
> 
> > -	(message "No more miss-spelled word!"))))
> > +        (message "No more misspelled words!"))))
>  
> >  ;;*---------------------------------------------------------------------*/
> >  ;;*    flyspell-overlay-p ...                                           */
> > -- 
> > 2.6.2
> 
> I think there's a good chance that that missssspelling is deliberate, a
> little joke, something to irritate people slightly with.  It would be a
> shame to correct it.

Given the person who wrote that code, I agree with Alan.



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] Fix misspelling in flyspell.el
  2015-11-07  8:41   ` Eli Zaretskii
@ 2015-11-07  9:01     ` David Kastrup
  0 siblings, 0 replies; 8+ messages in thread
From: David Kastrup @ 2015-11-07  9:01 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: Alan Mackenzie, keshav.kini, emacs-devel

Eli Zaretskii <eliz@gnu.org> writes:

>> Date: Fri, 6 Nov 2015 21:46:30 +0000
>> From: Alan Mackenzie <acm@muc.de>
>> Cc: emacs-devel@gnu.org
>> 
>> > -	(message "No more miss-spelled word!"))))
>> > +        (message "No more misspelled words!"))))
>>  
>> >  ;;*---------------------------------------------------------------------*/
>> >  ;;*    flyspell-overlay-p ...                                           */
>> > -- 
>> > 2.6.2
>> 
>> I think there's a good chance that that missssspelling is deliberate, a
>> little joke, something to irritate people slightly with.  It would be a
>> shame to correct it.
>
> Given the person who wrote that code, I agree with Alan.

Since flyspell isn't a grammar checker, I'd still advocate replacing
"word" with "words" in the message.  "No more ... word!" seems like
stretching the concept of the joke.

-- 
David Kastrup



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] Fix misspelling in flyspell.el
  2015-11-06 21:46 ` Alan Mackenzie
  2015-11-06 22:29   ` Ashton Kemerling
  2015-11-07  8:41   ` Eli Zaretskii
@ 2015-11-09 20:53   ` Keshav Kini
  2015-11-09 21:52     ` Richard Stallman
  2 siblings, 1 reply; 8+ messages in thread
From: Keshav Kini @ 2015-11-09 20:53 UTC (permalink / raw)
  To: Alan Mackenzie; +Cc: emacs-devel

On 11/06/2015 03:46 PM, Alan Mackenzie wrote:
> I think there's a good chance that that missssspelling is deliberate, a
> little joke, something to irritate people slightly with.  It would be a
> shame to correct it.

Hi Alan (and others),

Oops, it didn't cross my mind that it could be a joke!  I think David is
right, though.  As the message stands, it might not be that obvious that
it's a joke -- indeed, I'm living witness to that fact :)  I mistook it
for just an error message written by a non-native English speaker, and
didn't think to run a `git blame` to find out who wrote it.

I'll reroll the patch with a different message that preserves the joke
but makes it more obvious that it's a joke.  That way I'll hopefully
prevent anyone else from writing similar patches in the future ;)

Thanks,
    Keshav



^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH v1] Reword message in flyspell.el
  2015-11-06 21:16 [PATCH] Fix misspelling in flyspell.el Keshav Kini
  2015-11-06 21:46 ` Alan Mackenzie
@ 2015-11-09 20:55 ` Keshav Kini
  1 sibling, 0 replies; 8+ messages in thread
From: Keshav Kini @ 2015-11-09 20:55 UTC (permalink / raw)
  To: emacs-devel; +Cc: Keshav Kini

It wasn't abundantly clear that the message "No more miss-spelled word!"
was supposed to be a joke, so I tweaked it a bit.

* flyspell.el (flyspell-goto-next-error): reword message

Copyright-paperwork-exempt: yes
---
 lisp/textmodes/flyspell.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lisp/textmodes/flyspell.el b/lisp/textmodes/flyspell.el
index 6c4a731..a8a36db 100644
--- a/lisp/textmodes/flyspell.el
+++ b/lisp/textmodes/flyspell.el
@@ -1673,7 +1673,7 @@ FLYSPELL-BUFFER."
     (setq flyspell-old-buffer-error (current-buffer))
     (goto-char pos)
     (if (= pos max)
-	(message "No more miss-spelled word!"))))
+        (message "No more missspelled words! ;-)"))))
 
 ;;*---------------------------------------------------------------------*/
 ;;*    flyspell-overlay-p ...                                           */
-- 
2.6.2




^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH] Fix misspelling in flyspell.el
  2015-11-09 20:53   ` Keshav Kini
@ 2015-11-09 21:52     ` Richard Stallman
  0 siblings, 0 replies; 8+ messages in thread
From: Richard Stallman @ 2015-11-09 21:52 UTC (permalink / raw)
  To: Keshav Kini; +Cc: acm, emacs-devel

[[[ To any NSA and FBI agents reading my email: please consider    ]]]
[[[ whether defending the US Constitution against all enemies,     ]]]
[[[ foreign or domestic, requires you to follow Snowden's example. ]]]

  > Oops, it didn't cross my mind that it could be a joke!  I think David is
  > right, though.

We will cast you into Misspelheim for this!

-- 
Dr Richard Stallman
President, Free Software Foundation (gnu.org, fsf.org)
Internet Hall-of-Famer (internethalloffame.org)
Skype: No way! See stallman.org/skype.html.




^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2015-11-09 21:52 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-11-06 21:16 [PATCH] Fix misspelling in flyspell.el Keshav Kini
2015-11-06 21:46 ` Alan Mackenzie
2015-11-06 22:29   ` Ashton Kemerling
2015-11-07  8:41   ` Eli Zaretskii
2015-11-07  9:01     ` David Kastrup
2015-11-09 20:53   ` Keshav Kini
2015-11-09 21:52     ` Richard Stallman
2015-11-09 20:55 ` [PATCH v1] Reword message " Keshav Kini

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.