Outdated comment in timers

[benjamingr]

Currently, the timers contain an outdated comment in https://github.com/nodejs/node/blob/master/lib/timers.js#L742 referring to a try/catch statement that is not there.

We should remove it.

[timcosta]

I'm on this.

[targos]

It's still there. The comment should just say try/finally instead of try/catch

[timcosta]

PR updated

[benjamingr]

@targos yes, but the "allow it to be optimized" part is no longer true with TurboFan

[targos]

True, but that's not the reason you gave in the OP. I still think the comment is relevant because it explains why the call is made in another function. If that is not relevant anymore, we shouldn't just remove the comment but revert the optimization altogether.

[Fishrock123]

This comment is intentional in it's current position: 6f75b66#diff-0a5d4868b2b9b17cf9e2c11f1bd1311eR545

(And also correct IIRC)

EDIT: ok maybe not with TurboFan

[sam0410]

Hi !
Can I make the PR for this?
Thanks

[BridgeAR]

This got fixed by a2b6872