Skip to content

Conversation

@GlenTiki
Copy link
Contributor

@GlenTiki GlenTiki commented Feb 2, 2015

Removed DTRACE_NET_SOCKET_READ and DTRACE_NET_SOCKET_WRITE
as they were never called from within the source code.

Removed DTRACE_NET_SOCKET_READ and DTRACE_NET_SOCKET_WRITE
as they were never called from within the source code.
@GlenTiki
Copy link
Contributor Author

GlenTiki commented Feb 2, 2015

This will need to be tested by someone who can build with dtrace enabled, please.

This resolves #692

bnoordhuis pushed a commit that referenced this pull request Feb 2, 2015
Removed DTRACE_NET_SOCKET_READ and DTRACE_NET_SOCKET_WRITE
as they were never called from within the source code.

PR-URL: #694
Reviewed-By: Ben Noordhuis <[email protected]>
@bnoordhuis
Copy link
Member

Thanks Glen, LGTM and I can confirm that --with-dtrace still works on OS X. Landed in d75fecf with an amended Author field (used your full name.) Thanks again!

@bnoordhuis bnoordhuis closed this Feb 2, 2015
@GlenTiki
Copy link
Contributor Author

GlenTiki commented Feb 2, 2015

Awesome! Happy to help. :)

@rvagg
Copy link
Member

rvagg commented Feb 3, 2015

pls see #698

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.

3 participants