[LyX/master] Rename MathStream to MathMLStream.

Pavel Sanda sanda at lyx.org
Wed Jan 6 12:28:17 UTC 2021


On Sun, Jan 03, 2021 at 06:46:18PM +0100, Kornel Benko wrote:
> Am Sun, 3 Jan 2021 12:35:36 -0500
> schrieb Scott Kostyshak <skostysh at lyx.org>:
> 
> > On Sun, Jan 03, 2021 at 06:16:48PM +0100, Pavel Sanda wrote:
> > > On Sat, Jan 02, 2021 at 07:52:27PM +0100, Thibaut Cuvelier wrote:  
> > > > commit ecafea8227d0c51f36dbf8ff45e9c3ebc15e19cf
> > > > Author: Thibaut Cuvelier <tcuvelier at lyx.org>
> > > > Date:   Sat Dec 26 20:02:46 2020 +0100
> > > > 
> > > >     Rename MathStream to MathMLStream.  
> > > 
> > > This is what I see in older Qt/gcc:  
> > 
> > I also get the following:
> > 
> > /home/scott/lyxbuilds/master/repo/src/lyxfind.cpp:3060:12: error: ???int
> > lyx::{anonymous}::computeSize(QStringRef, int)??? defined but not used
> > [-Werror=unused-function] 3060 | static int computeSize(QStringRef s, int len)
> > |            ^~~~~~~~~~~
> > 
> > Scott
> 
> Yes, sorry, I am still fighting.
> (Locally already commented out)
> Cannot commit, since the source is full of debug output, but there is hope.
> The newest version is(many!) times faster than before, although regex-searching without
> enabled formats is faster.

The offeding commit for the first error is not ecafea8227d, but 8c67cb8c:
Author: Kornel Benko <kornel at lyx.org>
Date:   Sat Jan 2 18:37:14 2021 +0100

    Amend f500a287 (FindAdv: Try to make regex search with format enabled somehow faster)
    
    Remove 1 out of range access,
    Estimate the search result of regular expression for further processing

Pavel


More information about the lyx-devel mailing list