ds: set event flags directly at initialization

[public-inbox.git] / lib / PublicInbox / EvCleanup.pm

diff --git a/lib/PublicInbox/EvCleanup.pm b/lib/PublicInbox/EvCleanup.pm
index afed24ffc965fe4f039a661daabdd5e26dba9dfd..d60ac2cc8c88d3de63288c2d51d517b796e65c1d 100644 (file)
--- a/lib/PublicInbox/EvCleanup.pm
+++ b/lib/PublicInbox/EvCleanup.pm
@@ -6,7 +6,6 @@ package PublicInbox::EvCleanup;
 use strict;
 use warnings;
 use base qw(PublicInbox::DS);
-use fields qw(rd);
 
 my $ENABLED;
 sub enabled { $ENABLED }
@@ -24,8 +23,13 @@ sub once_init () {
        # fires in the next event loop iteration.
        pipe($r, $w) or die "pipe: $!";
        fcntl($w, 1031, 4096) if $^O eq 'linux'; # 1031: F_SETPIPE_SZ
-       $self->SUPER::new($w);
-       $self->{rd} = $r; # never read, since we never write..
+       $self->SUPER::new($w, 0);
+
+       # always writable, since PublicInbox::EvCleanup::event_step
+       # never drains wbuf.  We can avoid wasting a hash slot by
+       # stuffing the read-end of the pipe into the never-to-be-touched
+       # wbuf
+       $self->{wbuf} = $r;
        $self;
 }
 
@@ -53,7 +57,7 @@ sub _run_later () {
 }
 
 # Called by PublicInbox::DS
-sub event_write {
+sub event_step {
        my ($self) = @_;
        $self->watch_write(0);
        _run_asap();