Skip to content

Limit the encoding detection to 3 bytes for now. #199

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Feb 23, 2016
Merged

Conversation

Ms2ger
Copy link
Contributor

@Ms2ger Ms2ger commented Feb 23, 2016

This is intended to avoid issues when there's a parser-blocking script in the first 1024 bytes. See also #198.

Review on Reviewable

This is intended to avoid issues when there's a parser-blocking script in the first 1024 bytes. See also #198.
@SimonSapin
Copy link
Member

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit 199a84a has been approved by SimonSapin

@bors-servo
Copy link
Contributor

⌛ Testing commit 199a84a with merge 40d49d5...

bors-servo pushed a commit that referenced this pull request Feb 23, 2016
Limit the encoding detection to 3 bytes for now.

This is intended to avoid issues when there's a parser-blocking script in the first 1024 bytes. See also #198.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.svg" height="40" alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/html5ever/199)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

☀️ Test successful - travis

@bors-servo bors-servo merged commit 199a84a into master Feb 23, 2016
@SimonSapin SimonSapin deleted the prescan branch February 23, 2016 17:19
bors-servo pushed a commit that referenced this pull request Jul 25, 2018
Add more documentation

I recently submitted a PR to add documentation. This PR adds some more.

Note it won't merge as-is because it relies on PR [#199 in servo/string-cache].

[#199 in servo/string-cache]: servo/string-cache#199
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.

3 participants