On Sat, Apr 20 2019, Daniel Kahn Gillmor wrote:
> This makes it easier to reuse format_part_sigstatus_sprinter() when we
> have other places that we want to display a signature status.
Change looks trivial enough (and reduces the "scope" function sees which is
always good if it is enough -- I trust it is)
Tomi
> ---
> notmuch-show.c | 6 ++
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/notmuch-show.c b/notmuch-show.c
> index 07e9a5db..88699e90 100644
> --- a/notmuch-show.c
> +++ b/notmuch-show.c
> @@ -429,13 +429,11 @@ format_signature_errors (sprinter_t *sp, GMimeSignature
> *signature)
>
> /* Signature status sprinter (GMime 2.6) */
> static void
> -format_part_sigstatus_sprinter (sprinter_t *sp, mime_node_t *node)
> +format_part_sigstatus_sprinter (sprinter_t *sp, GMimeSignatureList *siglist)
> {
> /* Any changes to the JSON or S-Expression format should be
> * reflected in the file devel/schemata. */
>
> -GMimeSignatureList *siglist = node->sig_list;
> -
> sp->begin_list (sp);
>
> if (!siglist) {
> @@ -684,7 +682,7 @@ format_part_sprinter (const void *ctx, sprinter_t *sp,
> mime_node_t *node,
>
> if (node->verify_attempted) {
> sp->map_key (sp, "sigstatus");
> - format_part_sigstatus_sprinter (sp, node);
> + format_part_sigstatus_sprinter (sp, node->sig_list);
> }
>
> sp->map_key (sp, "content-type");
> --
> 2.20.1
>
> ___
> notmuch mailing list
> notmuch@notmuchmail.org
> https://notmuchmail.org/mailman/listinfo/notmuch
___
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch
