bug#2696: 23.0.91; flyspell in TeX cite commands

bug#2696: 23.0.91; flyspell in TeX cite commands

[Juan Fiol]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

When a \cite command is too large, flyspell starts to check it after a given lenght. For
instance, flyspell would not check (CORRECTLY) for:
\cite{Macek1970PRAp235,Shakesh1978PRLp1037,Belkic1979PRp279,Meckbac1981PRAp1793,Garibot1980PRAp572}
but would highlight the following:
\cite{Macek1970PRAp235,Shakesh1978PRLp1037,Belkic1979PRp279,Meckbac1981PRAp1793,Garibot1980PRAp572,Garibot1983PRAp2792}
(it would highlight the last Garibot and PRAp in Garibot1983PRAp2792).
It probably is related to the length Emacs search for regular expresions but I do not know
how to fix it.

Thanks, Juan
- --------------------------------------------------------------------------------------
In GNU Emacs 23.0.91.1 (i686-pc-linux-gnu, GTK+ Version 2.14.7)
 of 2009-02-27 on cabcat26.cnea.gov.ar
Windowing system distributor `The X.Org Foundation', version 11.0.10300000
configured using `configure  '--prefix=/usr' '--host=i686-pc-linux-gnu'
'--mandir=/usr/share/man' '--infodir=/usr/share/info' '--datadir=/usr/share'
'--sysconfdir=/etc' '--localstatedir=/var/lib' '--program-suffix=-emacs-23'
'--infodir=/usr/share/info/emacs-23' '--with-sound' '--with-x'
'--with-toolkit-scroll-bars' '--with-gif' '--with-jpeg' '--with-png' '--with-rsvg'
'--with-tiff' '--with-xpm' '--with-freetype' '--with-xft' '--without-libotf'
'--without-m17n-flt' '--with-x-toolkit=gtk' '--without-hesiod' '--with-kerberos'
'--with-kerberos5' '--with-gpm' '--with-dbus' '--build=i686-pc-linux-gnu'
'build_alias=i686-pc-linux-gnu' 'host_alias=i686-pc-linux-gnu' 'CFLAGS=-march=prescott -O2
- -pipe' 'LDFLAGS=-Wl,-O1''

Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: en_US.UTF-8
  value of $LC_CTYPE: en_US.UTF-8
  value of $LC_MESSAGES: en_US.UTF-8
  value of $LC_MONETARY: en_US.UTF-8
  value of $LC_NUMERIC: en_US.UTF-8
  value of $LC_TIME: en_US.UTF-8
  value of $LANG: en_US.UTF-8
  value of $XMODIFIERS: nil
  locale-coding-system: utf-8-unix
  default-enable-multibyte-characters: t

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkm/yfcACgkQqiWjWCO20uwj0ACdFwiCxDfb5C0AQF0ClzKCqEmL
DMMAnRrqEmYRmu7uYbJDNtg/j+yfOPnm
=+bsK
-----END PGP SIGNATURE-----

bug#2696: 23.0.91; flyspell in TeX cite commands

[martin rudalics]

 > When a \cite command is too large, flyspell starts to check it after a given lenght. For
 > instance, flyspell would not check (CORRECTLY) for:
 > \cite{Macek1970PRAp235,Shakesh1978PRLp1037,Belkic1979PRp279,Meckbac1981PRAp1793,Garibot1980PRAp572}
 > but would highlight the following:
 > \cite{Macek1970PRAp235,Shakesh1978PRLp1037,Belkic1979PRp279,Meckbac1981PRAp1793,Garibot1980PRAp572,Garibot1983PRAp2792}
 > (it would highlight the last Garibot and PRAp in Garibot1983PRAp2792).
 > It probably is related to the length Emacs search for regular expresions but I do not know
 > how to fix it.

`flyspell-tex-command-p' has

	  (and (re-search-backward "\\\\" (- (point) 100) t)

so maybe replacing 100 by some larger value might help.

martin

bug#2696: 23.0.91; flyspell in TeX cite commands

[Juan Fiol]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

martin rudalics wrote:
>> When a \cite command is too large, flyspell starts to check it after a
> given lenght. For
>> instance, flyspell would not check (CORRECTLY) for:
>>
> \cite{Macek1970PRAp235,Shakesh1978PRLp1037,Belkic1979PRp279,Meckbac1981PRAp1793,Garibot1980PRAp572}
> 
>> but would highlight the following:
>>
> \cite{Macek1970PRAp235,Shakesh1978PRLp1037,Belkic1979PRp279,Meckbac1981PRAp1793,Garibot1980PRAp572,Garibot1983PRAp2792}
> 
>> (it would highlight the last Garibot and PRAp in Garibot1983PRAp2792).
>> It probably is related to the length Emacs search for regular
> expresions but I do not know
>> how to fix it.
> 
> `flyspell-tex-command-p' has
> 
>       (and (re-search-backward "\\\\" (- (point) 100) t)
> 
> so maybe replacing 100 by some larger value might help.
> 
> martin
> 
> 
Thanks for the prompt reply,
Martin's suggestion worked. I fixed it and the problem was fixed. How does it
follows now? Will it be corrected for next version of flyspell?
Thanks for your time,
Juan
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAknAXnAACgkQqiWjWCO20uy/LQCfaFw7aU98NLbYSpRtTgHQhWQl
+G8An2JQpNwul8vyQ3LYzeUmx9nMm9NT
=aq26
-----END PGP SIGNATURE-----

bug#2696: 23.0.91; flyspell in TeX cite commands

[martin rudalics]

 >  Will it be corrected for next version of flyspell?

You can't "correct" this in a reasonable way because flyspell inherently
works on the word-like object before `point'.  The current value makes
flyspell already search the last 100 chars for a backslash every time it
investigates such an object.  Hardcoding a larger value here will just
make flyspell even more sloppy for the average user.

But maybe you can convince Agustín Martín to make this a customizable
option ;-)

martin

bug#2696: 23.0.91; flyspell in TeX cite commands

[Lars Magne Ingebrigtsen]

martin rudalics <rudalics@gmx.at> writes:

> You can't "correct" this in a reasonable way because flyspell inherently
> works on the word-like object before `point'.  The current value makes
> flyspell already search the last 100 chars for a backslash every time it
> investigates such an object.  Hardcoding a larger value here will just
> make flyspell even more sloppy for the average user.

I think this is just a limitation of flyspell, so I'm closing this bug
report.

-- 
(domestic pets only, the antidote for overdose, milk.)
  bloggy blog http://lars.ingebrigtsen.no/