Re: Please revamp the way floatrow.el handles figure defaults

[Arash Esbati]

Hi Keita,

Ikumi Keita <ikumi@ikumi.que.jp> writes:

> Honestly, I don't think this is an ideal treatment. Normally,
> `TeX-auto-parse' is called in `TeX-auto-apply' called in
> `TeX-update-style'. Thus adding `TeX-auto-parse' in
> `TeX-update-style-hook' leads to calling `TeX-auto-parse' twice; it's
> apparently an overhead. :-(

Yes, I agree about the duplication.

> ,----
> | (defun TeX-update-style (&optional force)
> | [...]
> |     (if (and TeX-parse-self
> |              (null (cdr-safe (assoc (TeX-strip-extension nil nil t)
> |                                     TeX-style-hook-list))))
> |         (TeX-auto-apply))               <---
> |     (run-hooks 'TeX-update-style-hook)  <---
> `----
> However, I suppose it is very difficult to avoid this duplication under
> the current implementation of "style application" of AUCTeX. In
> addition, it's very complicated and it doesn't seem to me that we can
> refactor it into a more sophisticated form to allow such fine-grained
> separation keeping backward compatibility.

Again, agreed, the "style application" of AUCTeX isn't really fun.

Best, Arash