On Mon, 28 Dec 2020 at 19:33, Jean-Marc Lasgouttes <[email protected]> wrote:
> Le 26 décembre 2020 20:07:19 GMT+01:00, Thibaut Cuvelier < > [email protected]> a écrit : >> >> Dear list, >> >> In https://www.lyx.org/trac/ticket/12055, there was some confusion about >> the name of the streams used to output math: there is OctaveStream for >> Octave, MathematicaStream for Mathematica, etc., but MathStream for MathML >> (the name is much more generic than what the class does) and WriteStream >> for standard LaTeX math (the name is unrelated to the features of the >> class). >> >> As proposed in the ticket, I attach two patches to rename these classes >> so that their name is much more descriptive. Is it OK to push these >> commits? >> >> Thibaut >> > > What you propose makes sense, but LaTeXMathStream is a bit long. What > about using just MathStream? Or TeXMathStream like we have otexstream. > The problem with MathStream is that, currently, the MathML stream is called MathStream, it would be best to avoid reusing that name for something very similar that soon. I also wanted to avoid (La)TeXStream, because that could be mistaken with the generation of the TeX file, while it's restricted to maths. I like TeXMathStream, though. > Also, I am not sure why math streams use different capitalization rules > than their text counterpart. > What do you mean with "text counterpart"? Things like MathematicaStream or MaximaStream just rely on odocstream (odocstringstream for HTMLStream and the one for MathML), there is no real counterpart for these streams. Otherwise, it's quite common in the whole code base to have capitalised names (camel case) for classes, it's more like odocstream & co. follow a different convention (closer to the STL, but I guess that's because they just act like stringstream).
-- lyx-devel mailing list [email protected] http://lists.lyx.org/mailman/listinfo/lyx-devel
