NotificationsYou must be signed in to change notification settings
Fork425
Star2k

Commitdd69300

committed

Remove user-declared constructors for response class

Removed the following defaulted constructors:* Default Constructor* Copy Constructor* Move ConstructorThe defaulted move constructor prevents implicit declaration ofthe copy assignment operator, which results in compiler errorssince std::promise requires it.All 3 user-declared methods removed here are implicitly declared bythe compiler anyway, so no need to default them in the first place.Relates to#547

1 parent769e5c6 commitdd69300Copy full SHA for dd69300

File tree

1 file changed

-17

lines changed

http/src/network/http/v2/client
- response.hpp

1 file changed

-17

lines changed

`‎http/src/network/http/v2/client/response.hpp`

Lines changed: 0 additions & 17 deletions

Original file line number	Diff line number	Diff line change
`@@ -60,23 +60,6 @@ class response {`
`60`	`60`	`*/`
`61`	`61`	`typedef headers_type::const_iterator const_headers_iterator;`
`62`	`62`
`63`		`-/**`
`64`		`- * \brief Constructor.`
`65`		`-*/`
`66`		`-response() =default;`
`67`		`-`
`68`		`-/**`
`69`		`- * \brief Copy constructor.`
`70`		`- * \param other The other response object.`
`71`		`-*/`
`72`		`-response(const response &other) =default;`
`73`		`-`
`74`		`-/**`
`75`		`- * \brief Move constructor.`
`76`		`- * \param other The other response object.`
`77`		`-*/`
`78`		`-response(response &&other)noexcept =default;`
`79`		`-`
`80`	`63`	`/**`
`81`	`64`	`* \brief Swap function.`
`82`	`65`	`* \param other The other response object.`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

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

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitdd69300

File tree

1 file changed

1 file changed

`‎http/src/network/http/v2/client/response.hpp`

0 commit comments