After updating the beamer example I noticed several messages

LColor::getFromLyXName: Unknown color "default"

in the terminal. The reason was that the "inherit" color was explicitly 
changed to "default" in lyxfont.C. Unfortunately that obsolete name (which 
is changed to "inherit" by lyx2lyx for format 223) cannot be read anymore. 
The error message is printed, and the color is set to "none".

The attached patch fixes that and goes in trunk and 1.4 now since it is 
clearly needed and correct.

Does nobody use colors with LyX, or why did nobody see it before?


Georg
Index: src/lyxfont.C
===================================================================
--- src/lyxfont.C	(Revision 15651)
+++ src/lyxfont.C	(Arbeitskopie)
@@ -724,14 +724,8 @@ void LyXFont::lyxWriteChanges(LyXFont co
 	if (orgfont.noun() != noun()) {
 		os << "\\noun " << LyXMiscNames[noun()] << "\n";
 	}
-	if (orgfont.color() != color()) {
-		// To make us file compatible with older
-		// lyx versions we emit "default" instead
-		// of "inherit"
-		string col_str(lcolor.getLyXName(color()));
-		if (col_str == "inherit") col_str = "default";
-		os << "\\color " << col_str << "\n";
-	}
+	if (orgfont.color() != color())
+		os << "\\color " << lcolor.getLyXName(color()) << '\n';
 	if (orgfont.language() != language() &&
 	    language() != latex_language) {
 		if (language())

Reply via email to