On Tue, 1 Mar 2022 at 13:52, Jürgen Spitzmüller <sp...@lyx.org> wrote:

> Am Di., 1. März 2022 um 13:51 Uhr schrieb Jürgen Spitzmüller <
> sp...@lyx.org>:
>
>> Am Di., 1. März 2022 um 04:19 Uhr schrieb Thibaut Cuvelier <
>> tcuvel...@lyx.org>:
>>
>>> On Tue, 1 Mar 2022 at 03:58, Thibaut Cuvelier <tcuvel...@lyx.org> wrote:
>>>
>>>> On Mon, 28 Feb 2022 at 08:04, Jürgen Spitzmüller <sp...@lyx.org> wrote:
>>>>
>>>>> Am Sonntag, dem 27.02.2022 um 20:48 +0100 schrieb Thibaut Cuvelier:
>>>>> > I'm doing this in the attached patches (also for HTML, XML being the
>>>>> > common part between the two). I would like some feedback more
>>>>> > specifically for the typing of the enum: as I could find, it's never
>>>>> > sure that enums are coded with enough bits to add a code (there are
>>>>> > already enough codes to fill 32 bits). I thus forced the types to be
>>>>> > unsigned long long, to have 64 available bits
>>>>> > (https://en.cppreference.com/w/cpp/language/types).
>>>>> >
>>>>> > I also tried to change as many occurrences I could find, but there
>>>>> > were not many (or my searching skills are not on par).
>>>>>
>>>>> Thinking more about it, another option would be to rename LATEX to
>>>>> something more generic (e.g. OUTFILE), maybe keeping "latex" as an
>>>>> alias on the command line. After all, only one of these is used at a
>>>>> given time, and depending on the output chain, it might not even be
>>>>> clear which one. Also, we do not have a PLAINTEXT mode.
>>>>>
>>>>
>>>> What do you think about this new iteration?
>>>>
>>>
>>> I forgot to include the changes to debug.cpp in the previous patch, here
>>> is a new complete patch.
>>>
>>
>> Looks good. I would probably rephrase the help text as "Output source
>> file generation/processing"
>>
>
> ... and add an alias "latex" for backwards compatibility, like we have for
> "any" and "all".
>

I've pushed the updated patch, thanks :)!
-- 
lyx-devel mailing list
lyx-devel@lists.lyx.org
http://lists.lyx.org/mailman/listinfo/lyx-devel

Reply via email to