-
Notifications
You must be signed in to change notification settings - Fork 2
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
edit original post #2
Conversation
@@ -22,87 +31,77 @@ func main() { | |||
} | |||
``` | |||
|
|||
Looks about as innocuous as nonsensical, right? If we run it naively, nothing happens: | |||
Looks about as innocuous as nonsensical, right? If we run it naively, nothing |
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.
s/as nonsensical/as it is nonsensical/g
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.
On Tue, Dec 15, 2015 at 11:15 AM Peter Mattis notifications@github.com
wrote:
In cgo_libc_static/README.md
#2 (comment):@@ -22,87 +31,77 @@ func main() {
}-Looks about as innocuous as nonsensical, right? If we run it naively, nothing happens: +Looks about as innocuous as nonsensical, right? If we run it naively, nothing
s/as nonsensical/as it is nonsensical/g
—
Reply to this email directly or view it on GitHub
https://github.com/tschottdorf/goplay/pull/2/files#r47658032.
LGTM |
@@ -111,26 +110,27 @@ fatal error: unexpected signal during runtime execution | |||
goroutine 3890 [syscall, locked to thread]: | |||
runtime.cgocall(0x44c7f0, 0xc82036a8d8, 0xc800000000) | |||
/usr/local/go/src/runtime/cgocall.go:120 +0x11b fp=0xc82036a888 sp=0xc82036a858 | |||
os/user._Cfunc_mygetpwuid_r(0x0, 0xc8203a8390, 0x7f3c5c000a10, 0x400, 0xc8200e4058, 0x7f3c00000000) |
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 wouldn't remove this, this is the symbol you'd actually find in the symbol table
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.
hmm, good point. Reinstating.
hopefully making it better.