diff options
author | normal <normal@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2015-06-05 22:21:47 +0000 |
---|---|---|
committer | normal <normal@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2015-06-05 22:21:47 +0000 |
commit | 5a67d8e2e9cac2bdd41c588fc13e041759028ce0 (patch) | |
tree | 06f501c425bafe2187d5f7665c6cea5adee71724 | |
parent | 6f586ac405ece968b7e85c88075399cdfb7c47a1 (diff) | |
download | ruby-5a67d8e2e9cac2bdd41c588fc13e041759028ce0.tar.gz |
* ext/socket/ancdata.c (bsock_sendmsg_internal): drop redundant assignment
We call memset to zero `mh' a few lines above anyways, making
explicitly zero-ing each field unnecessary.
Compilers optimize this code away anyways, but it still costs me
4 lines of my small terminal and even smaller attention span.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@50786 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 4 | ||||
-rw-r--r-- | ext/socket/ancdata.c | 4 |
2 files changed, 4 insertions, 4 deletions
@@ -1,3 +1,7 @@ +Sat Jun 6 07:08:45 2015 Eric Wong <e@80x24.org> + + * ext/socket/ancdata.c (bsock_sendmsg_internal): drop redundant assignment + Sat Jun 6 01:00:06 2015 Naohisa Goto <ngotogenome@gmail.com> * ext/socket/ancdata.c (bsock_sendmsg_internal): all arguments are diff --git a/ext/socket/ancdata.c b/ext/socket/ancdata.c index 756cf7ed64..6e59694723 100644 --- a/ext/socket/ancdata.c +++ b/ext/socket/ancdata.c @@ -1268,10 +1268,6 @@ bsock_sendmsg_internal(int argc, VALUE *argv, VALUE sock, int nonblock) mh.msg_control = RSTRING_PTR(controls_str); mh.msg_controllen = RSTRING_SOCKLEN(controls_str); } - else { - mh.msg_control = NULL; - mh.msg_controllen = 0; - } #endif rb_io_check_closed(fptr); |