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
net: enable File method on Windows #3827
Labels
Comments
I think you are wrong about DuplicateHandle. It succeeds for a socket in a small program I wrote. But, I think, our argument is pointless. If you are reporting a bug, then, please, provide a reproducible example. If you are asking for a new feature, then I am not clear about what you are after. If you want new SetTTL function in net, then https://golang.org/issue/3826 should be enough. If you just must have access to socket handle, then you should say so. I do not see what having "implement dup for netFD on Windows" will do for you. Alex Status changed to WaitingForReply. |
I provided some implementation details because I discovered it wasn't quite straightforward. Naturally it should be decided by someone who submits a patch; they're just notes. The data hiding is a *real* PITA, especially as the language is new, many things are hidden so that rewriting or patching entire core libraries is necessary to expose things. That's a separate complaint. It would be great if the socket handle was just exposed (and the dangers of abusing it made clear in the documentation, naturally). I don't really want to duplicate it; my end goal is to get at a `syscall.Handle` for the socket to set the TTL. |
File method on Conn/PacketConn already exists on Unix variants. I'm happy with this method, it would be help for fixing issue #3684. Dear Mr. Brainman: Do you have a time to review http://golang.org/cl/6445070/ ? |
Status changed to Duplicate. Merged into issue #3684. |
This issue was closed.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
The text was updated successfully, but these errors were encountered: