[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#70524: [PATCH] Fix `map-elt` with `setf` for subplaces
From: |
Michael Heerdegen |
Subject: |
bug#70524: [PATCH] Fix `map-elt` with `setf` for subplaces |
Date: |
Fri, 26 Apr 2024 14:19:34 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
okamsn@protonmail.com writes:
> My purpose with this patch and for bug#68863 regarding `seq-subseq`
> (which does not currently support `setf`, and I think should allow
> subplaces like `substring` claims to)
The `substring' gv-setter doesn't need a loop however, it creates a new
string using `concat'. Your patch would probably "work" ok in this
case, but I'm not convinced that this would be an improvement, still for
the same reasons.
> was for destructuring as `setf`-able places, like in cl-loop's `for
> VAR in-ref LIST`. I have implemented that for my Emacs Lisp package
> (https://github.com/okamsn/loopy), but not all of the `setf`-able
> destructuring constructs support sub-places in the expected way, due
> to how some of the GV expansions are defined.
But if loopy would base on an inefficient implementations this would not
be useful.
Are there examples where your patch is really a clear improvement?
Michael.
- bug#70524: [PATCH] Fix `map-elt` with `setf` for subplaces, Okamsn, 2024/04/22
- bug#70524: [PATCH] Fix `map-elt` with `setf` for subplaces, Michael Heerdegen, 2024/04/24
- bug#70524: [PATCH] Fix `map-elt` with `setf` for subplaces, Stefan Monnier, 2024/04/24
- bug#70524: [PATCH] Fix `map-elt` with `setf` for subplaces, okamsn, 2024/04/24
- bug#70524: [PATCH] Fix `map-elt` with `setf` for subplaces, Augusto Stoffel, 2024/04/25
- bug#70524: [PATCH] Fix `map-elt` with `setf` for subplaces,
Michael Heerdegen <=
- bug#70524: [PATCH] Fix `map-elt` with `setf` for subplaces, okamsn, 2024/04/28
- bug#70524: [PATCH] Fix `map-elt` with `setf` for subplaces, Stefan Monnier, 2024/04/28
- bug#70524: [PATCH] Fix `map-elt` with `setf` for subplaces, Michael Heerdegen, 2024/04/30
bug#70524: [PATCH] Fix `map-elt` with `setf` for subplaces, Augusto Stoffel, 2024/04/25