Struct 'TryClean' has a constructor with 1 argument that is not explicit. Such constructors should in general be explicit for type safety reasons. Using the explicit keyword in the constructor means some mistakes when using the class can be avoided.

Local variable 'data' shadows outer argument

Consider using std::find_if algorithm instead of a raw loop.

Local variable 'cleanJob' shadows outer variable

Local variable 'rhi' shadows outer variable

Variable 'rhi' is assigned a value that is never used.

Local variable 'texture' shadows outer variable

Local variable 'rhi' shadows outer variable

Local variable 'image' shadows outer variable