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

Response needs a default constructor #40

Closed
whoshuu opened this issue Sep 5, 2015 · 0 comments
Closed

Response needs a default constructor #40

whoshuu opened this issue Sep 5, 2015 · 0 comments
Milestone

Comments

@whoshuu
Copy link
Contributor

whoshuu commented Sep 5, 2015

MSVC complains (see #38) that Response has no default constructor when building the tests that use Response inside of a std::future. The fix is probably as simple as adding this:

Response() = default;

to the constructor list.

References #26

@whoshuu whoshuu added the Easy label Sep 5, 2015
@whoshuu whoshuu added this to the 1.2 milestone Sep 5, 2015
@whoshuu whoshuu closed this as completed in 1747a95 Sep 6, 2015
@whoshuu whoshuu mentioned this issue Sep 8, 2015
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

No branches or pull requests

1 participant