Le 11/04/2016 18:33, Richard Heck a écrit :
Here is a very lightly tested patch. It looks OK to me, but this kind of changes has potential to do more harm than good. So if we do not have 1/ good user testing and 2/ a +1 from someone who understands this code, I am not sure we want it in rc1. And rc1 really needs to happen soon IMO.This is exactly what I was going to do. A quicker and dirtier and very safe (IMO) fix is: @@ -75,6 +75,8 @@ void InsetFlex::write(ostream & os) const // use il.name(), since this resolves obsoleted // InsetLayout names string name = to_utf8(il.name()); + if (name == from_ascii("undefined")) + name == name_; // Remove the "Flex:" prefix, if it is present if (support::prefixIs(name, "Flex:")) name = support::token(name, ':', 1);
Definitely not code that we would want to keep, IMO, but maybe safer for rc1, indeed.
JMarc
