Subject: Re: bug#67559: 29.1; Fix indentation in code example of optional arguments

From: Xiyue Deng <[email-masked]>
Date: Fri, 01 Dec 2023 02:34:29 -0800

Eli Zaretskii <[email-masked]> writes:

>> From: Xiyue Deng <[email-masked]>
>> Date: Thu, 30 Nov 2023 17:45:57 -0800
>> 
>> The code example in the optional arguments section seems a little off in
>> indentation according to Elisp convention.  The attached patch tries to
>> improve it.
>
> Thanks, but "git am" rejects this patch with the following message:
>
>   Line longer than 78 characters in commit message
>   Commit aborted; please see the file CONTRIBUTE
>
> Please fix the commit log message (I think the problem is in the
> Subject line, actually) and resubmit.

Ah indeed the message was too long.  Here is the updated patch with
shorter title.

>From 300340562ba08696b05ec6580e8f240205218702 Mon Sep 17 00:00:00 2001
From: Xiyue Deng <[email-masked]>
Date: Fri, 1 Dec 2023 02:31:49 -0800
Subject: [PATCH] * doc/lispintro/emacs-lisp-intro.texi (Optional Arguments):
 Fix indentation

---
 doc/lispintro/emacs-lisp-intro.texi | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/doc/lispintro/emacs-lisp-intro.texi b/doc/lispintro/emacs-lisp-intro.texi
index e4a0f585f69..62692d21fac 100644
--- a/doc/lispintro/emacs-lisp-intro.texi
+++ b/doc/lispintro/emacs-lisp-intro.texi
@@ -5982,12 +5982,12 @@ In outline, the whole function looks like this:
       (and @var{are-both-transient-mark-mode-and-mark-active-true})
       (push-mark))
   (let (@var{determine-size-and-set-it})
-  (goto-char
-    (@var{if-there-is-an-argument}
-        @var{figure-out-where-to-go}
-      @var{else-go-to}
-      (point-min))))
-   @var{do-nicety}
+    (goto-char
+      (@var{if-there-is-an-argument}
+          @var{figure-out-where-to-go}
+        @var{else-go-to}
+        (point-min))))
+  @var{do-nicety}
 @end group
 @end smallexample
 
-- 
2.39.2

-- 
Xiyue Deng