From 64f4a7c22d570097f316a5cbaf25b99a412bf54d Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Fri, 28 Aug 2020 10:12:59 +0000 Subject: imaptracker: update_last: simplify callers 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. --- lib/PublicInbox/WatchMaildir.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'lib/PublicInbox/WatchMaildir.pm') 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; } -- cgit v1.2.3-24-ge0c7