[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Use vtable for eww-bookmarks
From: |
Sebastián Monía |
Subject: |
Re: [PATCH] Use vtable for eww-bookmarks |
Date: |
Wed, 29 Jan 2025 08:54:37 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
I realized this morning I compared numbers using 'equal' instead of = in
some of the new code.
Attached updated patch :)
0001-Use-vtable-in-eww-list-bookmarks.patch
Description: Use vtable for eww-bookmarks
Sebastián Monía <sebastian@sebasmonia.com> writes:
> Thanks for detecting these bugs, Jim!
>
> Jim Porter <jporterbugs@gmail.com> writes:
>> 1. Killing a bookmark works, but yanking at the end of the buffer
>> produces the following error:
>
> This happened because the reverting function must be called with point
> inside the vtable instance. Modified so that in the special case that
> point is at the end of the buffer after removing the row, we move one
> line up.
>
>> 2. Killing the last bookmark in the buffer produces this error:
>
> In this case, I will handle adding to the end of the list as a special
> case. Once again, with point outside the vtable, a lot of vtable calls
> don't work . And then we also have to move point inside the table to
> revert it, just like in #1.
>
> Attached the patch with these corrections.
>
> Thank you,
> Seb
--
Sebastián Monía
https://site.sebasmonia.com/
- Re: [PATCH] Use vtable for eww-bookmarks, (continued)
- Re: [PATCH] Use vtable for eww-bookmarks, Stefan Kangas, 2025/01/07
- Re: [PATCH] Use vtable for eww-bookmarks, Jim Porter, 2025/01/07
- Re: [PATCH] Use vtable for eww-bookmarks, Thierry Volpiatto, 2025/01/08
- Re: [PATCH] Use vtable for eww-bookmarks, Adam Porter, 2025/01/08
- Re: [PATCH] Use vtable for eww-bookmarks, Sebastián Monía, 2025/01/18
- Re: [PATCH] Use vtable for eww-bookmarks, Stefan Kangas, 2025/01/21
- Re: [PATCH] Use vtable for eww-bookmarks, Sebastián Monía, 2025/01/21
- Re: [PATCH] Use vtable for eww-bookmarks, Jim Porter, 2025/01/21
- Re: [PATCH] Use vtable for eww-bookmarks, Sebastián Monía, 2025/01/26
- Re: [PATCH] Use vtable for eww-bookmarks, Sebastián Monía, 2025/01/28
- Re: [PATCH] Use vtable for eww-bookmarks,
Sebastián Monía <=