search

tc39 / test262-parser-tests

tests designed for ECMA-262 parsers
53 stars 14 forks source link

Allow for-in initializers #21

Open bakkot opened 6 years ago

bakkot commented 6 years ago

This test is misclassified as of https://github.com/tc39/ecma262/pull/614.

littledan commented 6 years ago

For-in initializers? I thought ES2015 got rid of those. What idiot would put those back in?

(I was!)

leobalter commented 6 years ago

@littledan if that's a joke it is going without a context and it's not fun. I recommend avoiding this tone.

littledan commented 6 years ago

Yes, it was a joke since I was the one to reintroduce it.

jmdyck commented 6 years ago

Yeah, but if the reader doesn't know that, it looks to them like you're calling someone else an idiot. And maybe they think "Ah, it's okay to insult people here."

If, in the same post, you had revealed that "the idiot" was you, then everyone would have realized you were being self-deprecating, and (in my opinion) the joke wouldn't have been a problem.

littledan commented 6 years ago

OK, edited the original comment to include the reveal.