Assignment operators should return non-const references, according to cppcheck