FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Normalize obsolete folded header whitespace by AG0708 · Pull Request #198 · python-hyper/h11 · GitHub

Normalize obsolete folded header whitespace - #198

Open
AG0708 wants to merge 1 commit into
python-hyper:masterfrom
AG0708:codex/strip-obs-fold-ows
Open

Normalize obsolete folded header whitespace#198
AG0708 wants to merge 1 commit into
python-hyper:masterfrom
AG0708:codex/strip-obs-fold-ows

Conversation

AG0708 commented Aug 12, 2026

Copy link
Copy Markdown

Summary

  • Treat optional whitespace immediately before an obsolete folded line as part of the fold.
  • Replace each complete fold with one space, including consecutive folds.
  • Cover whole-buffer, incremental, and trailing-data parsing through the existing reader test harness.

This follows the corrected obs-fold = OWS CRLF RWS interpretation discussed in the issue.

Closes #59.

Validation

  • Full test suite: 79 passed.
  • Changed reader retains 100% statement and branch coverage.
  • Mypy, Black, isort, and Git whitespace checks passed.
  • Added the required towncrier fragment.

AI assistance is disclosed in the commit trailer.

Strip optional whitespace before each continuation before inserting the single replacement space. Add a regression covering consecutive folds and incremental parsing.

Signed-off-by: Abhinav Gorrepati <gorrepatiabhinav1@gmail.com>
Co-authored-by: OpenAI Codex <noreply@openai.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

Tiny maybe-bug in obsolete line folding handling

1 participant


Back | FazBrowse Home | New Git URL