about summary refs log tree commit homepage
path: root/lib/PublicInbox/WatchMaildir.pm
diff options
context:
space:
mode:
authorEric Wong <e@yhbt.net>2020-08-28 10:12:59 +0000
committerEric Wong <e@yhbt.net>2020-08-28 22:19:27 +0000
commit64f4a7c22d570097f316a5cbaf25b99a412bf54d (patch)
tree8532835e0223d28982db12836f6a5bac452fb92c /lib/PublicInbox/WatchMaildir.pm
parentb1054300d1e33cb7b2279922c8f73628551fbf8f (diff)
downloadpublic-inbox-64f4a7c22d570097f316a5cbaf25b99a412bf54d.tar.gz
By making it a no-op if last_uid is not defined.  This isn't a
hot code path, so the extra method dispatch isn't an issue.
It'll save some indentation/wrapping in future commits.
Diffstat (limited to 'lib/PublicInbox/WatchMaildir.pm')
-rw-r--r--lib/PublicInbox/WatchMaildir.pm4
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/PublicInbox/WatchMaildir.pm b/lib/PublicInbox/WatchMaildir.pm
index 78aec8a2..a227a6fd 100644
--- a/lib/PublicInbox/WatchMaildir.pm
+++ b/lib/PublicInbox/WatchMaildir.pm
@@ -494,7 +494,7 @@ sub imap_fetch_all ($$$) {
                         last if $self->{quit};
                 }
                 _done_for_now($self);
-                $itrk->update_last($r_uidval, $last_uid) if defined $last_uid;
+                $itrk->update_last($r_uidval, $last_uid);
         } until ($err || $self->{quit});
         $err;
 }
@@ -919,7 +919,7 @@ sub nntp_fetch_all ($$$) {
                 $last_art = $art;
         }
         _done_for_now($self);
-        $itrk->update_last(0, $last_art) if defined $last_art;
+        $itrk->update_last(0, $last_art);
         $err;
 }