Re: [PATCH 1/2] CLI/reply: use static const for reply_to_map

Subject: Re: [PATCH 1/2] CLI/reply: use static const for reply_to_map

Date: Thu, 25 Jun 2026 08:36:08 +0300

To: Dick Marinus, notmuch@notmuchmail.org

Cc:

From: Tomi Ollila

On Tue, Jun 23 2026, Dick Marinus wrote:

> Makes the structure compile-time constant and often stored in
> read-only location.
>
> Workaround to a problem with GCC (bugzilla #125252) which causes
> a segmentation fault.

This change looks good and commit message looks tolerable to me,

Tomi

> ---
>  notmuch-reply.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/notmuch-reply.c b/notmuch-reply.c
> index 44297251d..c72ca227e 100644
> --- a/notmuch-reply.c
> +++ b/notmuch-reply.c
> @@ -329,7 +329,7 @@ add_recipients_from_message (GMimeMessage *reply,
>  			     GMimeMessage *message,
>  			     bool reply_all)
>  {
> -    struct {
> +    static const struct {
>  	InternetAddressList * (*get_header)(GMimeMessage *message);
>  	GMimeAddressType recipient_type;
>      } reply_to_map[] = {
> -- 
> 2.54.0
>
> _______________________________________________
> notmuch mailing list -- notmuch@notmuchmail.org
> To unsubscribe send an email to notmuch-leave@notmuchmail.org
_______________________________________________
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-leave@notmuchmail.org

Thread: