auctex-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Parsing of \renewcommand-ed macros


From: Arash Esbati
Subject: Re: Parsing of \renewcommand-ed macros
Date: Fri, 01 Apr 2022 13:15:53 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/29.0.50

Hi Tassilo,

Tassilo Horn <tsdh@gnu.org> writes:

> I haven't looked at the actual changes but just found this nitpick:
>
> @@ -1803,7 +1805,7 @@ This is necessary since index entries may contain 
> commands and stuff.")
>  
>  (defun LaTeX-split-bibs (match)
>    "Extract bibliography resources from MATCH.
> -Split the string at commas and remove Biber file extensions."
> +  Split the string at commas and remove Biber file extensions."
>    (let ((bibs (split-string (TeX-match-buffer match) " *, *")))
>      (dolist (bib bibs)
>        (LaTeX-add-bibliographies (replace-regexp-in-string
> @@ -1826,7 +1828,7 @@ Split the string at commas and remove Biber file 
> extensions."
>  
>  (defun LaTeX-listify-package-options (options)
>    "Return a list from a comma-separated string of package OPTIONS.
> -The input string may include LaTeX comments and newlines."
> +  The input string may include LaTeX comments and newlines."
>    ;; We jump through all those hoops and don't just use `split-string'
>    ;; or the like in order to be able to deal with key=value package
>    ;; options which can look like this: "pdftitle={A Perfect Day},
>
> Why are you indenting the docstrings?  That looks ugly in C-h f.

Upps, thanks for catching this.  It wasn't me, I suspect it was
aggressive-indent (but don't ask me why).

New version is attached.

Best, Arash

Attachment: latex-parser.diff
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]