> It works for me. Great! > > > > I've not yet investigated the XeTeX version of > instant-preview, and > > > > how it works. I'll do it. > > > > Not managed in this patch. How many people use XeTeX? > > Well, it seems that instant-preview is semi-broken when using XeTeX. > The preview snippets are generated but they are not cropped, such that > any of them takes the space of a full page... > Given that, I'd say that your patch is OK, even if it does > not addresses > XeTeX, because instant-preview is actually unusable in this case.
Right now I'm quite busy, maybe in future we could go back to this. > > I also changed PreviewLoader.cpp to ignore files referenced > from metrics file > > but not actually existing. It was causing a SIGSEV in > another part of the code. > > Good catch! I'd already noticed it but had no time to investigate. > Your patch is an improvement over the current situation, so I am going > to commit a revised version. Very well! I saw you removed the .dtl file generation, I changed that from the first patch thinking that a DTL of 500 shouldn't stay in memory, however it's 133 kB for Math.lyx, it shouldn't be a serious problem. You also renamed the legacy version of extrac_metrics_info, that's a good idea. Thanks ;) > There is a number of preview snippets failing in the math manual, but > it seems that hyperref is the culprit here. For example, see: > https://bugs.launchpad.net/ubuntu/+source/auctex/+bug/524961 > https://bugs.launchpad.net/ubuntu/+source/auctex/+bug/592509 > http://bugs.gentoo.org/show_bug.cgi?id=267930 Don't know, we should try them with pdflatex, it worked with TikZ, but I think it works just beacuse pdflatex ignores the errors while a raw call to GhostScript fails, but I'm not sure. Do you think we should (or makes sense to) ask GhostScript developers to implement a switch to ignore errors? In future we could better investigate this. However it's not a LyX problem, we can't fix all the compatibility issues preview-latex has, except those we're personally intrested in, like TikZ :D Maybe it's a dumb question but... What's the use of preview-latex? Couldn't we simply create a page for each snippet and then make the usual calls to the converters? Maybe this would simplify the process and remove all the incompatibilities. But for the simple fact that a package like preview-latex exists I think I'm missing something. Julien? Thanks for your support in this patch, Enrico! venom00
