* Re: master 63255de48b: * src/pdumper.c (dump_vectorlike): Handle PVEC_SYMBOL_WITH_POS.
[not found] ` <20220127113052.0F1F3C423B2@vcs2.savannah.gnu.org>
@ 2022-01-27 12:10 ` Po Lu
2022-01-27 12:13 ` Andreas Schwab
0 siblings, 1 reply; 2+ messages in thread
From: Po Lu @ 2022-01-27 12:10 UTC (permalink / raw)
To: emacs-devel; +Cc: Andreas Schwab
Andreas Schwab <schwab@gnu.org> writes:
> * src/pdumper.c (dump_vectorlike): Handle PVEC_SYMBOL_WITH_POS.
> ---
> src/pdumper.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/src/pdumper.c b/src/pdumper.c
> index 60280fcb04..f4e8e4af28 100644
> --- a/src/pdumper.c
> +++ b/src/pdumper.c
> @@ -2948,7 +2948,7 @@ dump_vectorlike (struct dump_context *ctx,
> Lisp_Object lv,
> dump_off offset)
> {
> -#if CHECK_STRUCTS && !defined HASH_pvec_type_19F6CF5169
> +#if CHECK_STRUCTS && !defined HASH_pvec_type_AFF6FED5BD
> # error "pvec_type changed. See CHECK_STRUCTS comment in config.h."
> #endif
> const struct Lisp_Vector *v = XVECTOR (lv);
> @@ -3032,6 +3032,8 @@ dump_vectorlike (struct dump_context *ctx,
> error_unsupported_dump_object (ctx, lv, "sqlite");
> case PVEC_MODULE_FUNCTION:
> error_unsupported_dump_object (ctx, lv, "module function");
> + case PVEC_SYMBOL_WITH_POS:
> + error_unsupported_dump_object (ctx, lv, "symbol with pos");
> default:
> error_unsupported_dump_object(ctx, lv, "weird pseudovector");
> }
Hmm, why not actually dump those symbols? I think it makes sense to do
so, though admittedly didn't take a very close look at their
implementation.
Thanks.
^ permalink raw reply [flat|nested] 2+ messages in thread