ljharb / es-abstract

ECMAScript spec abstract operations.
MIT License
114 stars 30 forks source link

TypeError or $TypeError ? #107

Closed motla closed 4 years ago

motla commented 4 years ago

Hi!

Line 36 should probably rather be new $TypeError like on line 43.

https://github.com/ljharb/es-abstract/blob/599961920d3b977df94527601772e19cee1474fe/2019/ToPropertyDescriptor.js#L33-L46

It appears in every ToPropertyDescriptor.js, and also you can search the project for "new TypeError", it appears without the $ in files NumberToString.js and CopyDataProperties.js as well.

ljharb commented 4 years ago

Thanks, will be fixed in the next version :-)