Re: Parsing of \renewcommand-ed macros

[Tassilo Horn]

Arash Esbati <arash@gnu.org> writes:

Hi Arash,

>> IIUC AUCTeX doesn't have any provisions to parse \renewcommand-ed
>> macros in `LaTeX-auto-regexp-list' and subsequently in the function
>> `LaTeX-auto-cleanup'.  Before fiddling with the parser, I'd like to
>> know if I'm missing something.
>
> Ok, I fiddled with the parser and the outcome is attached.  Any
> comments welcome.

I haven't looked at the actual changes but just found this nitpick:

--8<---------------cut here---------------start------------->8---
@@ -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},
--8<---------------cut here---------------end--------------->8---

Why are you indenting the docstrings?  That looks ugly in C-h f.

Bye,
Tassilo