Commit 5af99187 authored by Tassilo Horn's avatar Tassilo Horn

Allow back-references in syntax-propertize-rules.

* lisp/emacs-lisp/syntax.el (syntax-propertize--shift-groups-and-backrefs):
Renamed from syntax-propertize--shift-groups, and also shift
(syntax-propertize-rules): Adapt docstring and use renamed function.
* test/lisp/emacs-lisp/syntax-tests.el: New test.
(syntax-propertize--shift-groups-and-backrefs): New ERT test.
parent 659ed857
Pipeline #5620 failed with stage
in 56 minutes and 43 seconds
......@@ -139,14 +139,28 @@ delimiter or an Escaped or Char-quoted character."))
(cons beg end))
(defun syntax-propertize--shift-groups (re n)
(lambda (s)
(number-to-string (+ n (string-to-number (match-string 1 s))))
t t s 1))
re t t))
(defun syntax-propertize--shift-groups-and-backrefs (re n)
(let ((new-re (replace-regexp-in-string
(lambda (s)
(+ n (string-to-number (match-string 1 s))))
t t s 1))
re t t))
(pos 0))
(while (string-match "\\\\\\([0-9]+\\)" new-re pos)
(setq pos (+ 1 (match-beginning 1)))
(when (save-match-data
;; With \N, the \ must be in a subregexp context, i.e.,
;; not in a character class or in a \{\} repetition.
(subregexp-context-p new-re (match-beginning 0)))
(let ((shifted (+ n (string-to-number (match-string 1 new-re)))))
(when (> shifted 9)
(error "There may be at most nine back-references"))
(setq new-re (replace-match (number-to-string shifted)
t t new-re 1)))))
(defmacro syntax-propertize-precompile-rules (&rest rules)
"Return a precompiled form of RULES to pass to `syntax-propertize-rules'.
......@@ -190,7 +204,8 @@ for subsequent HIGHLIGHTs.
Also SYNTAX is free to move point, in which case RULES may not be applied to
some parts of the text or may be applied several times to other parts.
Note: back-references in REGEXPs do not work."
Note: There may be at most nine back-references in the REGEXPs of
all RULES in total."
(declare (debug (&rest &or symbolp ;FIXME: edebug this eval step.
(form &rest
......@@ -219,7 +234,7 @@ Note: back-references in REGEXPs do not work."
;; tell when *this* match 0 has succeeded.
(cl-incf offset)
(setq re (concat "\\(" re "\\)")))
(setq re (syntax-propertize--shift-groups re offset))
(setq re (syntax-propertize--shift-groups-and-backrefs re offset))
(let ((code '())
;;; syntax-tests.el --- tests for syntax.el -*- lexical-binding: t; -*-
;; Copyright (C) 2020 Free Software Foundation, Inc.
;; This file is part of GNU Emacs.
;; GNU Emacs is free software; you can redistribute it and/or modify
;; it under the terms of the GNU General Public License as published by
;; the Free Software Foundation, either version 3 of the License, or
;; (at your option) any later version.
;; GNU Emacs is distributed in the hope that it will be useful,
;; but WITHOUT ANY WARRANTY; without even the implied warranty of
;; GNU General Public License for more details.
;; You should have received a copy of the GNU General Public License
;; along with GNU Emacs. If not, see <>.
;;; Code:
(require 'ert)
(require 'syntax)
(ert-deftest syntax-propertize--shift-groups-and-backrefs ()
"Test shifting of numbered groups and back-references in regexps."
;; A numbered group must be shifted.
"\\(?2:[abc]+\\)foobar" 2)
;; A back-reference \1 on a normal sub-regexp context must be
;; shifted.
(syntax-propertize--shift-groups-and-backrefs "\\(a\\)\\1" 2)
;; Shifting must not happen if the \1 appears in a character class,
;; or in a \{\} repetition construct (although \1 isn't valid there
;; anyway).
(let ((rx-with-class "\\(a\\)[\\1-2]")
(rx-with-rep "\\(a\\)\\{1,\\1\\}"))
(syntax-propertize--shift-groups-and-backrefs rx-with-class 2)
(syntax-propertize--shift-groups-and-backrefs rx-with-rep 2)
;; Now numbered groups and back-references in combination.
"\\(?2:[abc]+\\)foo\\(\\2\\)" 2)
;; Emacs supports only the back-references \1,...,\9, so when a
;; shift would result in \10 or more, an error must be signalled.
(syntax-propertize--shift-groups-and-backrefs "\\(a\\)\\3" 7)))
;; Local Variables:
;; no-byte-compile: t
;; End:
;;; syntax-tests.el ends here.
Markdown is supported
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment