unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Theodor Thornhill <theo@thornhill.no>
Cc: larsi@gnus.org, 59662@debbugs.gnu.org, casouri@gmail.com,
	monnier@iro.umontreal.ca, dgutov@yandex.ru
Subject: bug#59662: 29.0.50; [PATCH] Add treesit--indent-defun
Date: Fri, 09 Dec 2022 22:20:05 +0200	[thread overview]
Message-ID: <834ju4z5vu.fsf@gnu.org> (raw)
In-Reply-To: <87wn70injy.fsf@thornhill.no> (message from Theodor Thornhill on Fri, 09 Dec 2022 16:50:25 +0100)

> From: Theodor Thornhill <theo@thornhill.no>
> Cc: larsi@gnus.org, 59662@debbugs.gnu.org, casouri@gmail.com,
>  monnier@iro.umontreal.ca
> Date: Fri, 09 Dec 2022 16:50:25 +0100
> 
> diff --git a/doc/emacs/programs.texi b/doc/emacs/programs.texi
> index ba8475e86a..cc06a003e2 100644
> --- a/doc/emacs/programs.texi
> +++ b/doc/emacs/programs.texi
> @@ -409,6 +409,14 @@ Multi-line Indent
>  @table @kbd
>  @item C-M-q
>  Reindent all the lines within one parenthetical grouping.
> +@item M-q
> +Major modes that derive from @code{prog-mode} can reindent all the
> +lines within a defun around point using
> +@code{prog-mode-fill-reindent-defun}.  The bounds of of a defun is
> +decided by @code{beginning-of-defun-function} and
> +@code{end-of-defun-function}.  If point is inside a comment or a
> +doc-string, refill that paragraph instead, using
> +@code{fill-paragraph-function}.

The 2 variables you mention should have @vindex entries.  Also, please
add a cross-reference to the mode in elisp manual where these two
variables are described.

> +*** New command 'prog-fill-reindent-defun'
> +This command either fills a single paragraph in a defun, such as a
> +doc-string, or a comment.  If point is inside normal code, it
> +(re)indents the surrounding defun.

These two sentences should be one sentence, otherwise that 'either" is
out of place.  Something like

  This command either fills a single paragraph in a defun, such as a
  doc-string, or a comment, or (re)indents the surrounding defun if
  point is not in a comment or a string.

>                                        Bound by default in
> +'prog-mode-map' to 'M-q'.

"It is by default bound to 'M-q' in 'prog-mode' and all its
descendants."

> +(defun prog-fill-reindent-defun (&optional argument)
> +  "Refill paragraph or reindent the definition that the point is on.
> +
> +If the point is in a string, or in a comment, or there is a
> +comment on the current line, fill the paragraph that the point is
> +in or is on the same line.

I don't understand the difference between "point is in a string, or in
a comment" and "there is a comment on the current line".  

> +Otherwise, reindent the definition around or below point."

Likewise this one is confusing.





  parent reply	other threads:[~2022-12-09 20:20 UTC|newest]

Thread overview: 97+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-28 19:32 bug#59662: 29.0.50; [PATCH] Add treesit--indent-defun Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
     [not found] ` <handler.59662.B.16696639455818.ack@debbugs.gnu.org>
2022-11-29  6:10   ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-11-29 12:01 ` Eli Zaretskii
2022-11-29 12:14   ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-11-29 14:15     ` Eli Zaretskii
2022-11-30 10:33 ` Yuan Fu
2022-11-30 23:23 ` Yuan Fu
2022-12-01  1:07   ` Yuan Fu
2022-12-01  7:19     ` Eli Zaretskii
2022-12-01  6:10   ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-01  7:13   ` Eli Zaretskii
2022-12-01  7:22     ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-01  7:48       ` Eli Zaretskii
2022-12-01  8:13         ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-02 15:06         ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-02 15:45           ` Eli Zaretskii
2022-12-02 18:32             ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-02 18:52               ` Eli Zaretskii
2022-12-02 19:02           ` Daniel Martín via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-02 20:24             ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-02 21:09               ` Daniel Martín via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-02 21:19                 ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-04 12:33                   ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-04 12:40                     ` Eli Zaretskii
2022-12-07  9:31                       ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-07 13:44                         ` Eli Zaretskii
2022-12-02 19:53     ` Dmitry Gutov
2022-12-02 20:01       ` Dmitry Gutov
2022-12-02 20:07       ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-02 22:27         ` Dmitry Gutov
2022-12-03  6:47           ` Eli Zaretskii
2022-12-03 15:59             ` Dmitry Gutov
2022-12-03 17:54               ` Eli Zaretskii
2022-12-08  1:37                 ` Dmitry Gutov
2022-12-08  7:16                   ` Eli Zaretskii
2022-12-08  8:11                     ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-08  8:27                       ` Eli Zaretskii
2022-12-08  8:56                         ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-08  9:08                           ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-08 10:39                           ` Eli Zaretskii
2022-12-08 11:09                             ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-08 11:38                               ` Eli Zaretskii
2022-12-08 14:22                               ` Dmitry Gutov
2022-12-08 14:58                                 ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-08 16:06                                   ` Dmitry Gutov
2022-12-08 16:33                                     ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-08 22:03                                       ` Dmitry Gutov
2022-12-08 14:18                           ` Dmitry Gutov
2022-12-08 14:53                             ` Eli Zaretskii
2022-12-08 15:43                               ` Dmitry Gutov
2022-12-08 16:04                                 ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-08 16:23                                   ` Dmitry Gutov
2022-12-08 19:15                                     ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-08 19:50                                       ` Stefan Monnier via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-08 20:12                                       ` Eli Zaretskii
2022-12-08 21:21                                         ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-09  5:06                                           ` Eli Zaretskii
2022-12-09 13:24                                             ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-09 13:38                                               ` Dmitry Gutov
2022-12-09 14:46                                                 ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-09 14:56                                                   ` Dmitry Gutov
2022-12-09 15:13                                                     ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-09 15:27                                                       ` Dmitry Gutov
2022-12-09 15:29                                                         ` Dmitry Gutov
2022-12-09 15:50                                                         ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-09 16:05                                                           ` Dmitry Gutov
2022-12-09 19:15                                                             ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-09 20:28                                                               ` Dmitry Gutov
2022-12-09 20:31                                                                 ` Dmitry Gutov
2022-12-09 20:20                                                           ` Eli Zaretskii [this message]
2022-12-09 20:30                                                             ` Dmitry Gutov
2022-12-10 14:04                                                               ` Eli Zaretskii
2022-12-10 19:25                                                                 ` Dmitry Gutov
2022-12-09 21:09                                                             ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-10 12:02                                                               ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-10 14:27                                                                 ` Eli Zaretskii
2022-12-10 14:09                                                               ` Eli Zaretskii
2022-12-10 20:21                                                                 ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-11  8:11                                                                   ` Eli Zaretskii
2022-12-11  8:39                                                                     ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-11 10:03                                                                       ` Eli Zaretskii
2022-12-11 14:13                                                                         ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-11 20:36                                                                           ` Dmitry Gutov
2022-12-11 20:39                                                                             ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-08 22:05                                       ` Dmitry Gutov
2022-12-08 22:10                                         ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-08 22:49                                           ` Dmitry Gutov
2022-12-08 14:24                     ` Dmitry Gutov
2022-12-08 15:37                       ` Dmitry Gutov
2022-12-08 15:46                         ` Eli Zaretskii
2022-12-07 19:54 ` Yuan Fu
2022-12-07 20:06   ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-07 21:40     ` Yuan Fu
2022-12-07 21:53       ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-07 23:09         ` Yuan Fu
2022-12-08  6:02           ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-01-07 23:01 ` Yuan Fu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=834ju4z5vu.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=59662@debbugs.gnu.org \
    --cc=casouri@gmail.com \
    --cc=dgutov@yandex.ru \
    --cc=larsi@gnus.org \
    --cc=monnier@iro.umontreal.ca \
    --cc=theo@thornhill.no \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).