Am Donnerstag, 1. Dezember 2016 um 01:01:44, schrieb Enrico Forestieri <for...@lyx.org> > On Thu, Dec 01, 2016 at 12:35:49AM +0100, Enrico Forestieri wrote: > > On Wed, Nov 30, 2016 at 05:23:59PM +0100, Enrico Forestieri wrote: > > > On Wed, Nov 30, 2016 at 10:40:09AM -0500, Scott Kostyshak wrote: > > > > > > > > Thanks for the explanations. It seems we should report the LyX display > > > > bug you mention above, and then just not worry about the compilation > > > > failures. > > > > > > I found that enclosing the nested macros in brace insets circumvents > > > both issues (display glitch and latex error). Unfortunately, lyx duly > > > removes the braces when reloading the document, so that this only > > > takes care of the display issue, in practice. See attached. > > > > The attached patch fixes the display glitch for me. > > > > > So, the conclusion is that the error is actually due to a limitation > > > of xkeyval. > > > > The only way to avoid this error still remains enclosing a nested macro > > and its parameters in a brace inset. However, this workaround will be > > frustrated by lyx on reloading the document. > > Or we can add the braces on output, as done in the attached patch (also > including the previous one). The braces will be added anytime a macro > has optional arguments (not only when it is nested), but, given that this > avoids a latex error, I think it is acceptable. > > The test case now displays and compiles fine for me. >
Confirmed, works very nice. Please commit. Kornel
Description: This is a digitally signed message part.