On Thu, Jun 18, 2020 at 02:03:38AM +0200, Thibaut Cuvelier wrote: > On Mon, 15 Jun 2020 at 23:51, Pavel Sanda <sa...@lyx.org> wrote: > > > On Mon, Jun 15, 2020 at 02:46:47PM +0200, Thibaut Cuvelier wrote: > > > Here is a new version of the first patch. Indeed, I forgot to completely > > > refactor a few font-related things. It should be much better now. (I also > > > > Ok, we are bit closer, I committed update of first patch. > > > > There is still missing id='magicparlabel-XX' part which need to be fixed. > > Please check xHTML output of e.g. Math manual to be identical. > > > > Here is a new version of the patches. Among the changes: > - removal of a TODO for xml::ParTag (the constructor with parid is no more > used anywhere). ParTag still exists, as it is used to perform some dispatch > in the stream. > - restoration of the parids (looks like it was a reversed condition at some > point). > - slight code cleanups (chain the << calls in streams, as is done in most > places), in a separate commit. > A quick note: the MathML patches remove useless spaces around symbols (they > are not present in output from MathType, for instance). This is similar to > how simpler things are already handled: before this patch, LyX output > <mi>A</mi> but <mi> λ </mi>; now, there are no spaces for lambda. > The remaining differences are due to previous differences in treatment > between the old DocBook and the existing HTML exporters worked.
Thanks, I committed the first patch with consts added (binding to temporaries). I propose to drop the second patch altogether. Unless you use simple expressions chaining is not particularly safe, << is not a sequence point so order of evaluation is undefined. Pavel -- lyx-devel mailing list lyx-devel@lists.lyx.org http://lists.lyx.org/mailman/listinfo/lyx-devel
