]> Sergey Matveev's repositories - public-inbox.git/commitdiff
initialize scalar for `vec' perlop modification
authorEric Wong <e@yhbt.net>
Sun, 17 Jan 2021 07:09:56 +0000 (07:09 +0000)
committerEric Wong <e@80x24.org>
Mon, 18 Jan 2021 09:25:10 +0000 (09:25 +0000)
Older Perls (tested 5.16.3) would warn on uninitialized scalars while
newer (tested 5.28.1) do not.  Just initialize it to an empty string
since it'll be filled in by `vec'.

lib/PublicInbox/LEI.pm
lib/PublicInbox/Syscall.pm

index 1f4a30829a917d734c8ca2cb46abcba2bfa3073b..2784ca6b85d26936aac16aa24dcc96e43b456c39 100644 (file)
@@ -670,7 +670,7 @@ sub accept_dispatch { # Listener {post_accept} callback
        my ($sock) = @_; # ignore other
        $sock->autoflush(1);
        my $self = bless { sock => $sock }, __PACKAGE__;
-       vec(my $rvec, fileno($sock), 1) = 1;
+       vec(my $rvec = '', fileno($sock), 1) = 1;
        select($rvec, undef, undef, 1) or
                return send($sock, 'timed out waiting to recv FDs', MSG_EOR);
        my @fds = $recv_cmd->($sock, my $buf, 4096 * 33); # >MAX_ARG_STRLEN
index a1f532353eab391020be7a94710433839fc2d278..5ff1d65fc9365df5c2ee022df19395e3e7d59970 100644 (file)
@@ -224,7 +224,7 @@ sub epoll_ctl_mod8 {
 # epoll_wait wrapper
 # ARGS: (epfd, maxevents, timeout (milliseconds), arrayref)
 #  arrayref: values modified to be [$fd, $event]
-our $epoll_wait_events;
+our $epoll_wait_events = '';
 our $epoll_wait_size = 0;
 sub epoll_wait_mod4 {
        my ($epfd, $maxevents, $timeout_msec, $events) = @_;