Unused variables (I have left a couple of these in, as they are used in commented out code, so may be wanted still)
Brackets around sub-object inits
return std::move preventing copy elision (I can't claim to 100% understand this fix, but based on what research I've done, it's better not to return std::move. Feel free to change this back if you feel otherwise)
Including but not limited to:
sf::RenderTexture::create(unsigned int, unsigned int, bool). Replaced withcreate(unsigned int, unsigned int, sf::ContextSettings)std::movepreventing copy elision (I can't claim to 100% understand this fix, but based on what research I've done, it's better not to returnstd::move. Feel free to change this back if you feel otherwise)