Skip to content

/ servo

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

Jump to bottom

Use inclusive rust ranges in Fetch file code #25751

Closed
gterzian opened this issue Feb 13, 2020 · 1 comment
Closed

Use inclusive rust ranges in Fetch file code #25751

gterzian opened this issue Feb 13, 2020 · 1 comment
Labels
A-network I-cleanup

Comments

@gterzian
Copy link
Member

@gterzian gterzian commented Feb 13, 2020 •
edited

you can use inclusive rust ranges here if we want to

Originally posted by @Manishearth in #25724

See

servo/components/net/filemanager_thread.rs

Line 203 in 19a06d1

// HTTP Range requests are specified with closed ranges,

@gterzian gterzian mentioned this issue Feb 13, 2020
Merged
0 of 5 tasks complete
@Manishearth
Copy link
Member

@Manishearth Manishearth commented Feb 14, 2020

Actually given that the other branches shouldn't use inclusive ranges, I think we can just not do this.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-network I-cleanup
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@jdm @Manishearth @gterzian
You can’t perform that action at this time.