Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Issue #161#163

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
glynos merged 4 commits intocpp-netlib:masterfrommaxim-ky:master
Dec 8, 2012
Merged

Issue #161#163

glynos merged 4 commits intocpp-netlib:masterfrommaxim-ky:master
Dec 8, 2012

Conversation

@maxim-ky
Copy link

No description provided.

@ghostghost assignedglynosOct 30, 2012
@deanberris
Copy link
Member

I think I'll be a little more comfortable merging this if there was also a test making sure that this did what it's supposed to do. Glyn, please feel free to merge and add the test later if you think this is a reasonable implementation.

@glynos
Copy link
Member

Yes, please add a test case that should fail before the code fix and that succeeds after. You can put this test inside uri_test.cpp.

@maxim-ky
Copy link
Author

I added the test case and the fix with addition of typedef's in network::http::request_header/response_header because header concept requirements. Async http server can't be build without this fix.

@glynosglynos merged commit2f1dee8 intocpp-netlib:masterDec 8, 2012
deanberris added a commit that referenced this pull requestJan 20, 2014
deanberris added a commit that referenced this pull requestJan 20, 2014
leecoder pushed a commit to leecoder/cpp-netlib that referenced this pull requestApr 14, 2015
leecoder pushed a commit to leecoder/cpp-netlib that referenced this pull requestApr 14, 2015
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

@glynosglynos

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@maxim-ky@deanberris@glynos

[8]ページ先頭

©2009-2025 Movatter.jp