Skip to content

Conversation

@matsl
Copy link
Collaborator

@matsl matsl commented Feb 9, 2026

What

Verify highlighting range is in sync with WikiWord range.

Why

Extend verification to verify that the range of the WikiWord and the
range of the highlighted text is equal iff a highlighted WikiWord
exists.

Note

This is a rework of the proposed change in #850 regarding doing a side effect verification of the ranges. The functionality apart from that is the same.

@matsl matsl force-pushed the verify_range_of_highlighting branch from c67b686 to 82f3432 Compare February 9, 2026 22:37
(hywiki-word-at))))
"Return potential HyWikiWord and optional #section:Lnum:Cnum at point or nil.
When `hywiki-tests--with-face-test' is non-nil the HyWikiWord must be
highlighted to be returned. When it is highlighted the range of the
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add commas in this and next lines.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add commas in this and next lines.

I don't know where to add the comma in the second line. Feel free to update that separately or advice me and I'll do it in another update.

* test/hywiki-tests.el (hywiki-tests--word-at): Extend verification to
    verify that the range of the WikiWord and the range of the
    highlighted text is equal iff a highlighted WikiWord exists.
@matsl matsl force-pushed the verify_range_of_highlighting branch from 82f3432 to 6773d56 Compare February 10, 2026 15:43
@matsl matsl merged commit b032531 into master Feb 10, 2026
4 checks passed
@matsl matsl deleted the verify_range_of_highlighting branch February 10, 2026 15:59
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.

2 participants