From 96671974e110760964f630237113e674e0f00e1d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Dag-Erling=20Sm=C3=B8rgrav?= Date: Mon, 15 Apr 2002 06:36:22 +0000 Subject: [PATCH] Long-forgotten patch: warn if a closed PR has no "closed by" --- tools/tools/prstats/prstats.pl | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tools/tools/prstats/prstats.pl b/tools/tools/prstats/prstats.pl index f3a4d0ed632d..bb437a6bbc0b 100644 --- a/tools/tools/prstats/prstats.pl +++ b/tools/tools/prstats/prstats.pl @@ -124,6 +124,10 @@ sub scan_pr($) { if ($pr->{'state'} eq 'closed') { $pr->{'_closed'} = $pr->{'closed-date'} || $pr->{'state-changed-when'}; $pr->{'_closed_by'} = $pr->{'state-changed-by'}; + if (!defined($pr->{'_closed_by'})) { + warn("PR $pr->{'category'}/$pr->{'number'} is incomplete\n"); + return; + } ++$CLOSER{$pr->{'_closed_by'}}; } else { $age = $pr->{'arrival-date'} / 86400;