bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#69454: Not possible to insert an empty vtable


From: Joost Kremers
Subject: bug#69454: Not possible to insert an empty vtable
Date: Fri, 31 May 2024 08:54:33 +0200

>> From: Joost Kremers <joostkremers@fastmail.fm>
>> Actually, forget about this one. I just realised that
>> `vtable--compute-columns` doesn't override the alignment property because it
>> may have been set explicitly in `make-vtable`. The actual problem is that if
>> a table is created without data, the alignment property of each column is set
>> to `right`, even though there's no reason to do so.

Actually, that's not entirely true. The problem was that once the 'align'
property was set, it could not be changed anymore. ('vtable-insert-object'
assumed that it could, though.) The new patch makes this possible, but only if
'align' wasn't set explicitly in the call to 'make-vtable'.

I'm including all three patches here, even though two of them haven't changed:

0001-Allow-empty-vtable.patch : same as before

0002-Enable-inserting-new-objects-into-empty-vtable.patch : same as before
(though it was 0003 then)

0003-vtable-allow-resetting-column-alignment-when-table-d.patch : new, as
described above.

I haven't updated the documentation yet nor did I add a NEWS entry, because I
first wanted to make sure you agree with the direction of this patch: Adam
suggested empty vtables should not be allowed, but this patch explicitly allows
them.

Thanks,

Joost


-- 
Joost Kremers
Life has its moments


Attachment: 0001-Allow-empty-vtable.patch
Description: Text Data

Attachment: 0002-Enable-inserting-new-objects-into-empty-vtable.patch
Description: Text Data

Attachment: 0003-vtable-allow-resetting-column-alignment-when-table-d.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]