From: Agustin Martin <agustin.martin@hispalinux.es>
To: 12768@debbugs.gnu.org, rrt@sc3d.org
Subject: bug#12768: 24.1; flyspell highlights words which ispell accepts
Date: Fri, 2 Nov 2012 02:19:51 +0100 [thread overview]
Message-ID: <CAKy3oZpxp4KO6sqq6LROqJGOYrpAbeV3miD38hY+7L-nFoEsmA@mail.gmail.com> (raw)
In-Reply-To: <83625p758u.fsf@gnu.org>
2012/11/1 Eli Zaretskii <eliz@gnu.org>:
>> From: Agustin Martin <agustin.martin@hispalinux.es>
>>
>> Really strange. With your steps I can even reproduce the problem with
>> something as simple as
>>
>> dnl -- D
>>
>> If I add something before dnl I still get the problem.
>>
>> sdfdsfsdnl -- D
>>
>> IIf I either break the double dash (no double dashes at all) or the
>> dnl or remove the "D" after the dashes problem disappear
>>
>> dfnl -- D
>> dnl - - D
>> dnl --
>>
>> autotest-mode.el is a small file, but II do not see any hint about
>> what is happening here. Seems to do something strange with double
>> dashes associated with dnl, but I am absolutely clueless, help
>> welcome.
>
> If you step through ispell-region, you will see that it goes bananas
> because ispell-skip-region-alist has "--+" as part of it, which
> matches the dashes between "dnl" and the text of the comment. What
> ispell-region does is remove the dashes, and then _prepend_ "dnl"
> (because it's a comment-start string). The result is that we send
> something like "^dnl FOO" to the speller, and the character positions
> get screwed up, because the characters between "dnl" and "FOO" were
> removed.
Thanks a lot for debugging. I only had time to note that as you point
put more that one dash is removed causing the misalignment. Was
currently looking at the wrong site (autoconf.el).
> FWIW, I really don't understand why ispell.el removes patterns that
> pertain to email messages from buffers that have nothing to do with
> mail-mode or its derivatives.
No idea. I never looked before into ispell-region and friends for
anything than trying to have a fallback behavior on misalignments, and
that was beyond my skills.
>> Dirty workaround in the meantime, avoid double dashes and try something like
>>
>> dnl - - - - Hi, whatever I want to add here
>>
>> But I really wonder what is happening here.
>
> See above. A single dash doesn't get removed, so the workaround works
> around.
>
> Btw, WIBNI ispell.el had a debug option, which, if set, would collect
> in a special buffer everything we send to the speller and its
> responses. I find myself adding '(message "sent %s" strin)'
> etc. every time I need to debug the communications between ispell.el
> and the speller program. WDYT?
I think you are right. I am also an extensive user of the (message
...) debugger and it is really useful, but needing to re-add things
for every new debug is a pity.
flyspell has some of such debug options sending things to a
flyspell-debug buffer, something in the same line would be useful.
Thanks again for the info. Will try to have a look at ispell-region to
try understanding things better
--
Agustin
next prev parent reply other threads:[~2012-11-02 1:19 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-30 13:23 bug#12768: 24.1; flyspell highlights words which ispell accepts Reuben Thomas
2012-10-31 12:26 ` Agustin Martin
2012-10-31 12:31 ` Agustin Martin
2012-10-31 12:35 ` Reuben Thomas
2012-10-31 14:53 ` Agustin Martin
2012-10-31 15:20 ` Reuben Thomas
2012-10-31 20:17 ` Reuben Thomas
2012-10-31 21:31 ` Agustin Martin
2012-10-31 21:50 ` Reuben Thomas
2012-11-01 0:10 ` Agustin Martin
2012-11-01 0:55 ` Reuben Thomas
2012-11-01 14:44 ` Eli Zaretskii
2012-11-02 1:19 ` Agustin Martin [this message]
[not found] ` <24814310.2560.1351781107917.JavaMail.root@mx1-new.spamfiltro.es>
2012-11-06 17:48 ` Agustin Martin
2012-10-31 16:01 ` Glenn Morris
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAKy3oZpxp4KO6sqq6LROqJGOYrpAbeV3miD38hY+7L-nFoEsmA@mail.gmail.com \
--to=agustin.martin@hispalinux.es \
--cc=12768@debbugs.gnu.org \
--cc=rrt@sc3d.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.