Your comments come across as pretty mean-spirited. Let me rewrite one line for you. You wrote:

> Someday one of your coworkers is going to break it and this code will be trashing the heap. Mysterious memory corruption failures are your destiny.

You could have written something like, "It's possible this code will break in the future and you won't even realize the cause, because it will appear as a mysterious memory corruption failure."

See how I left out a word like "destiny" and took out the whole bit about coworkers? Yeah, much nicer and less personalized.