I am dealing with a pretty big codebase and I was given a few months to refactor existing code. The refactor process is needed because soon we will need to add many new features to our product and as for now we are no longer able to add any feature without breaking something else. In short: messy, huge, buggy code, that many of us have seen in their careers.
During refactoring, from time to time I encounter the class, method or lines of code which have comments like
Time out set to give Module A some time to do stuff. If its not timed like this, it will break.
Do not change this. Trust me, you will break things.
I know using setTimeout is not a good practice, but in this case I had to use it
My question is: should I refactor the code when I encounter such warnings from the authors (no, I can't get in touch with authors)?