[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#51883: 29.0.50; Command to get accidentally deleted frames back
From: |
Stefan Monnier |
Subject: |
bug#51883: 29.0.50; Command to get accidentally deleted frames back |
Date: |
Fri, 21 Jan 2022 12:52:35 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/29.0.50 (gnu/linux) |
Eli Zaretskii [2021-11-16 19:40:08] wrote:
>> +(add-hook 'delete-frame-functions #'undeleted-frame--save-deleted-frame)
>
> I'd rather we didn't do that by default. Several reasons:
>
> . the startup code deletes the terminal frame, so the above means we
> will always load frameset, which is not a small package, at
> startup, even if the user has no use for this functionality
Clearly we should avoid putting the special initial-frame in the undo
log since we don't want (and can't) recreate it later anyway.
But as for loading the package, the problem is more general in the sense
that maybe we should refrain from loading the package just because
a frame is deleted.
So maybe we should change the code so the hook's function doesn't
require loading `frameset.el`. AFAICT fundamentally, the only function
from `frameset.el` that it needs is `frameset-filter-params`, so maybe
we should move some of that code to `frame.el`.
To make up for it, maybe we can move some of the `undelete-frame` code
to `frameset.el` since it can't be used without `frameset.el` anyway?
> . saving configurations of 16 deleted frames _by_default_ means we
> again impose on all users something that only some of them will use
IIUC framesets are designed to be serializable so they shouldn't hold on
to external data like buffers and windows, so such 16 elements should
cost very little in terms of heap use.
Stefan
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, (continued)
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, Juri Linkov, 2022/01/14
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, Eli Zaretskii, 2022/01/17
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, Juri Linkov, 2022/01/17
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, Eli Zaretskii, 2022/01/17
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, Juri Linkov, 2022/01/18
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, Juri Linkov, 2022/01/19
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, Eli Zaretskii, 2022/01/19
bug#51883: 29.0.50; Command to get accidentally deleted frames back,
Stefan Monnier <=
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, Juri Linkov, 2022/01/22
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, Stefan Monnier, 2022/01/22
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, Juri Linkov, 2022/01/23
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, Stefan Monnier, 2022/01/23
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, Juri Linkov, 2022/01/23
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, Stefan Monnier, 2022/01/23
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, Juri Linkov, 2022/01/24
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, Eli Zaretskii, 2022/01/24
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, martin rudalics, 2022/01/25
- bug#51883: 29.0.50; Command to get accidentally deleted frames back, Eli Zaretskii, 2022/01/25