Re: [elpa] master 1a6a4a3: Search further if d-g-cur-status is nil (Bug#35121)

[Michael Albinus]

address@hidden (Noam Postavsky)
writes:

Hi Noam,

> --- a/packages/debbugs/debbugs-gnu.el
> +++ b/packages/debbugs/debbugs-gnu.el
> @@ -1592,7 +1592,9 @@ removed instead."
>             current-prefix-arg
>             (when (derived-mode-p 'message-mode)
>               (current-buffer)))))
> -  (let* ((status (debbugs-gnu-current-status))
> +  (let* ((status (or (debbugs-gnu-current-status)
> +                     (gethash bugid debbugs-cache-data)
> +                     (debbugs-get-status bugid)))
>           (version
>            (if (and
>                 (member message '("close" "done"

Yesterday evening I was a little bit tired, so I didn't check precisely.

`debbugs-get-status' consults the cache first, so there is no reason to
call `gethash' explicitly. Furthermore, `debbugs-get-status' checks
whether cache entries have been expired, and refuses them.

That's why `gethash' isn't called anywhere else in debbugs-gnu.el; it is
an implementation detail of debbugs.el we shouldn't care.

Best regards, Michael.