Skip to content

fix: store length for Twitch specials - #7220

Merged
pajlada merged 7 commits into
Chatterino:masterfrom
Nerixyz:feat/gifs-2-of-n
Sep 4, 2026
Merged

pajlada merged 7 commits into
Chatterino:masterfrom
Nerixyz:feat/gifs-2-of-n

Conversation

@Nerixyz

@Nerixyz Nerixyz commented Sep 2, 2026 •

Copy link
Copy Markdown
Contributor

Storing a length instead of an end makes more sense. We sometimes move the specials (e.g. in ignores). Both can be derived from each other.

This is part of a stack

  1. refactor: Generalize Twitch emote parsing #7219
  2. fix: store length for Twitch specials #7220 ← this one
  3. refactor: Message tokenization #7221
  4. feat: add Twitch gifs as links #7222
  5. feat: add Twitch gifs as images #7223

@pajlada

pajlada commented Sep 2, 2026

Copy link
Copy Markdown
Member

I think it would be clearer if we went for start and length instead of start and end, if we're going to be modifying this code.

@Nerixyz Nerixyz changed the title fix: use exclusive end for specials fix: store length for Twitch specials Sep 2, 2026

@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/MessageBuilder.cpp
@Nerixyz
Nerixyz marked this pull request as ready for review September 4, 2026 12:20
@pajlada
pajlada merged commit 1ea60d7 into Chatterino:master Sep 4, 2026
18 checks passed
VlaDexa pushed a commit to VlaDexa/chatterino7 that referenced this pull request Sep 9, 2026
Storing a length instead of an `end` makes more sense. We sometimes move
the specials (e.g. in ignores). Both can be derived from each other.

Reviewed-by: pajlada <rasmus.karlsson@pajlada.com>
Reviewed-by: Mm2PL <mm2pl+gh@kotmisia.pl>
babicekmarek pushed a commit to babicekmarek/streamally-chatterino that referenced this pull request Sep 14, 2026
Storing a length instead of an `end` makes more sense. We sometimes move
the specials (e.g. in ignores). Both can be derived from each other.

Reviewed-by: pajlada <rasmus.karlsson@pajlada.com>
Reviewed-by: Mm2PL <mm2pl+gh@kotmisia.pl>
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.

3 participants