| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
There was a problem hiding this comment.
Instead of deprecating it, could we just alias it to Function.prototype.call.bind(String.prototype.codePointAt) and note that it's there for legacy reasons in the docs?
Sorry, something went wrong.
There was a problem hiding this comment.
it is actually undocumented and is not supposed to be used anyway. Unfortunately we have a lot of private not underscored stuff
Sorry, something went wrong.
There was a problem hiding this comment.
If it's undocumented, isn't is subject to disappearing at any time? (I realize someone could be using it)
Sorry, something went wrong.
There was a problem hiding this comment.
FWIW, I agree with @vkurchatkin. I believe the reason it was exported was so it could be unit-tested in the test suite.
Sorry, something went wrong.
There was a problem hiding this comment.
I guess that would a bit irresponsible. People use repl to discover stuff. We need a roadmap to avoid these in the future
Sorry, something went wrong.
There was a problem hiding this comment.
Yes people use the repl and other reads the code to find and use undocumented apis. I've done it myself, but I do not expect that code to work between versions.
Sorry, something went wrong.
Semver: patch PR-URL: #825 Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>
| Back | FazBrowse Home | New Git URL |
R=have no idea