- unused alias - inefficient use of push_back - exceptions not inheriting from std::exception - weird use of a comma - value compared against itself