[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] New LaTeX code export option: engraved
From: |
Ihor Radchenko |
Subject: |
Re: [PATCH] New LaTeX code export option: engraved |
Date: |
Sat, 07 May 2022 12:13:10 +0800 |
Max Nikulin <manikulin@gmail.com> writes:
>> +(defun org-latex-src-block--verbatim
>> + (src-block info _lang caption caption-above-p _label
>> + _num-start _retain-labels _attributes float)
>
> On the one hand I have no a better suggestion, but on the other hand 10
> arguments is too much for straightforward code from my point of view.
> Unsure that getting parameters in each function or passing a property
> list would not be worse however.
I agree that having that many unused arguments is awkward, especially
when the arguments are repeated in all the calls.
Maybe the functions could be rewritten using cl-defun with keys and
&allow-other-keys and then called via apply on a let-bound arg plist?
Best,
Ihor
- Re: [PATCH] New LaTeX code export option: engraved, (continued)
- Re: [PATCH] New LaTeX code export option: engraved, Daniel Fleischer, 2022/05/07
- Re: [PATCH] (v2) New LaTeX code export option: engraved, Timothy, 2022/05/08
- Re: [PATCH] (v2) New LaTeX code export option: engraved, Ihor Radchenko, 2022/05/09
- Re: [PATCH] (v2) New LaTeX code export option: engraved, Timothy, 2022/05/09
- Re: [PATCH] (v2) New LaTeX code export option: engraved, Max Nikulin, 2022/05/10
- Re: [PATCH] (v2) New LaTeX code export option: engraved, Ihor Radchenko, 2022/05/11
- Re: [PATCH] (v3) New LaTeX code export option: engraved, Timothy, 2022/05/11
- Re: [PATCH] (v3) New LaTeX code export option: engraved, Daniel Fleischer, 2022/05/12
- Re: [PATCH] (v3) New LaTeX code export option: engraved, Timothy, 2022/05/12
- Re: [PATCH] New LaTeX code export option: engraved, Max Nikulin, 2022/05/05
- Re: [PATCH] New LaTeX code export option: engraved,
Ihor Radchenko <=
Re: [PATCH] New LaTeX code export option: engraved, Sébastien Miquel, 2022/05/09