Skip to content

Common misc - #912

Merged
Matafou merged 2 commits into
ProofGeneral:masterfrom
Matafou:common-misc
Nov 6, 2026
Merged

Matafou merged 2 commits into
ProofGeneral:masterfrom
Matafou:common-misc

Conversation

@Matafou

@Matafou Matafou commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

Coq: search blacklist in all backends, 3-window policy in proof-useropts.el

  • proof-guess-3win-display-policy moves from pg-response.el to proof-useropts.el, unchanged. Other backends can then use it without loading pg-response.
  • Search blacklist: the options, the history and the commands (coq-change-search-blacklist, coq-change-search-blacklist-interactive) move from coq.el to coq-queries.el.
    • Each backend applies the change through the new hook coq-set-search-blacklist-function, which gets the old and the new value.
    • Proof General sets it to coq-pg-set-search-blacklist in coq.el, which sends Remove/Add Search Blacklist.
    • Without a backend, the command signals a user-error.
  • Tests: Print Table Search Blacklist with the proof shell, and the hook without Proof General. Also a CHANGES entry.

Matafou and others added 2 commits November 6, 2026 01:26
Next to proof-three-window-mode-policy, the option it interprets, so
that other backends can use it without loading pg-response.el.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
coq-search-blacklist-string, coq-search-blacklist-current-string and
the commands coq-change-search-blacklist(-interactive) move to
coq-queries.el.  They call coq-set-search-blacklist-function, set by
the backend, with the old and new blacklists: Coq can only add or
remove strings, and a backend may keep the settings to replay before
each query.  Proof General still removes the old blacklist and adds
the new one, in coq-pg-set-search-blacklist.  The menu entry given in
the docstring of coq-search-blacklist-string is fixed.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Matafou
Matafou merged commit d8a41be into ProofGeneral:master Nov 6, 2026
140 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant