bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)

classic Classic list List threaded Threaded
9 messages Options
Reply | Threaded
Open this post in threaded view
|

bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)

Bob Weiner
The next Texinfo release will have an @ktable keyword just like
@ftable and @vtable.
This patch to texinfo.el adds support for that in Texinfo-mode.

Bob

texinfo.el.patch (98 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)

Clément Pit--Claudel-2
On 2016-07-14 16:14, Robert Weiner wrote:
> The next Texinfo release will have an @ktable keyword just like
> @ftable and @vtable.
> This patch to texinfo.el adds support for that in Texinfo-mode.

Neat! I think there might have been a problem with this attachment though :/ I only see this:

    texinfo.el.patch
   
    Write file: /Applications/Emacs25.app/Contents/Resources/lisp/textmodes/

Clément.


signature.asc (836 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)

Bob Weiner
In reply to this post by Bob Weiner
Sorry, I grabbed the wrong file in the first message.  This one has the patch.

Bob

texinfo.el.patch (2K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)

Clément Pit--Claudel-2
On 2016-07-14 19:16, Robert Weiner wrote:
> Sorry, I grabbed the wrong file in the first message.  This one has the patch.

Thanks, looks good to me :)


signature.asc (836 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)

Eli Zaretskii
> From: Clément Pit--Claudel <[hidden email]>
> Date: Thu, 14 Jul 2016 19:26:14 +0200
>
> On 2016-07-14 19:16, Robert Weiner wrote:
> > Sorry, I grabbed the wrong file in the first message.  This one has the patch.
>
> Thanks, looks good to me :)

Agreed, please push to master.

In the future, please include the ChangeLog-style log entry for the
changes.

Thanks.



Reply | Threaded
Open this post in threaded view
|

bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)

Stefan Kangas
In reply to this post by Bob Weiner
Hi,

Robert Weiner <[hidden email]> writes:

> Sorry, I grabbed the wrong file in the first message.  This one has the patch.

This patch was approved to be pushed to master, see below:

    [ It seems you were not in the Cc list for the below messages. ]

Eli Zaretskii <[hidden email]> writes:

>> From: Clément Pit--Claudel <[hidden email]>
>> Date: Thu, 14 Jul 2016 19:26:14 +0200
>>
>> On 2016-07-14 19:16, Robert Weiner wrote:
>> > Sorry, I grabbed the wrong file in the first message.  This one has the patch.
>>
>> Thanks, looks good to me :)
>
> Agreed, please push to master.
>
> In the future, please include the ChangeLog-style log entry for the
> changes.
>
> Thanks.

Was it ever pushed?  Otherwise, could you please go ahead and push it?

Thanks.

Best regards,
Stefan Kangas



Reply | Threaded
Open this post in threaded view
|

bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)

Eli Zaretskii
> From: Stefan Kangas <[hidden email]>
> Date: Mon, 04 May 2020 03:11:18 +0200
> Cc: [hidden email], [hidden email],
>  Gavin Smith <[hidden email]>
>
> Was it ever pushed?  Otherwise, could you please go ahead and push it?

Btw, Texinfo still doesn't have @ktable, AFAICT, 4 years later.



Reply | Threaded
Open this post in threaded view
|

bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)

Gavin Smith
On Mon, May 04, 2020 at 05:21:07PM +0300, Eli Zaretskii wrote:
> > Was it ever pushed?  Otherwise, could you please go ahead and push it?
>
> Btw, Texinfo still doesn't have @ktable, AFAICT, 4 years later.

This is true, we took the feature out because the person proposing it
wasn't going to use it, so it wasn't clear that the feature would be
useful.



Reply | Threaded
Open this post in threaded view
|

bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)

Stefan Kangas
tags 23981 + wontfix
close 23981
thanks

Gavin Smith <[hidden email]> writes:

>> > Was it ever pushed?  Otherwise, could you please go ahead and push it?
>>
>> Btw, Texinfo still doesn't have @ktable, AFAICT, 4 years later.
>
> This is true, we took the feature out because the person proposing it
> wasn't going to use it, so it wasn't clear that the feature would be
> useful.

Thanks.  I assume that means that we should not add support for it to
Emacs either, and I'm therefore closing this bug.

Please reply back if this conclusion is incorrect.

Best regards,
Stefan Kangas