]> git.eshelyaron.com Git - emacs.git/commitdiff
Correct handling of template markers on deletion/insertion
authorAlan Mackenzie <acm@muc.de>
Wed, 21 Jun 2023 15:36:56 +0000 (15:36 +0000)
committerAlan Mackenzie <acm@muc.de>
Wed, 21 Jun 2023 15:43:41 +0000 (15:43 +0000)
This fixes bug#62841.

In particular, correct the syntax-table text properties on the
remaining <...>s.

* lisp/progmodes/cc-align.el
(c-lineup-template-args-indented-from-margin): New lineup
function.

* lisp/progmodes/cc-defs.el (c-put-char-properties): New macro.
(c-search-forward-non-nil-char-property): Handle terminating
limit correctly.

* lisp/progmodes/cc-engine.el
(c-clear-<-pair-props-if-match-after)
(c-clear->-pair-props-if-match-before): Return the position
outside the matching < or >, not merely t.
(c-end-of-literal): New function.
(c-unmark-<>-around-region): New function.
(c-before-change-check-<>-operators): Refactor, calling
c-unmark-<>-around-region.
(c-<>-get-restricted): New function, extracted from
c-restore-<>-properties.
(c-restore-<>-properties): Handle ">" characters whose matching
"<" has not yet been encountered.
(c-ml-string-opener-at-or-around-point): Fix an off by one
error.
(c-backward-<>-arglist): New parameter restricted-function, a
function which calculates c-restricted-<>-arglists for the
current operation.

* lisp/progmodes/cc-fonts.el (c-font-lock-c++-using): Check
point is less than limit in the loop.

* lisp/progmodes/cc-langs.el
(c-get-state-before-change-functions)
(c-before-font-lock-functions): Add the new function
c-unmark-<>-around-region into the C++ and Java values of the
variable.

* lisp/progmodes/cc-mode.el (c-clear-string-fences)
(c-restore-string-fences): Neutralize and restore the
syntax-table properties between an unbalanced " and EOL.

* lisp/progmodes/cc-vars.el (c-offsets-alist): Put new lineup
function c-lineup-template-args-indented-from-margin into entry
for template-args-cont.

* doc/misc/cc-mode.texi (List Line-Up): Document
c-lineup-template-args-indented-from-margin.

doc/misc/cc-mode.texi
lisp/progmodes/cc-align.el
lisp/progmodes/cc-defs.el
lisp/progmodes/cc-engine.el
lisp/progmodes/cc-fonts.el
lisp/progmodes/cc-langs.el
lisp/progmodes/cc-mode.el
lisp/progmodes/cc-vars.el

index 4ac9cc3ca2da6d7f56e2ccd44a9e1289c8643ef5..5f905be09d5956c08aaba57b08443c39397b41d0 100644 (file)
@@ -6252,6 +6252,16 @@ returned if there's no template argument on the first line.
 
 @comment ------------------------------------------------------------
 
+@defun c-lineup-template-args-indented-from-margin
+@findex lineup-template-args-indented-from-margin (c-)
+Indent a template argument line `c-basic-offset' from the left-hand
+margin of the line with the containing <.
+
+@workswith @code{template-args-cont}.
+@end defun
+
+@comment ------------------------------------------------------------
+
 @defun c-lineup-ObjC-method-call
 @findex lineup-ObjC-method-call @r{(c-)}
 For Objective-C code, line up selector args as Emacs Lisp mode does
index 34ef0b9c1af188715ab2c4155fbe9b2d53b9cc22..91a7665edbb230a628ef57344ec787eec9dfde32 100644 (file)
@@ -940,6 +940,16 @@ Works with: template-args-cont."
             (zerop (c-forward-token-2 1 nil (c-point 'eol))))
        (vector (current-column)))))
 
+(defun c-lineup-template-args-indented-from-margin (_langelem)
+  "Indent a template argument line `c-basic-offset' from the margin
+of the line with the containing <.
+
+Works with: template-args-cont."
+  (save-excursion
+    (goto-char (c-langelem-2nd-pos c-syntactic-element))
+    (back-to-indentation)
+    (vector (+ (current-column) c-basic-offset))))
+
 (defun c-lineup-ObjC-method-call (langelem)
   "Line up selector args as Emacs Lisp mode does with function args:
 Go to the position right after the message receiver, and if you are at
index 1d98b2155253e1a483b2f9174bfef25ebc8f9920..f9b63cbeed61c0ee060a38990b15d7b5be802e2a 100644 (file)
@@ -1284,6 +1284,29 @@ MODE is either a mode symbol or a list of mode symbols."
        pos)
       (most-positive-fixnum))))
 
