Subject: Re: bug#48264: [PATCH v3 06/15] Rearrange set_internal for buffer forwarded symbols

From: Eli Zaretskii <[email-masked]>
Date: Fri, 07 May 2021 13:45:50 +0300

> From: Spencer Baugh <[email-masked]>
> Date: Thu,  6 May 2021 17:33:37 -0400
> Cc: Spencer Baugh <[email-masked]>
> 
> --- a/src/data.c
> +++ b/src/data.c
> @@ -1320,6 +1320,9 @@ store_symval_forwarding (lispfwd valcontents, Lisp_Object newval,
>  	if (buf == NULL)
>  	  buf = current_buffer;
>  	set_per_buffer_value (buf, offset, newval);
> +        int idx = PER_BUFFER_IDX (offset);
> +        if (idx > 0)
> +          SET_PER_BUFFER_VALUE_P (buf, idx, 1);

It looks like you customized indent-tabs-mode to nil in C modes (or
maybe don't use Emacs?) because your indentation uses only spaces.
Please use our conventions in C sources, which uses TABs and spaces,
per indent-tabs-mode's default value.

This is a general comment, because I see signs of this in all of your
patches.