Skip to content

fix: recognize all ASCII whitespace - #601

Merged
LunaStev merged 1 commit into
wavefnd:masterfrom
nightcityblade:fix/issue-591
Sep 12, 2026
Merged

fix: recognize all ASCII whitespace#601
LunaStev merged 1 commit into
wavefnd:masterfrom
nightcityblade:fix/issue-591

Conversation

@nightcityblade

Copy link
Copy Markdown
Contributor

Summary

Complete ASCII whitespace classification with vertical tab and form feed, and reuse it from trimming.

Motivation

Fixes #591.

Target and compatibility impact

All targets now consistently recognize the six ASCII whitespace bytes.

Validation

  • target/debug/wavec check tests/cases/shared/test81.wave
  • git diff --check
  • The full run was attempted but this macOS environment lacks ld64.lld.

Checklist

  • Commits include a DCO Signed-off-by line.
  • Tests cover new behavior or the PR explains why no test is needed.
  • User-facing changes include documentation or diagnostics updates.
  • The change preserves the license boundary between the compiler and std/.

Signed-off-by: nightcityblade <nightcityblade@gmail.com>
@LunaStev

Copy link
Copy Markdown
Member

LGTM thanks

@LunaStev
LunaStev merged commit 1800787 into wavefnd:master Sep 12, 2026
24 of 26 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.

Recognize vertical tab and form feed as ASCII whitespace

2 participants