Skip to content

parser: implement {% raw %} and Lit::parse() without skip_till#489

Merged
GuillaumeGomez merged 3 commits intoaskama-rs:masterfrom
Kijewski:pr-no-memchr
Jun 17, 2025
Merged

parser: implement {% raw %} and Lit::parse() without skip_till#489
GuillaumeGomez merged 3 commits intoaskama-rs:masterfrom
Kijewski:pr-no-memchr

Conversation

@Kijewski
Copy link
Copy Markdown
Member

The implementation is longer than before, but also more correct: Before, the input {% raw %}{% endraw }}{% endraw %} would be a syntax error, now it renders the string {% endraw }}.

Kijewski added 3 commits June 17, 2025 01:01
The implementation is longer than before, but also more correct:
Before, the input `{% raw %}{% endraw }}{% endraw %}` would be a syntax
error, now it renders the string `{% endraw }}`.
@Kijewski Kijewski changed the title parser: implement {% raw %} without skip_till parser: implement {% raw %} and Lit::parse() without skip_till Jun 17, 2025
@GuillaumeGomez
Copy link
Copy Markdown
Collaborator

One less dependency, yeay! \o/

@GuillaumeGomez GuillaumeGomez merged commit 40bb5da into askama-rs:master Jun 17, 2025
42 checks passed
@Kijewski
Copy link
Copy Markdown
Member Author

The dependency is still there, just transitively now. :) I had a look: Our implementation was needed with nom, only winnow made it possible to scan for multiple literals in skip_until.

@Kijewski Kijewski deleted the pr-no-memchr branch June 17, 2025 09:31
@Kijewski Kijewski mentioned this pull request Jun 17, 2025
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