all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)
@ 2016-07-14 14:14 Robert Weiner
  2016-07-14 16:49 ` Clément Pit--Claudel
  2016-07-14 17:16 ` Robert Weiner
  0 siblings, 2 replies; 9+ messages in thread
From: Robert Weiner @ 2016-07-14 14:14 UTC (permalink / raw)
  To: 23981; +Cc: Gavin Smith

[-- Attachment #1: Type: text/plain, Size: 154 bytes --]

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

[-- Attachment #2: texinfo.el.patch --]
[-- Type: application/octet-stream, Size: 72 bytes --]

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

^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)
  2016-07-14 14:14 bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing) Robert Weiner
@ 2016-07-14 16:49 ` Clément Pit--Claudel
  2016-07-14 17:16 ` Robert Weiner
  1 sibling, 0 replies; 9+ messages in thread
From: Clément Pit--Claudel @ 2016-07-14 16:49 UTC (permalink / raw)
  To: rswgnu, 23981; +Cc: Gavin Smith


[-- Attachment #1.1: Type: text/plain, Size: 421 bytes --]

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.


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)
  2016-07-14 14:14 bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing) Robert Weiner
  2016-07-14 16:49 ` Clément Pit--Claudel
@ 2016-07-14 17:16 ` Robert Weiner
  2016-07-14 17:26   ` Clément Pit--Claudel
  2020-05-04  1:11   ` Stefan Kangas
  1 sibling, 2 replies; 9+ messages in thread
From: Robert Weiner @ 2016-07-14 17:16 UTC (permalink / raw)
  To: 23981; +Cc: Gavin Smith

[-- Attachment #1: Type: text/plain, Size: 84 bytes --]

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

Bob

[-- Attachment #2: texinfo.el.patch --]
[-- Type: application/octet-stream, Size: 1574 bytes --]

*** texinfo-orig.el	2016-07-14 13:12:25.000000000 -0400
--- texinfo.el		2016-07-14 13:12:25.000000000 -0400
***************
*** 336,342 ****
      "documentdescription" "enumerate" "example" "flushleft"
      "flushright" "format" "ftable" "group" "html" "ifclear" "ifset"
      "ifhtml" "ifinfo" "ifnothtml" "ifnotinfo" "ifnotplaintext"
!     "ifnottex" "ifplaintext" "iftex" "ignore" "itemize" "lisp"
      "macro" "menu" "multitable" "quotation" "smalldisplay"
      "smallexample" "smallformat" "smalllisp" "table" "tex"
      "titlepage" "verbatim" "vtable")
--- 336,342 ----
      "documentdescription" "enumerate" "example" "flushleft"
      "flushright" "format" "ftable" "group" "html" "ifclear" "ifset"
      "ifhtml" "ifinfo" "ifnothtml" "ifnotinfo" "ifnotplaintext"
!     "ifnottex" "ifplaintext" "iftex" "ignore" "itemize" "ktable" "lisp"
      "macro" "menu" "multitable" "quotation" "smalldisplay"
      "smallexample" "smallformat" "smalllisp" "table" "tex"
      "titlepage" "verbatim" "vtable")
***************
*** 658,664 ****
    ;; Blocks that take parameters: all the def* blocks take parameters,
    ;;  plus a few others.
    (if (or (string-match "\\`def" str)
!           (member str '("table" "ftable" "vtable")))
        '(nil " " -))
    \n _ \n "@end " str \n \n)
  
--- 665,671 ----
    ;; Blocks that take parameters: all the def* blocks take parameters,
    ;;  plus a few others.
    (if (or (string-match "\\`def" str)
!           (member str '("table" "ftable" "ktable" "vtable")))
        '(nil " " -))
    \n _ \n "@end " str \n \n)
  

^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)
  2016-07-14 17:16 ` Robert Weiner
@ 2016-07-14 17:26   ` Clément Pit--Claudel
  2016-07-14 19:14     ` Eli Zaretskii
  2020-05-04  1:11   ` Stefan Kangas
  1 sibling, 1 reply; 9+ messages in thread
From: Clément Pit--Claudel @ 2016-07-14 17:26 UTC (permalink / raw)
  To: 23981


[-- Attachment #1.1: Type: text/plain, Size: 158 bytes --]

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 :)


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)
  2016-07-14 17:26   ` Clément Pit--Claudel
@ 2016-07-14 19:14     ` Eli Zaretskii
  0 siblings, 0 replies; 9+ messages in thread
From: Eli Zaretskii @ 2016-07-14 19:14 UTC (permalink / raw)
  To: Clément Pit--Claudel; +Cc: 23981

> From: Clément Pit--Claudel <clement.pit@gmail.com>
> 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.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)
  2016-07-14 17:16 ` Robert Weiner
  2016-07-14 17:26   ` Clément Pit--Claudel
@ 2020-05-04  1:11   ` Stefan Kangas
  2020-05-04 14:21     ` Eli Zaretskii
  1 sibling, 1 reply; 9+ messages in thread
From: Stefan Kangas @ 2020-05-04  1:11 UTC (permalink / raw)
  To: Robert Weiner; +Cc: 23981, rswgnu, Gavin Smith

Hi,

Robert Weiner <rsw@gnu.org> 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 <eliz@gnu.org> writes:

>> From: Clément Pit--Claudel <clement.pit@gmail.com>
>> 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





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)
  2020-05-04  1:11   ` Stefan Kangas
@ 2020-05-04 14:21     ` Eli Zaretskii
  2020-05-04 16:43       ` Gavin Smith
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2020-05-04 14:21 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 23981, rswgnu, gavinsmith0123, rsw

> From: Stefan Kangas <stefan@marxist.se>
> Date: Mon, 04 May 2020 03:11:18 +0200
> Cc: 23981@debbugs.gnu.org, rswgnu@gmail.com,
>  Gavin Smith <gavinsmith0123@gmail.com>
> 
> Was it ever pushed?  Otherwise, could you please go ahead and push it?

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





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)
  2020-05-04 14:21     ` Eli Zaretskii
@ 2020-05-04 16:43       ` Gavin Smith
  2020-07-22 17:01         ` Stefan Kangas
  0 siblings, 1 reply; 9+ messages in thread
From: Gavin Smith @ 2020-05-04 16:43 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 23981, rswgnu, Stefan Kangas, rsw

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.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing)
  2020-05-04 16:43       ` Gavin Smith
@ 2020-07-22 17:01         ` Stefan Kangas
  0 siblings, 0 replies; 9+ messages in thread
From: Stefan Kangas @ 2020-07-22 17:01 UTC (permalink / raw)
  To: Gavin Smith; +Cc: 23981, rswgnu, rsw

tags 23981 + wontfix
close 23981
thanks

Gavin Smith <gavinsmith0123@gmail.com> 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





^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2020-07-22 17:01 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-07-14 14:14 bug#23981: 25.0.94: FEATURE ADDITION: Next Texinfo release supports @ktable (key sequence table with auto iitem indexing) Robert Weiner
2016-07-14 16:49 ` Clément Pit--Claudel
2016-07-14 17:16 ` Robert Weiner
2016-07-14 17:26   ` Clément Pit--Claudel
2016-07-14 19:14     ` Eli Zaretskii
2020-05-04  1:11   ` Stefan Kangas
2020-05-04 14:21     ` Eli Zaretskii
2020-05-04 16:43       ` Gavin Smith
2020-07-22 17:01         ` Stefan Kangas

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.