+(defmacro c-put-char-properties (from to property value)
+  ;; FIXME!!!  Doc comment here!
+  (declare (debug t))
+  (setq property (eval property))
+  `(let ((-to- ,to) (-from- ,from))
+     ,(if c-use-extents
+         ;; XEmacs
+         `(progn
+            (map-extents (lambda (ext ignored)
+                           (delete-extent ext))
+                         nil -from- -to- nil nil ',property)
+            (set-extent-properties (make-extent -from- -to-)
+                                   (cons property
+                                         (cons ,value
+                                               '(start-open t
+                                                            end-open t)))))
+       ;; Emacs
+       `(progn
+          ,@(when (and (fboundp 'syntax-ppss)
+                       (eq `,property 'syntax-table))
+              `((setq c-syntax-table-hwm (min c-syntax-table-hwm -from-))))
+          (put-text-property -from- -to- ',property ,value)))))
+
 (defmacro c-clear-char-properties (from to property)
   ;; Remove all the occurrences of the given property in the given
   ;; region that has been put with `c-put-char-property'.  PROPERTY is
@@ -1379,7 +1402,8 @@ isn't found, return nil; point is then left undefined."
        value)
       (t (let ((place (c-next-single-property-change
                       (point) ,property nil -limit-)))
-          (when place
+          (when (and place
+                     (< place -limit-))
             (goto-char (1+ place))
             (c-get-char-property place ,property)))))))
 
index 66cfd3dee9edd6ad5dc7af10016f69c0e910b155..0eadeafc8365183164887bd5ccbcdad7ecf6fcff 100644 (file)
@@ -2672,6 +2672,7 @@ comment at the start of cc-engine.el for more info."
          (progn (goto-char beg)
                 (c-skip-ws-forward end+1)
                 (eq (point) end+1))))))
+
 \f
 ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
 ;; We maintain a sopisticated cache of positions which are in a literal,
@@ -7039,8 +7040,8 @@ comment at the start of cc-engine.el for more info."
   ;; POS (default point) is at a < character.  If it is both marked
   ;; with open/close paren syntax-table property, and has a matching >
   ;; (also marked) which is after LIM, remove the property both from
-  ;; the current > and its partner.  Return t when this happens, nil
-  ;; when it doesn't.
+  ;; the current > and its partner.  Return the position after the >
+  ;; when this happens, nil when it doesn't.
   (save-excursion
     (if pos
        (goto-char pos)
@@ -7054,15 +7055,15 @@ comment at the start of cc-engine.el for more info."
                        c->-as-paren-syntax)) ; should always be true.
        (c-unmark-<->-as-paren (1- (point)))
        (c-unmark-<->-as-paren pos)
-       (c-truncate-lit-pos-cache pos))
-      t)))
+       (c-truncate-lit-pos-cache pos)
+      (point)))))
 
 (defun c-clear->-pair-props-if-match-before (lim &optional pos)
   ;; POS (default point) is at a > character.  If it is both marked
   ;; with open/close paren syntax-table property, and has a matching <
   ;; (also marked) which is before LIM, remove the property both from
-  ;; the current < and its partner.  Return t when this happens, nil
-  ;; when it doesn't.
+  ;; the current < and its partner.  Return the position of the < when
+  ;; this happens, nil when it doesn't.
   (save-excursion
     (if pos
        (goto-char pos)
@@ -7076,8 +7077,8 @@ comment at the start of cc-engine.el for more info."
                        c-<-as-paren-syntax)) ; should always be true.
        (c-unmark-<->-as-paren (point))
        (c-truncate-lit-pos-cache (point))
-       (c-unmark-<->-as-paren pos))
-      t)))
+       (c-unmark-<->-as-paren pos)
+       (point)))))
 
 ;; Set by c-common-init in cc-mode.el.
 (defvar c-new-BEG)
