Skip to content

Correction de l'emplacement du meta avec l'encodage : Dans le head - #227

Closed
emmguyot wants to merge 7 commits into
xroche:masterfrom
emmguyot:master
Closed

emmguyot wants to merge 7 commits into
xroche:masterfrom
emmguyot:master

Conversation

@emmguyot

Copy link
Copy Markdown

No description provided.

@Neustradamus

Copy link
Copy Markdown

@xroche: Have you seen this PR?

@AnyOldName3

Copy link
Copy Markdown

This caused mitchcapper#26. I don't know if the monkey patch I came up with to avoid the symptom is the proper fix, but it didn't break anything that I noticed with the site I needed to archive.

@emmguyot

Copy link
Copy Markdown
Author

This caused mitchcapper#26. I don't know if the monkey patch I came up with to avoid the symptom is the proper fix, but it didn't break anything that I noticed with the site I needed to archive.

It is fixed with my last commit

@emmguyot

Copy link
Copy Markdown
Author

Interested in cleaning this PR (rebase / squash) for merging?

@xroche

xroche commented Jun 9, 2026

Copy link
Copy Markdown
Owner

Thanks, this targets a real FIXME, but I can't take it as-is: in the parser it reads past the buffer bound and makes unchecked cursor jumps, and it bundles an unrelated change with no test. Closing for now; a scoped, bounds-checked rewrite with a test would be very welcome.

@xroche xroche closed this Jun 9, 2026
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.

4 participants