Skip to content

Conversation

nevkontakte
Copy link
Member

The vast majority of the code (even in errors.go) is not used by
GopherJS, so starting with this commit the package is entirely defined
by our own implementation in natives. As a bonus, this allowed to drop
a few unexported symbols that existed purely to let other unused
functions to compile.

Updates #989

The vast majority of the code (even in errors.go) is not used by
GopherJS, so starting with this commit the package is entirely defined
by our own implementation in natives. As a bonus, this allowed to drop
a few unexported symbols that existed purely to let other unused
functions to compile.
@nevkontakte nevkontakte requested a review from flimzy February 28, 2021 18:22
Copy link
Member

@flimzy flimzy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. A couple argument variables can optionally be omitted, if we wish, but totally unimportant.

Co-authored-by: Jonathan Hall <flimzy@flimzy.com>
@nevkontakte nevkontakte merged commit e1112ef into gopherjs:go1.16-stdlib Mar 7, 2021
@nevkontakte nevkontakte deleted the go1.16-stdlib-runtime branch March 7, 2021 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants