unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#20800: perl mode reads into comments too deeply
@ 2015-06-13  6:23 積丹尼 Dan Jacobson
  2015-06-16  1:03 ` Stefan Monnier
  0 siblings, 1 reply; 2+ messages in thread
From: 積丹尼 Dan Jacobson @ 2015-06-13  6:23 UTC (permalink / raw)
  To: 20800

For this snippet of code perl mode gets the colors wrong:

return 'W' if               #/^Not Available on Mobile/m;    #W=Web only
    /This video is not available on mobile devices./m;
#After the m all is the wrong color.





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

* bug#20800: perl mode reads into comments too deeply
  2015-06-13  6:23 bug#20800: perl mode reads into comments too deeply 積丹尼 Dan Jacobson
@ 2015-06-16  1:03 ` Stefan Monnier
  0 siblings, 0 replies; 2+ messages in thread
From: Stefan Monnier @ 2015-06-16  1:03 UTC (permalink / raw)
  To: 積丹尼 Dan Jacobson; +Cc: 20800-done

> For this snippet of code perl mode gets the colors wrong:
> return 'W' if               #/^Not Available on Mobile/m;    #W=Web only
>     /This video is not available on mobile devices./m;
> #After the m all is the wrong color.

Indeed, the code did not handle the case of a comment between the "if"
and the "/": it treated this slash as a division operator.
I installed a patch into the "master" branch which should fix it,


        Stefan





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

end of thread, other threads:[~2015-06-16  1:03 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-06-13  6:23 bug#20800: perl mode reads into comments too deeply 積丹尼 Dan Jacobson
2015-06-16  1:03 ` Stefan Monnier

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).