GNU bug report logs - #55230
CC Mode 5.35.1 (C++//l); Cache inconsistency

Previous Next

Package: cc-mode;

Reported by: Michael Welsh Duggan <mwd <at> md5i.com>

Date: Mon, 2 May 2022 19:31:02 UTC

Severity: normal

Done: Alan Mackenzie <acm <at> muc.de>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Alan Mackenzie <acm <at> muc.de>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#55230: closed (CC Mode 5.35.1 (C++//l); Cache inconsistency)
Date: Sun, 22 May 2022 17:13:02 +0000
[Message part 1 (text/plain, inline)]
Your message dated Sun, 22 May 2022 17:12:11 +0000
with message-id <Yopu64qoL4fBqDJj <at> ACM>
and subject line Re: bug#55230: CC Mode 5.35.1 (C++//l); Cache inconsistency
has caused the debbugs.gnu.org bug report #55230,
regarding CC Mode 5.35.1 (C++//l); Cache inconsistency
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
55230: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=55230
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Michael Welsh Duggan <mwd <at> md5i.com>
To: bug-gnu-emacs <at> gnu.org
Subject: CC Mode 5.35.1 (C++//l); Cache inconsistency
Date: Mon, 02 May 2022 15:30:09 -0400
Package: cc-mode

From emacs -Q:

C-x C-f /tmp/foo.cpp RET
M-x c-toggle-parse-state-debug RET
"{}

After typing the close brace, I get the following:
c-parse-state inconsistency at 3: using cache: (2), from scratch: nil.  POINT-MIN: 1
Old state:
(setq c-state-cache '(2)  c-state-cache-good-pos 1  c-state-nonlit-pos-cache nil  c-state-nonlit-pos-cache-limit 0  c-state-brace-pair-desert nil  c-state-point-min 1  c-state-point-min-lit-type nil  c-state-point-min-lit-start nil  c-state-min-scan-pos 1  c-state-old-cpp-beg nil  c-state-old-cpp-end nil  c-parse-state-point 3)

I have not noticed any indentation errors due to this, but any cache
inconsistency might lead to errors down the line, hence this report.

Emacs  : GNU Emacs 29.0.50 (build 1, x86_64-pc-linux-gnu, X toolkit, cairo version 1.16.0)
 of 2022-04-09
Package: CC Mode 5.35.1 (C++//l)
Buffer Style: gnu
c-emacs-features: (pps-extended-state col-0-paren posix-char-classes gen-string-delim gen-comment-delim syntax-properties 1-bit)

current state:
==============
(setq
 c-basic-offset 2
 c-comment-only-line-offset '(0 . 0)
 c-indent-comment-alist '((anchored-comment column . 0) (end-block space . 1)
			  (cpp-end-block space . 2))
 c-indent-comments-syntactically-p nil
 c-block-comment-prefix ""
 c-comment-prefix-regexp '((pike-mode . "//+!?\\|\\**") (awk-mode . "#+")
			   (other . "//+\\|\\**"))
 c-doc-comment-style '((java-mode . javadoc) (pike-mode . autodoc)
		       (c-mode . gtkdoc) (c++-mode . gtkdoc))
 c-cleanup-list '(scope-operator)
 c-hanging-braces-alist '((substatement-open before after)
			  (arglist-cont-nonempty))
 c-hanging-colons-alist nil
 c-hanging-semi&comma-criteria '(c-semi&comma-inside-parenlist)
 c-backslash-column 48
 c-backslash-max-column 72
 c-special-indent-hook '(t c-gnu-impose-minimum)
 c-label-minimum-indentation 1
 c-offsets-alist '((inexpr-class . +)
		   (inexpr-statement . +)
		   (lambda-intro-cont . +)
		   (inlambda . 0)
		   (template-args-cont c-lineup-template-args +)
		   (incomposition . +)
		   (inmodule . +)
		   (innamespace . +)
		   (inextern-lang . +)
		   (composition-close . 0)
		   (module-close . 0)
		   (namespace-close . 0)
		   (extern-lang-close . 0)
		   (composition-open . 0)
		   (module-open . 0)
		   (namespace-open . 0)
		   (extern-lang-open . 0)
		   (objc-method-call-cont
		    c-lineup-ObjC-method-call-colons
		    c-lineup-ObjC-method-call
		    +
		    )
		   (objc-method-args-cont . c-lineup-ObjC-method-args)
		   (objc-method-intro . [0])
		   (friend . 0)
		   (cpp-define-intro c-lineup-cpp-define +)
		   (cpp-macro-cont . +)
		   (cpp-macro . [0])
		   (inclass . +)
		   (stream-op . c-lineup-streamop)
		   (arglist-cont-nonempty
		    c-lineup-gcc-asm-reg
		    c-lineup-arglist
		    )
		   (arglist-cont c-lineup-gcc-asm-reg 0)
		   (comment-intro
		    c-lineup-knr-region-comment
		    c-lineup-comment
		    )
		   (catch-clause . 0)
		   (else-clause . 0)
		   (do-while-closure . 0)
		   (access-label . -)
		   (case-label . 0)
		   (substatement . +)
		   (statement-case-intro . +)
		   (statement . 0)
		   (brace-entry-open . 0)
		   (brace-list-entry . 0)
		   (brace-list-close . 0)
		   (block-close . 0)
		   (block-open . 0)
		   (inher-cont . c-lineup-multi-inher)
		   (inher-intro . +)
		   (member-init-cont . c-lineup-multi-inher)
		   (member-init-intro . +)
		   (annotation-var-cont . +)
		   (annotation-top-cont . 0)
		   (topmost-intro . 0)
		   (knr-argdecl . 0)
		   (func-decl-cont . +)
		   (inline-close . 0)
		   (class-close . 0)
		   (class-open . 0)
		   (defun-block-intro . +)
		   (defun-close . 0)
		   (defun-open . 0)
		   (c . c-lineup-C-comments)
		   (string . c-lineup-dont-change)
		   (topmost-intro-cont
		    first
		    c-lineup-topmost-intro-cont
		    c-lineup-gnu-DEFUN-intro-cont
		    )
		   (brace-list-intro
		    first
		    c-lineup-2nd-brace-entry-in-arglist
		    c-lineup-class-decl-init-+
		    +
		    )
		   (brace-list-open . +)
		   (inline-open . 0)
		   (arglist-close . c-lineup-arglist)
		   (arglist-intro . c-lineup-arglist-intro-after-paren)
		   (statement-cont . +)
		   (statement-case-open . +)
		   (label . 0)
		   (substatement-label . 0)
		   (substatement-open . +)
		   (knr-argdecl-intro . 5)
		   (statement-block-intro . +)
		   )
 c-buffer-is-cc-mode 'c++-mode
 c-tab-always-indent t
 c-syntactic-indentation t
 c-syntactic-indentation-in-macros t
 c-ignore-auto-fill '(string cpp code)
 c-auto-align-backslashes t
 c-backspace-function 'backward-delete-char-untabify
 c-delete-function 'delete-char
 c-electric-pound-behavior nil
 c-default-style '((java-mode . "java") (awk-mode . "awk") (other . "gnu"))
 c-enable-xemacs-performance-kludge-p nil
 c-old-style-variable-behavior nil
 defun-prompt-regexp nil
 tab-width 8
 comment-column 32
 parse-sexp-ignore-comments t
 parse-sexp-lookup-properties t
 auto-fill-function nil
 comment-multi-line t
 comment-start-skip "\\(?://+\\|/\\*+\\)\\s *"
 fill-prefix nil
 fill-column 70
 paragraph-start "[ 	]*\\(//+\\|\\**\\)[ 	]*$\\|^\f"
 adaptive-fill-mode t
 adaptive-fill-regexp "[ 	]*\\(//+\\|\\**\\)[ 	]*\\([ 	]*\\([-–!|#%;>*·•‣⁃◦]+[ 	]*\\)*\\)"
 )
 

