Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Get
console
messages to go code #77base: master
Are you sure you want to change the base?
Get
console
messages to go code #77Changes from 18 commits
7a4ddfa
e83f0c5
68bccb5
18242ac
0a33595
39d6d17
b7fe7b0
5afe486
df8b531
50b0c4b
4f4f6e7
186f8fa
873d919
46df38e
de2cf3d
ed8be0f
f336ba1
fd8611b
c6890af
f231768
0b5ff86
21dec07
abd4f7c
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add some documentation for the exported functions and types? Ideally for someone who hasn't (yet) read the C++ docs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume, you mean the "exported to C" code function?
goHandleConsoleAPIMessageCallback
? Done!There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be great to take the values from the underlying enum, but at least document this is a copy of v8::Isolate::MessageErrorLevel.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point. For the values, I adapted how e.g.,
PropertyAttribute
is also implemented.I think even linking to v8 documentation would make sense, but this doesn't look like a stable URL>
https://v8.github.io/api/head/classv8_1_1Isolate.html#acfc7c4d5c93913201249045e2655d3dd
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed, though https://v8.github.io/api/head/classv8_1_1Isolate.html is probably useful on its own.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done!