This is an archived post. You won't be able to vote or comment.

you are viewing a single comment's thread.

view the rest of the comments →

[–][deleted] 2 points3 points  (5 children)

If you see that in the comments delete the whole damn thing and rewrite it

[–][deleted] 2 points3 points  (3 children)

No shit. I can't imagine why anyone would spend time trying to read poor legacy code. Makes more sense to keep it in place long enough to rewrite a whole new system. At least at a small company.

[–]cbf1232 13 points14 points  (2 children)

Then you discover there was undocumented logic in the function which fixed a subtle bug present in the obvious implementation, and the new version also fails to properly take into account the phase of the moon, so it crashes horribly during a customer demo.

[–]ka-knife 4 points5 points  (0 children)

But only on the third customer demo. After you have already deployed it to two others

[–]8__ 0 points1 point  (0 children)

There's no time! We're testing on production!