unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Josh Berdine <josh@berdine.net>
To: Lars Ingebrigtsen <larsi@gnus.org>
Cc: 19079@debbugs.gnu.org, Stefan Monnier <monnier@iro.umontreal.ca>
Subject: bug#19079: 24.4; smie mis-highlights unmatchable keywords
Date: Fri, 4 Dec 2020 22:03:23 +0000	[thread overview]
Message-ID: <07166D84-7897-464C-81CB-90A32AC9C823@berdine.net> (raw)
In-Reply-To: <87wnxxlve3.fsf@gnus.org>

> On Dec 4, 2020, at 10:26 AM, Lars Ingebrigtsen <larsi@gnus.org> wrote:
> 
> Josh Berdine <josh@berdine.net> writes:
> 
>> emacs -Q
>> M-x load-file
>> ~/.opam/4.02.1/share/tuareg/tuareg.el
>> M-x tuareg-mode
>> M-x show-paren-mode
>> let f () =
>>  let x = 0 in
>>  ()
>> C-u 17 C-b
>> 
>> This starts a fresh emacs, loads the tuareg mode, enables
>> show-paren-mode, enters some ocaml code, and puts the point on a 'let'
>> keyword.  This 'let' is highlighted in 'show-paren-mismatch-face'.  As
>> there is no way for this 'let' to be matched (as far as smie is
>> concerned) this 'let' should not be highlighted.
> 
> (This bug report unfortunately got no response at the time.)
> 
> Stefan M says:
> 
>> And sorry for overlooking the fact that "let" gets highlighted as unmatched; that's  
>> indeed an error. Could you report it via "M-x report-emacs-bug" since it's a bug in  
>> smie.el rather than in Tuareg?  
> 
> This was six years ago.  Is this still an issue in more recent versions
> of Emacs?

Thanks, yes, it still behaves the same way. I have just redone the repro with 27.1:

emacs -nw -Q
M-x load-file
.emacs.d/elpa/tuareg-20200518.1820/tuareg-opam.el
M-x load-file
.emacs.d/elpa/tuareg-20200518.1820/tuareg.el
M-x tuareg-mode
M-x show-paren-mode
let f () =
 let x = 0 in
 ()
C-u 17 C-b


In case it is useful, here is the current output from report-emacs-bug:


In GNU Emacs 27.1 (build 1, x86_64-apple-darwin19.5.0, NS appkit-1894.50 Version 10.15.5 (Build 19F101))
 of 2020-11-14 built on d12frosted.local
System Description:  Mac OS X 10.15.7

Recent messages:
For information about GNU Emacs and the GNU system, type C-h C-a.
Loading /Users/jjb/.emacs.d/elpa/tuareg-20200518.1820/tuareg-opam.el (source)...done
Loading /Users/jjb/.emacs.d/elpa/tuareg-20200518.1820/tuareg.el (source)...done
Show-Paren mode enabled
Mark set

Configured using:
 'configure --disable-dependency-tracking --disable-silent-rules
 --enable-locallisppath=/usr/local/share/emacs/site-lisp
 --infodir=/usr/local/Cellar/emacs-plus@27/27.1/share/info/emacs
 --prefix=/usr/local/Cellar/emacs-plus@27/27.1 --with-xml2 --with-gnutls
 --without-dbus --with-imagemagick --with-modules --with-rsvg --with-ns
 --disable-ns-self-contained'

Configured features:
RSVG IMAGEMAGICK GLIB NOTIFY KQUEUE ACL GNUTLS LIBXML2 ZLIB
TOOLKIT_SCROLL_BARS NS MODULES THREADS JSON PDUMPER LCMS2 GMP

Important settings:
  value of $LANG: en_US.UTF-8
  locale-coding-system: utf-8-unix

Major mode: Tuareg

Minor modes in effect:
  show-paren-mode: t
  tooltip-mode: t
  global-eldoc-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t
  transient-mark-mode: t

Load-path shadows:
None found.

Features:
(shadow sort mail-extr emacsbug message rmc puny dired dired-loaddefs
format-spec rfc822 mml mml-sec password-cache epa epg epg-config
gnus-util rmail rmail-loaddefs text-property-search time-date subr-x seq
mm-decode mm-bodies mm-encode mail-parse rfc2231 mailabbrev gmm-utils
mailheader sendmail rfc2047 rfc2045 ietf-drums mm-util mail-prsvr
mail-utils cus-start cus-load paren tuareg speedbar sb-image ezimage
dframe cl-macs pcase find-file tuareg-opam derived skeleton flymake-proc
flymake easymenu compile comint ansi-color ring warnings thingatpt
cl-loaddefs cl-lib smie term/xterm xterm byte-opt gv bytecomp
byte-compile cconv tooltip eldoc electric uniquify ediff-hook vc-hooks
lisp-float-type mwheel term/ns-win ns-win ucs-normalize mule-util
term/common-win tool-bar dnd fontset image regexp-opt fringe
tabulated-list replace newcomment text-mode elisp-mode lisp-mode
prog-mode register page tab-bar menu-bar rfn-eshadow isearch timer
select scroll-bar mouse jit-lock font-lock syntax facemenu font-core
term/tty-colors frame minibuffer cl-generic cham georgian utf-8-lang
misc-lang vietnamese tibetan thai tai-viet lao korean japanese eucjp-ms
cp51932 hebrew greek romanian slovak czech european ethiopic indian
cyrillic chinese composite charscript charprop case-table epa-hook
jka-cmpr-hook help simple abbrev obarray cl-preloaded nadvice loaddefs
button faces cus-face macroexp files text-properties overlay sha1 md5
base64 format env code-pages mule custom widget hashtable-print-readable
backquote threads kqueue cocoa ns lcms2 multi-tty make-network-process
emacs)

Memory information:
((conses 16 86470 4933)
 (symbols 48 9347 1)
 (strings 32 24410 1630)
 (string-bytes 1 762726)
 (vectors 16 10024)
 (vector-slots 8 108386 8922)
 (floats 8 26 495)
 (intervals 56 252 0)
 (buffers 1000 11))






  reply	other threads:[~2020-12-04 22:03 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-17 11:28 bug#19079: 24.4; smie mis-highlights unmatchable keywords Josh Berdine
2020-12-04 10:26 ` Lars Ingebrigtsen
2020-12-04 22:03   ` Josh Berdine [this message]
2020-12-06 13:12     ` Lars Ingebrigtsen

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

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=07166D84-7897-464C-81CB-90A32AC9C823@berdine.net \
    --to=josh@berdine.net \
    --cc=19079@debbugs.gnu.org \
    --cc=larsi@gnus.org \
    --cc=monnier@iro.umontreal.ca \
    /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 public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).