Skip to content
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

feat(headers): Add If-Range header #491

Merged
merged 1 commit into from
Apr 30, 2015
Merged

feat(headers): Add If-Range header #491

merged 1 commit into from
Apr 30, 2015

Conversation

pyfisch
Copy link
Contributor

@pyfisch pyfisch commented Apr 30, 2015

Closes #388

cc @jdm

@@ -286,6 +289,38 @@ macro_rules! header {
$($tf)*
}
};

// Enum header
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this supposed to be part of this PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No.

@pyfisch
Copy link
Contributor Author

pyfisch commented Apr 30, 2015

Updated pr, enum headers do not work as macro, so removed the code.

seanmonstar added a commit that referenced this pull request Apr 30, 2015
feat(headers): Add If-Range header
@seanmonstar seanmonstar merged commit 1426a4c into hyperium:master Apr 30, 2015
@pyfisch pyfisch deleted the ifrangeheader branch May 1, 2015 03:59
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.

Implement If-Range header parsing
2 participants