@@ -7085,7 +7086,48 @@ comment at the start of cc-engine.el for more info."
 ;; Set by c-before-change-check-raw-strings.
 (defvar c-old-END-literality)
 
-(defun c-before-change-check-<>-operators (beg end)
+(defun c-end-of-literal (pt-s pt-search)
+  ;; If a literal is open in the `c-semi-pp-to-literal' state PT-S, return the
+  ;; end point of this literal (or point-max) assuming PT-S is valid at
+  ;; PT-SEARCH.  Otherwise, return nil.
+  (when (car (cddr pt-s))              ; Literal start
+    (let ((lit-type (cadr pt-s))
+         (lit-beg (car (cddr pt-s)))
+         ml-end-re
+         )
+      (save-excursion
+       (cond
+        ((eq lit-type 'string)
+         (if (and c-ml-string-opener-re
+                  (c-ml-string-opener-at-or-around-point lit-beg))
+             (progn
+               (setq ml-end-re
+                     (funcall c-make-ml-string-closer-re-function
+                              (match-string 1)))
+               (goto-char (max (- pt-search (1- (length ml-end-re)))
+                               (point-min)))
+               (re-search-forward ml-end-re nil 'stay))
+           ;; For an ordinary string, we can't use `parse-partial-sexp' since
+           ;; not all syntax-table properties have yet been set.
+           (goto-char pt-search)
+           (re-search-forward
+              "\\(?:\\\\\\(?:.\\|\n\\)\\|[^\"\n\\]\\)*[\"\n]" nil 'stay)))
+        ((memq lit-type '(c c++))
+         ;; To work around a bug in parse-partial-sexp, where effect is given
+         ;; to the syntax of a backslash, even the the scan starts with point
+         ;; just after it.
+         (if (and (eq (char-before pt-search) ?\\)
+                  (eq (char-after pt-search) ?\n))
+             (progn
+               (c-put-char-property (1- pt-search) 'syntax-table '(1))
+               (parse-partial-sexp pt-search (point-max) nil nil (car pt-s)
+                                   'syntax-table)
+               (c-clear-char-property (1- pt-search) 'syntax-table))
+         (parse-partial-sexp pt-search (point-max) nil nil (car pt-s)
+                             'syntax-table))))
+       (point)))))
+
+(defun c-unmark-<>-around-region (beg end &optional old-len)
   ;; Unmark certain pairs of "< .... >" which are currently marked as
   ;; template/generic delimiters.  (This marking is via syntax-table text
   ;; properties), and expand the (c-new-BEG c-new-END) region to include all
@@ -7099,66 +7141,201 @@ comment at the start of cc-engine.el for more info."
   ;; enclose a brace or semicolon, so we use these as bounds on the
   ;; region we must work on.
   ;;
+  ;; The buffer is widened, and point is undefined, both at entry and exit.
+  ;;
+  ;; FIXME!!!  This routine ignores the possibility of macros entirely.
+  ;; 2010-01-29.
+
+  (when (> end beg)
+    ;; Extend the region (BEG END) to deal with any complicating literals.
+    (let* ((lit-search-beg (if (memq (char-before beg) '(?/ ?*))
+                              (1- beg) beg))
+          (lit-search-end (if (memq (char-after end) '(?/ ?*))
+                              (1+ end) end))
+          ;; Note we can't use c-full-pp-to-literal here, since we haven't
+          ;; yet applied syntax-table properties to ends of lines, etc.
+          (lit-search-beg-s (c-semi-pp-to-literal lit-search-beg))
+          (beg-literal-beg (car (cddr lit-search-beg-s)))
+          (lit-search-end-s (c-semi-pp-to-literal lit-search-end))
+          (end-literal-beg (car (cddr lit-search-end-s)))
+          (beg-literal-end (c-end-of-literal lit-search-beg-s beg))
+          (end-literal-end (c-end-of-literal lit-search-end-s end))
+          new-beg new-end search-region)
+
+      ;; Determine any new end of literal resulting from the insertion/deletion.
+      (setq search-region
+           (if (and (eq beg-literal-beg end-literal-beg)
+                    (eq beg-literal-end end-literal-end))
+               (if beg-literal-beg
+                   nil
+                 (cons beg
+                       (max end
+                            (or beg-literal-end (point-min))
+                            (or end-literal-end (point-min)))))
+             (cons (or beg-literal-beg beg)
+                   (max end
+                        (or beg-literal-end (point-min))
+                        (or end-literal-end (point-min))))))
+
+      (when search-region
+       ;; If we've just inserted text, mask its syntaxes temporarily so that
+       ;; they won't interfere with the undoing of the properties on the <s
+       ;; and >s.
+       (c-save-buffer-state (syn-tab-settings syn-tab-value
+                                              swap-open-string-ends)
+         (unwind-protect
+             (progn
+               (when old-len
+                 ;; Special case: If a \ has just been inserted into a
+                 ;; string, escaping or unescaping a LF, temporarily swap
+                 ;; the LF's syntax-table text property with that of the
+                 ;; former end of the open string.
+                 (goto-char end)
+                 (when (and (eq (cadr lit-search-beg-s) 'string)
+                            (not (eq beg-literal-end end-literal-end))
+                            (skip-chars-forward "\\\\")
+                            (eq (char-after) ?\n)
+                            (not (zerop (skip-chars-backward "\\\\"))))
+                   (setq swap-open-string-ends t)
+                   (if (c-get-char-property (1- beg-literal-end)
+                                            'syntax-table)
+                       (progn
+                         (c-clear-char-property (1- beg-literal-end)
+                                                'syntax-table)
+                         (c-put-char-property (1- end-literal-end)
+                                              'syntax-table '(15)))
+                     (c-put-char-property (1- beg-literal-end)
+                                          'syntax-table '(15))
+                     (c-clear-char-property (1- end-literal-end)
+                                            'syntax-table)))
+
+                 ;; Save current settings of the 'syntax-table property in
+                 ;; (BEG END), then splat these with the punctuation value.
+                 (goto-char beg)
+                 (while (progn (skip-syntax-forward "" end)
+                               (< (point) end))
+                   (setq syn-tab-value
+                         (c-get-char-property (point) 'syntax-table))
+                   (when (not (c-get-char-property (point) 'category))
+                     (push (cons (point) syn-tab-value) syn-tab-settings))
+                   (forward-char))
+
+                 (c-put-char-properties beg end 'syntax-table '(1))
+                 ;; If an open string's opener has just been neutralized,
+                 ;; do the same to the terminating LF.
+                 (when (and end-literal-end
+                            (eq (char-before end-literal-end) ?\n)
+                            (equal (c-get-char-property
+                                    (1- end-literal-end) 'syntax-table)
+                                   '(15)))
+                   (push (cons (1- end-literal-end) '(15)) syn-tab-settings)
+                   (c-put-char-property (1- end-literal-end) 'syntax-table
+                                        '(1))))
+
+               (let
+                   ((beg-lit-start (progn (goto-char beg) (c-literal-start)))
+                    beg-limit end-limit <>-pos)
+                 ;; Locate the earliest < after the barrier before the
+                 ;; changed region, which isn't already marked as a paren.
+                 (goto-char (or beg-lit-start beg))
+                 (setq beg-limit (c-determine-limit 5000))
+
+                 ;; Remove the syntax-table/category properties from each pertinent <...>
+                 ;; pair.  Firstly, the ones with the < before beg and > after beg....
+                 (goto-char (cdr search-region))
+                 (while (progn (c-syntactic-skip-backward "^;{}<" beg-limit)
+                               (eq (char-before) ?<))
+                   (c-backward-token-2)
+                   (when (eq (char-after) ?<)
+                     (when (setq <>-pos (c-clear-<-pair-props-if-match-after
+                                         (car search-region)))
+                       (setq new-end <>-pos))
+                     (setq new-beg (point))))
+
+                 ;; ...Then the ones with < before end and > after end.
+                 (goto-char (car search-region))
+                 (setq end-limit (c-determine-+ve-limit 5000))
+                 (while (and (c-syntactic-re-search-forward "[;{}>]" end-limit 'end)
+                             (eq (char-before) ?>))
+                   (when (eq (char-before) ?>)
+                     (if (and (looking-at c->-op-cont-regexp)
+                              (not (eq (char-after) ?>)))
+                         (goto-char (match-end 0))
+                       (when
+                           (and (setq <>-pos
+                                      (c-clear->-pair-props-if-match-before
+                                       (cdr search-region)
+                                       (1- (point))))
+                                (or (not new-beg)
+                                    (< <>-pos new-beg)))
+                         (setq new-beg <>-pos))
+                       (when (or (not new-end) (> (point) new-end))
+                         (setq new-end (point))))))))
+
+           (when old-len
+             (c-clear-char-properties beg end 'syntax-table)
+             (dolist (elt syn-tab-settings)
+               (if (cdr elt)
+                   (c-put-char-property (car elt) 'syntax-table (cdr elt)))))
+           ;; Swap the '(15) syntax-table property on open string LFs back
+           ;; again.
+           (when swap-open-string-ends
+             (if (c-get-char-property (1- beg-literal-end)
+                                      'syntax-table)
+                 (progn
+                   (c-clear-char-property (1- beg-literal-end)
+                                          'syntax-table)
+                   (c-put-char-property (1- end-literal-end)
+                                        'syntax-table '(15)))
+               (c-put-char-property (1- beg-literal-end)
+                                    'syntax-table '(15))
+               (c-clear-char-property (1- end-literal-end)
+                                      'syntax-table)))))
+         ;; Extend the fontification region, if needed.
+         (and new-beg
+              (< new-beg c-new-BEG)
+              (setq c-new-BEG new-beg))
+         (and new-end
+              (> new-end c-new-END)
+              (setq c-new-END new-end))))))
+
+(defun c-before-change-check-<>-operators (beg end)
+  ;; When we're deleting text, unmark certain pairs of "< .... >" which are
+  ;; currently marked as template/generic delimiters.  (This marking is via
+  ;; syntax-table text properties), and expand the (c-new-BEG c-new-END)
+  ;; region to include all unmarked < and > operators within the certain
+  ;; bounds (see below).
+  ;;
+  ;; These pairs are those which are in the current "statement" (i.e.,
+  ;; the region between the {, }, or ; before BEG and the one after
+  ;; END), and which enclose any part of the interval (BEG END).
+  ;; Also unmark a < or > which is about to become part of a multi-character
+  ;; operator, e.g. <=.
+  ;;
+  ;; Note that in C++ (?and Java), template/generic parens cannot
+  ;; enclose a brace or semicolon, so we use these as bounds on the
+  ;; region we must work on.
+  ;;
   ;; This function is called from before-change-functions (via
   ;; c-get-state-before-change-functions).  Thus the buffer is widened,
   ;; and point is undefined, both at entry and exit.
   ;;
   ;; FIXME!!!  This routine ignores the possibility of macros entirely.
   ;; 2010-01-29.
-  (when (and (or (> end beg)
-                (and (> c-<-pseudo-digraph-cont-len 0)
-                     (goto-char beg)
-                     (progn
-                       (skip-chars-backward
-                        "^<" (max (- (point) c-<-pseudo-digraph-cont-len)
-                                  (point-min)))
-                       (eq (char-before) ?<))
-                     (looking-at c-<-pseudo-digraph-cont-regexp)))
-            (or
-             (progn
-               (goto-char beg)
-               (search-backward "<" (max (- (point) 1024) (point-min)) t))
-             (progn
-               (goto-char end)
-               (search-forward ">" (min (+ (point) 1024) (point-max)) t))))
-    (save-excursion
-      (c-save-buffer-state
-         ((beg-lit-start (progn (goto-char beg) (c-literal-start)))
-          (end-lit-limits (progn (goto-char end) (c-literal-limits)))
-          new-beg new-end beg-limit end-limit)
-       ;; Locate the earliest < after the barrier before the changed region,
-       ;; which isn't already marked as a paren.
-       (goto-char (or beg-lit-start beg))
-       (setq beg-limit (c-determine-limit 512))
-
-       ;; Remove the syntax-table/category properties from each pertinent <...>
-       ;; pair.  Firstly, the ones with the < before beg and > after beg....
-       (while (progn (c-syntactic-skip-backward "^;{}<" beg-limit)
-                     (eq (char-before) ?<))
-         (c-backward-token-2)
-         (when (eq (char-after) ?<)
-           (c-clear-<-pair-props-if-match-after beg)
-           (setq new-beg (point))))
-       (c-forward-syntactic-ws)
-
-       ;; ...Then the ones with < before end and > after end.
-       (goto-char (if end-lit-limits (cdr end-lit-limits) end))
-       (setq end-limit (c-determine-+ve-limit 512))
-       (while (and (c-syntactic-re-search-forward "[;{}>]" end-limit 'end)
-                   (eq (char-before) ?>))
-         (c-end-of-current-token)
-         (when (eq (char-before) ?>)
-           (c-clear->-pair-props-if-match-before end (1- (point)))
-           (setq new-end (point))))
-       (c-backward-syntactic-ws)
-
-       ;; Extend the fontification region, if needed.
-       (and new-beg
-            (< new-beg c-new-BEG)
-            (setq c-new-BEG new-beg))
-       (and new-end
-            (> new-end c-new-END)
-            (setq c-new-END new-end))))))
+  (when (> end beg)
+  ;; Cope with removing (beg end) coalescing a < or > with, say, an = sign.
+    (goto-char beg)
+    (let ((ch (char-before)))
+      (if (and (memq ch '(?< ?>))
+              (c-get-char-property (1- (point)) 'syntax-table)
+              (progn
+                (goto-char end)
+                (looking-at (if (eq ch ?<)
+                                c-<-op-cont-regexp
+                              c->-op-cont-regexp)))
+              (or (eq ch ?<)
+                  (not (eq (char-after) ?>))))
+         (c-unmark-<>-around-region (1- beg) beg)))))
 
 (defun c-after-change-check-<>-operators (beg end)
   ;; This is called from `after-change-functions' when
@@ -7198,29 +7375,38 @@ comment at the start of cc-engine.el for more info."
            (c-clear-<>-pair-props)
            (forward-char)))))))
 
+(defun c-<>-get-restricted ()
+  ;; With point at the < at the start of the purported <>-arglist, determine
+  ;; the value of `c-restricted-<>-arglists' to use for the call of
+  ;; `c-forward-<>-arglist' starting there.
+  (save-excursion
+    (c-backward-token-2)
+    (and (not (looking-at c-opt-<>-sexp-key))
+        (progn (c-backward-syntactic-ws)                 ; to ( or ,
+               (and (memq (char-before) '(?\( ?,))       ; what about <?
+                    (not (eq (c-get-char-property (point) 'c-type)
+                             'c-decl-arg-start)))))))
+
 (defun c-restore-<>-properties (_beg _end _old-len)
   ;; This function is called as an after-change function.  It restores the
   ;; category/syntax-table properties on template/generic <..> pairs between
   ;; c-new-BEG and c-new-END.  It may do hidden buffer changes.
-  (c-save-buffer-state ((c-parse-and-markup-<>-arglists t)
-                       c-restricted-<>-arglists lit-limits)
+  (c-save-buffer-state ((c-parse-and-markup-<>-arglists t) lit-limits)
     (goto-char c-new-BEG)
     (if (setq lit-limits (c-literal-limits))
        (goto-char (cdr lit-limits)))
     (while (and (< (point) c-new-END)
-               (c-syntactic-re-search-forward "<" c-new-END 'bound))
-      (backward-char)
-      (save-excursion
-       (c-backward-token-2)
-       (setq c-restricted-<>-arglists
-            (and (not (looking-at c-opt-<>-sexp-key))
-                 (progn (c-backward-syntactic-ws) ; to ( or ,
-                        (and (memq (char-before) '(?\( ?,)) ; what about <?
-                             (not (eq (c-get-char-property (point) 'c-type)
-                                      'c-decl-arg-start)))))))
-      (or (c-forward-<>-arglist nil)
-         (c-forward-over-token-and-ws)
-         (goto-char c-new-END)))))
+               (c-syntactic-re-search-forward "[<>]" c-new-END 'bound))
+      (if (eq (char-before) ?<)
+         (progn
+           (backward-char)
+           (let ((c-restricted-<>-arglists (c-<>-get-restricted)))
+             (or (c-forward-<>-arglist nil)
+                 (c-forward-over-token-and-ws)
+                 (goto-char c-new-END))))
+       (save-excursion
+         (when (c-backward-<>-arglist nil nil #'c-<>-get-restricted)
+           (setq c-new-BEG (min c-new-BEG (point)))))))))
 
 \f
 ;; Handling of CC Mode multi-line strings.
@@ -7372,13 +7558,13 @@ multi-line strings (but not C++, for example)."
 
 (defun c-ml-string-opener-intersects-region (&optional start finish)
   ;; If any part of the region [START FINISH] is inside an ml-string opener,
-  ;; return a dotted list of the start, end and double-quote position of that
-  ;; opener.  That list will not include any "context characters" before or
-  ;; after the opener.  If an opener is found, the match-data will indicate
-  ;; it, with (match-string 1) being the entire delimiter, and (match-string
-  ;; 2) the "main" double-quote.  Otherwise, the match-data is undefined.
-  ;; Both START and FINISH default to point.  FINISH may not be at an earlier
-  ;; buffer position than START.
+  ;; return a dotted list of the start, end and double-quote position of the
+  ;; first such opener.  That list wlll not include any "context characters"
+  ;; before or after the opener.  If an opener is found, the match-data will
+  ;; indicate it, with (match-string 1) being the entire delimiter, and
+  ;; (match-string 2) the "main" double-quote.  Otherwise, the match-data is
+  ;; undefined.  Both START and FINISH default to point.  FINISH may not be at
+  ;; an earlier buffer position than START.
   (let ((here (point)) found)
     (or finish (setq finish (point)))
     (or start (setq start (point)))
@@ -7402,7 +7588,10 @@ multi-line strings (but not C++, for example)."
   ;; If POSITION (default point) is at or inside an ml string opener, return a
   ;; dotted list of the start and end of that opener, and the position of the
   ;; double-quote in it.  That list will not include any "context characters"
-  ;; before or after the opener.
+  ;; before or after the opener.  If an opener is found, the match-data will
+  ;; indicate it, with (match-string 1) being the entire delimiter, and
+  ;; (match-string 2) the "main" double-quote.  Otherwise, the match-data is
+  ;; undefined.
   (let ((here (point))
        found)
     (or position (setq position (point)))
@@ -7414,7 +7603,7 @@ multi-line strings (but not C++, for example)."
                c-ml-string-opener-re
                (min (+ position c-ml-string-max-opener-len) (point-max))
                'bound))
-        (<= (match-end 1) position)))
+        (< (match-end 1) position)))
     (prog1
        (and found
             (<= (match-beginning 1) position)
@@ -8821,7 +9010,7 @@ multi-line strings (but not C++, for example)."
       (if res
          (or c-record-found-types t)))))
 
-(defun c-backward-<>-arglist (all-types &optional limit)
+(defun c-backward-<>-arglist (all-types &optional limit restricted-function)
   ;; The point is assumed to be directly after a ">".  Try to treat it
   ;; as the close paren of an angle bracket arglist and move back to
   ;; the corresponding "<".  If successful, the point is left at
@@ -8830,7 +9019,12 @@ multi-line strings (but not C++, for example)."
   ;; `c-forward-<>-arglist'.
   ;;
   ;; If the optional LIMIT is given, it bounds the backward search.
-  ;; It's then assumed to be at a syntactically relevant position.
+  ;; It's then assumed to be at a syntactically relevant position.  If
+  ;; RESTRICTED-FUNCTION is non-nil, it should be a function taking no
+  ;; arguments, called with point at a < at the start of a purported
+  ;; <>-arglist, which will return the value of
+  ;; `c-restricted-<>-arglists' to be used in the `c-forward-<>-arglist'
+  ;; call starting at that <.
   ;;
   ;; This is a wrapper around `c-forward-<>-arglist'.  See that
   ;; function for more details.
@@ -8866,7 +9060,11 @@ multi-line strings (but not C++, for example)."
                   t
 
                 (backward-char)
-                (let ((beg-pos (point)))
+                (let ((beg-pos (point))
+                      (c-restricted-<>-arglists
+                       (if restricted-function
+                           (funcall restricted-function)
+                         c-restricted-<>-arglists)))
                   (if (c-forward-<>-arglist all-types)
                       (cond ((= (point) start)
                              ;; Matched the arglist.  Break the while.
index 9118e3253c2afc383d45f9acf4b6dce1133d451a..d220af2ab0e0cdae49b92e3ec769fd3fe1f643cc 100644 (file)
@@ -2659,7 +2659,9 @@ need for `c-font-lock-extra-types'.")
   ;; prevent a repeat invocation.  See elisp/lispref page "Search-based
   ;; fontification".
   (let (pos)
-    (while (c-syntactic-re-search-forward c-using-key limit 'end)
+    (while 
+       (and (< (point) limit)
+            (c-syntactic-re-search-forward c-using-key limit 'end))
       (while  ; Do one declarator of a comma separated list, each time around.
          (progn
            (c-forward-syntactic-ws)
index ffb8c5c7b16085a8b44ff3328a46fd32de5f51e3..d56366e17555171d463eb9c646ca47ac27f27870 100644 (file)
@@ -455,6 +455,7 @@ so that all identifiers are recognized as words.")
   c++ '(c-extend-region-for-CPP
        c-depropertize-CPP
        c-before-change-check-ml-strings
+       c-unmark-<>-around-region
        c-before-change-check-<>-operators
        c-before-after-change-check-c++-modules
        c-truncate-bs-cache
@@ -468,6 +469,7 @@ so that all identifiers are recognized as words.")
             c-parse-quotes-before-change
             c-before-change-fix-comment-escapes)
   java '(c-parse-quotes-before-change
+        c-unmark-<>-around-region
         c-before-change-check-unbalanced-strings
         c-before-change-check-<>-operators)
   pike '(c-before-change-check-ml-strings
@@ -516,6 +518,7 @@ parameters \(point-min) and \(point-max).")
        c-after-change-unmark-ml-strings
        c-parse-quotes-after-change
        c-after-change-mark-abnormal-strings
+       c-unmark-<>-around-region
        c-extend-font-lock-region-for-macros
        c-before-after-change-check-c++-modules
        c-neutralize-syntax-in-CPP
@@ -524,6 +527,7 @@ parameters \(point-min) and \(point-max).")
   java '(c-depropertize-new-text
         c-after-change-escape-NL-in-string
         c-parse-quotes-after-change
+        c-unmark-<>-around-region
         c-after-change-mark-abnormal-strings
         c-restore-<>-properties
         c-change-expand-fl-region)
index 5cf9b7e17f8c6aa178b59c7ea22761cea7eee50b..dd699b9a11975adb557fccc48d6cfd0cefe2aca1 100644 (file)
@@ -1367,7 +1367,9 @@ Note that the style variables are always made local to the buffer."
                      (and              ;(< (point) end)
                       (not (nth 3 s))
                       (c-get-char-property (1- (point)) 'c-fl-syn-tab))
-                   (c-put-char-property pos 'syntax-table '(1)))
+                   (c-put-char-property pos 'syntax-table '(1))
+                   (c-put-char-properties (1+ pos) (c-point 'eol pos)
+                                          'syntax-table '(1)))
                  (setq pos (point)))
              (setq pos (1+ pos)))))))))
 
@@ -1384,6 +1386,9 @@ Note that the style variables are always made local to the buffer."
           (setq pos
                 (c-min-property-position pos c-max-syn-tab-mkr 'c-fl-syn-tab))
           (< pos c-max-syn-tab-mkr))
+       (when (and (equal (c-get-char-property pos 'syntax-table) '(1))
+                  (equal (c-get-char-property pos 'c-fl-syn-tab) '(15)))
+         (c-clear-char-properties (1+ pos) (c-point 'eol pos) 'syntax-table))
        (c-put-char-property pos 'syntax-table
                             (c-get-char-property pos 'c-fl-syn-tab))
        (setq pos (1+ pos))))))
index 72d4b93ee59af592b13f539a0334dc56249cc2c6..286d569aaca351115d5b07a291d7ae0c6848d07e 100644 (file)
@@ -1219,7 +1219,8 @@ can always override the use of `c-default-style' by making calls to
        (incomposition         . +)
        ;; Anchor pos: At the extern/namespace/etc block open brace if
        ;; it's at boi, otherwise boi at the keyword.
-       (template-args-cont    . (c-lineup-template-args +))
+       (template-args-cont    . (c-lineup-template-args
+                                c-lineup-template-args-indented-from-margin))
        ;; Anchor pos: Boi at the decl start.  This might be changed;
        ;; the logical position is clearly the opening '<'.
        (inlambda              . 0)