would you write this
auto x = *it;
// modify it
if(error)
*it = x;

instead of this?
auto x = *it;
// modify x
if(ok)
*it = x;

#cpp

One of the severe problems with C++ (>C++98) is that it teaches horrible behaviors like the use of "auto"... which really translates to:"I have no idea what this is!".

@namark

@Firaas I take it your prefer
typename std::iterator_traits<typename std::remove_referece<It>::type>::value_type