Skip to content

messageelement: allow for exhaustive flag test - #7192

Merged
pajlada merged 5 commits into
masterfrom
feat/messageelement-exhaustive-match
Aug 23, 2026
Merged

pajlada merged 5 commits into
masterfrom
feat/messageelement-exhaustive-match

Conversation

@pajlada

@pajlada pajlada commented Aug 22, 2026

Copy link
Copy Markdown
Member

this allows most elements to be configured with the "exhaustiveFlags"
flag which ensures that all of the element's flags must be set in the
context for it to be laid out.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

clang-tidy made some suggestions

Comment thread src/messages/MessageElement.cpp Outdated
this allows most elements to be configured with the "exhaustiveFlags"
flag which ensures that _all_ of the element's flags must be set in the
context for it to be laid out.
@pajlada
pajlada force-pushed the feat/messageelement-exhaustive-match branch from f8a415f to 98bb7ad Compare August 23, 2026 10:03
@pajlada
pajlada marked this pull request as ready for review August 23, 2026 10:03
@pajlada
pajlada enabled auto-merge (squash) August 23, 2026 10:11
Comment thread src/messages/MessageElement.cpp
@pajlada
pajlada disabled auto-merge August 23, 2026 10:55
@pajlada
pajlada merged commit 036f2c1 into master Aug 23, 2026
18 checks passed
@pajlada
pajlada deleted the feat/messageelement-exhaustive-match branch August 23, 2026 10:55
invisibl3cat pushed a commit to invisibl3cat/chatterino2 that referenced this pull request Aug 23, 2026
this allows most elements to be configured with the "exhaustiveFlags"
flag which ensures that _all_ of the element's flags must be set in the
context for it to be laid out.
babicekmarek pushed a commit to babicekmarek/streamally-chatterino that referenced this pull request Sep 14, 2026
this allows most elements to be configured with the "exhaustiveFlags"
flag which ensures that _all_ of the element's flags must be set in the
context for it to be laid out.
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