-- 
Michael Welsh Duggan
(md5i <at> md5i.com)


[Message part 3 (message/rfc822, inline)]
From: Alan Mackenzie <acm <at> muc.de>
To: Michael Welsh Duggan <mwd <at> md5i.com>
Cc: acm <at> muc.de, 55230-done <at> debbugs.gnu.org
Subject: Re: bug#55230: CC Mode 5.35.1 (C++//l); Cache inconsistency
Date: Sun, 22 May 2022 17:12:11 +0000
Hello, Michael.

On Sun, May 22, 2022 at 09:33:53 -0400, Michael Welsh Duggan wrote:
> Alan Mackenzie <acm <at> muc.de> writes:

> > On Thu, May 19, 2022 at 00:24:18 -0400, Michael Welsh Duggan wrote:

> >> Unfortunately, it [the patch] has some major problems.  If you follow
> >> the original recipe, after the first brace is input cc-mode errors out
> >> with "Marker does not point anywhere".  Backtrace follows:

> >> Debugger entered--Lisp error: (error "Marker does not point anywhere")
> >>   c-restore-string-fences()
> >>   c-electric-brace(nil)
> >>   funcall-interactively(c-electric-brace nil)
> >>   command-execute(c-electric-brace)

> > Apologies for that.  I had failed to test the scenario from a freshly
> > started Emacs.  The problem was setting and clearing text properties
> > without masking before-change-functions and after-change-functions.
> > This allowed a recursive call to these text property functions which
> > fouled things up.

> > This is now corrected.  Please undo the previous faulty patch and apply
> > the one in this post instead.  I think it should have fixed the bug.
> > Thanks!  I look forward to hearing from you again.

> That seems to have done the trick.  Thank you!

Thank you for the testing!  I have now committed the patch, and I am
closing the bug with this post.

> >> >> > Emacs : GNU Emacs 29.0.50 (build 1, x86_64-pc-linux-gnu, X
> >> >> > toolkit, cairo version 1.16.0)
> >> >> >  of 2022-04-09
> >> >> > Package: CC Mode 5.35.1 (C++//l)
> >> >> > Buffer Style: gnu
> >> >> > c-emacs-features: (pps-extended-state col-0-paren
> >> >> > posix-char-classes gen-string-delim gen-comment-delim
> >> >> > syntax-properties 1-bit)

> -- 
> Michael Welsh Duggan
> (md5i <at> md5i.com)

-- 
Alan Mackenzie (Nuremberg, Germany).


This bug report was last modified 2 years and 363 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.