Diogo Benvenutti (2) [Avatar] Offline
#1
Hello there,

here is just a small suggestion that I would like to point out. On Listing 9.9. "Swap function for expected<T, E>", there is a alias to std::swap (using std::swap). But then on line 14 there is a call to std::swap (using std:: ), and then on line 21 there's a call without the std namespace prefix. Wouldn't it be better not to use the alias, so it clearly states the calls to std::swap? Also, since the name of the function is also swap, it can get a bit confusing.

Awesome book by the way! = )