Re: [PATCH] Implement texinfo @ref dwim

[Eli Zaretskii]

> From: Robert Pluim <address@hidden>
> Date: Tue, 05 Feb 2019 13:10:07 +0100
> 
> This has been sitting around in my patch queue for a while, and Iʼd
> like to get it in so I can also get the manual update into the texinfo
> project. I think Iʼve addressed all the comments from the last time
> around.

Thanks, I think this command will be useful.

> * etc/NEWS: Describe new texinfo dwim reference functionality.

I understand that the corresponding update to the Texinfo manual will
be submitted to the Texinfo maintainers?  If so, the NEWS entry should
be marked with "+++", I think.

> +(define-skeleton address@hidden
> +  "Insert appropriate address@hidden', address@hidden', or address@hidden' 
> command.
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Is Emacs smart enough not to highlight these as links?

> +   ;; parenthesis
> +   ((eq (char-before) ?\()
> +    "@pxref{")

@pxref is also good inside parentheses, not just after the opening
paren.  Can this command be smarter and support also cases like

  (bla-bla yak-yak @pxref{foo})

?

> +   ;; beginning of sentence
> +   ((looking-back (sentence-end) (point-at-bol 0))
> +    "@xref{")

What about BOB?  Can we produce @xref there as well?