Serveral users complain that the error messages our software produces are completely useless, so I get assigned to improve the error reporting.

Starting the task by checking what's there and how it works together.

Looking through the code I discover 4 completely different error reporting implementations, each having the same intent and promised functionality. All of them are used somewhere, sometimes next to each other. Needless to say: None of the implementations actually implements all it promises.

My favourite part is the setErrorMessage(string) method that is defined in one header and used in many places, which has the implementation

void setErrorMessage(string){}

Fuck my life.