[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#63513: [PATCH] Make persist-defvar work with records and hash tables
From: |
Joseph Turner |
Subject: |
bug#63513: [PATCH] Make persist-defvar work with records and hash tables |
Date: |
Fri, 08 Sep 2023 10:10:58 -0700 |
Ihor Radchenko <yantar92@posteo.net> writes:
> Stefan Monnier <monnier@iro.umontreal.ca> writes:
>
>>>>> Here you go!
>>>> Thanks, pushed!
>>> May it be possible to promote `persist-hash-equal' and
>>> `persist-copy-tree' to common subr.el functions?
>>> The topic of comparing hash tables has been previously discussed in
>>> https://yhetil.org/emacs-devel/871qvz4kdw.fsf@localhost/
>>
>> Hmm... AFAICT `persist-copy-tree` is a copy of our current `copy-tree`,
>> so there's nothing to do there :-)
>
> Err...
> Then, wouldn't it be better to contribute this function to compat.el and
> use it from there?
The new behavior of copy-tree has already been added to compat.el:
https://github.com/emacs-compat/compat/pull/25
However, that change currently only exists in compat's emacs-30 branch.
I did not know if it was acceptable for persist.el to require compat
when I wrote these patches. If we agree that it is acceptable, then I'm
happy to submit a patch to replace (persist-copy-tree ...) with
(compat-call copy-tree ...), but we'll have to wait to apply the patch
until after the compat.el emacs-30 branch is merged into master.
- bug#63513: [PATCH] Make persist-defvar work with records and hash tables, (continued)
- bug#63513: [PATCH] Make persist-defvar work with records and hash tables, Stefan Monnier, 2023/09/04
- bug#63513: [PATCH] Make persist-defvar work with records and hash tables, Joseph Turner, 2023/09/04
- bug#63513: [PATCH] Make persist-defvar work with records and hash tables, Stefan Monnier, 2023/09/05
- bug#63513: [PATCH] Make persist-defvar work with records and hash tables, Ihor Radchenko, 2023/09/08
- bug#63513: [PATCH] Make persist-defvar work with records and hash tables, Eli Zaretskii, 2023/09/08
- bug#63513: [PATCH] Make persist-defvar work with records and hash tables, Ihor Radchenko, 2023/09/08
- bug#63513: [PATCH] Make persist-defvar work with records and hash tables, Eli Zaretskii, 2023/09/08
- bug#63513: [PATCH] Make persist-defvar work with records and hash tables, Ihor Radchenko, 2023/09/08
- bug#63513: [PATCH] Make persist-defvar work with records and hash tables, Stefan Monnier, 2023/09/08
- bug#63513: [PATCH] Make persist-defvar work with records and hash tables, Ihor Radchenko, 2023/09/08
- bug#63513: [PATCH] Make persist-defvar work with records and hash tables,
Joseph Turner <=
- bug#63513: [PATCH] Make persist-defvar work with records and hash tables, Ihor Radchenko, 2023/09/09
- bug#63513: [PATCH] Make persist-defvar work with records and hash tables, Daniel Mendler, 2023/09/09
- bug#63513: [PATCH] Make persist-defvar work with records and hash tables, Ihor Radchenko, 2023/09/09
- bug#63513: [PATCH] Make persist-defvar work with records and hash tables, Daniel Mendler, 2023/09/09
- bug#63513: [PATCH] Make persist-defvar work with records and hash tables, phillip . lord, 2023/09/05