Skip to content
This repository was archived by the owner on Nov 6, 2022. It is now read-only.

Ensure to test SEARCH, PURGE and MKCALENDAR #241

Closed
wants to merge 2 commits into from
Closed

Conversation

Hywan
Copy link
Contributor

@Hywan Hywan commented Apr 24, 2015

These methods are defined in http_parser.h but not tested.

Hywan added 2 commits April 24, 2015 17:20
Be consistent with similar declarations and comments.
These methods are defined in `http_parser.h` but not tested.
@Hywan
Copy link
Contributor Author

Hywan commented Jul 15, 2015

ping?

case 'P': parser->method = HTTP_POST;
/* or PROPFIND|PROPPATCH|PUT|PATCH|PURGE */
break;
case 'P': parser->method = HTTP_POST; /* or PROPFIND, PROPPATCH, PUT, PATCH, PURGE */ break;

This comment was marked as off-topic.

This comment was marked as off-topic.

@indutny
Copy link
Member

indutny commented Jul 16, 2015

LGTM, with only one nit. Thank you!

indutny pushed a commit that referenced this pull request Jul 16, 2015
These methods are defined in `http_parser.h` but not tested.

PR-URL: #241
Reviewed-By: Fedor Indutny <[email protected]>
@indutny
Copy link
Member

indutny commented Jul 16, 2015

Landed in b5bcca8

@indutny indutny closed this Jul 16, 2015
@Hywan
Copy link
Contributor Author

Hywan commented Jul 16, 2015

Thanks :-]!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants