So, uh, weird professional question here.
Coworker posts a pull request for a huge thing he's been working on, has 2,500 new lines of code. I'm perusing through it and, in a C# file, randomly come across a comment at the end of a line:
Code:
private void OnLoad() {
cell.onSelected -= HandleSelect; // i hate myself and want to die
cell.onSelected += HandleSelect;
So, that's pretty ****ing dark. While to a certain extent I understand that we've all written code we felt dirty about and perhaps left some self-deprecating comments behind to shame ourselves for our misdeeds, that seems slightly overboard. The coworker in question has only been here a few months, is a little on the quiet side maybe, I don't know him very well. He's also not a native English speaker if that matters.
What do I do? Doing nothing seems reckless and broaching the topic with him directly seems super awkward, and he'd probably brush it off or something, and I may even be overreacting by thinking it's a big deal, I don't know. It's certainly a little bit of a creepy thing to have lurking in our code if anyone else ever comes across it too.
(and no, I don't know the purpose of the commented line, I guess it's a guard against subscribing to that event multiple times but that doesn't seem necessary given that method is only called on initialization)