Jürgen Spitzmüller wrote:

> As I wrote, I do not understand what the method is supposed to do. When we
> have argument insets for initials, why bother converting ERT?

It is not only used for initials, but in a lot more places. It is supposed 
to resolve hacks like an ERT "}{" or which Uwe used a lot before 
InsetArgument supported position indices.

IMHO it was a mistake to add convert_TeX_brace_to_Argument() at all: It 
works only for a restricted set of layouts which are used in the LyX 
documentation. It does not work for user defined layouts which might need 
the same hack in older versions. Furthermore, it is not a file format change 
in the strict sense, but tries to get rid of some workarounds which were 
needed in the old version. This can never work 100%, and IMHO we should 
never try to interpret ERT.

However, now we have this beast, and it is too late to get rid of it. I 
briefly looked at Richrads latest patch, and tested that it does not break 
LyX documentation roundtrip, so I believe it could go in. However, if 
possible it would be good if somebody could also test itz with some eother 
layouts which use convert_TeX_brace_to_Argument(), e.g. from IEEEtran (I 
don't have such files).


Georg


Reply via email to