Skip to content

interpreter: implement FIELDWIDTHS fixed-width splitting - #132

Open
Franklin-Qi wants to merge 1 commit into
uutils:mainfrom
Franklin-Qi:feature-issue#123-fieldwidths
Open

Franklin-Qi wants to merge 1 commit into
uutils:mainfrom
Franklin-Qi:feature-issue#123-fieldwidths

Conversation

@Franklin-Qi

Copy link
Copy Markdown
Collaborator

Replace the FIELDWIDTHS todo with gawk-compatible width/skip//skip: parsing and record splitting, plus regression tests for the manual cases.

Closes: #123

@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown

GNU awk testsuite comparison:

Test results comparison:
  Current:   TOTAL: 666 / PASSED: 96 / FAILED: 472 / SKIPPED: 98
  Reference: TOTAL: 666 / PASSED: 90 / FAILED: 478 / SKIPPED: 98

Changes from main branch:
  TOTAL: +0
  PASSED: +6
  FAILED: -6

Test improvements (6):
  + fieldwdth
  + fwtest
  + fwtest2
  + fwtest4
  + fwtest5
  + fwtest7

Comment thread interpreter/src/vm/symbols.rs
Comment thread interpreter/src/vm/symbols.rs Outdated
Comment thread interpreter/src/vm/symbols.rs Outdated
Comment thread interpreter/src/vm/symbols.rs Outdated
Comment thread tests/by-util/test_awk.rs
Comment thread interpreter/src/vm/symbols.rs
Parse FIELDWIDTHS once on assignment (tokenized blanks / skip:width),
cache designators for record splits, fatal on invalid specs, and match
gawk when a skip past EOF still yields an empty field (including *).

Closes: uutils#123
@Franklin-Qi
Franklin-Qi force-pushed the feature-issue#123-fieldwidths branch from b2ebd22 to e160a3f Compare October 10, 2026 10:12
@github-actions

Copy link
Copy Markdown

GNU awk testsuite comparison:

Test results comparison:
  Current:   TOTAL: 666 / PASSED: 96 / FAILED: 472 / SKIPPED: 98
  Reference: TOTAL: 666 / PASSED: 90 / FAILED: 478 / SKIPPED: 98

Changes from main branch:
  TOTAL: +0
  PASSED: +6
  FAILED: -6

Test improvements (6):
  + fieldwdth
  + fwtest
  + fwtest2
  + fwtest4
  + fwtest5
  + fwtest7

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.

Missing FIELDWIDTHS splitting mode

2 participants