<div dir="ltr">I'm just upping this conversation: can I push the patches to rename the classes to MathMLStream and TeXMathStream? <br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 28 Dec 2020 at 20:18, Thibaut Cuvelier <<a href="mailto:tcuvelier@lyx.org">tcuvelier@lyx.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 28 Dec 2020 at 19:33, Jean-Marc Lasgouttes <<a href="mailto:lasgouttes@lyx.org" target="_blank">lasgouttes@lyx.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div class="gmail_quote">Le 26 décembre 2020 20:07:19 GMT+01:00, Thibaut Cuvelier <<a href="mailto:tcuvelier@lyx.org" target="_blank">tcuvelier@lyx.org</a>> a écrit :<blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div dir="ltr"><div>Dear list, <br></div><div><br></div><div>In <a href="https://www.lyx.org/trac/ticket/12055" target="_blank">https://www.lyx.org/trac/ticket/12055</a>, 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). <br></div><div><br></div><div>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? <br></div><div><br></div><div>Thibaut<br></div></div>
</blockquote></div><br clear="all">What you propose makes sense, but LaTeXMathStream is a bit long. What about using just MathStream? Or TeXMathStream like we have otexstream.<br></div></blockquote><div><br></div><div>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. <br></div><div>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.</div><div><br></div><div>I like 
TeXMathStream, though.

</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>Also, I am not sure why math streams use different capitalization rules than their text counterpart. <br></div></blockquote><div><br></div><div>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.</div><div><br></div><div>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).<br></div></div></div>
</blockquote></div>