[LyX/master] Remove redundant font loader code

Scott Kostyshak skostysh at lyx.org
Fri Mar 20 13:00:07 UTC 2020


commit f8e40f8ab8d5f9daa66a56eef67426a8381b361f
Author: Scott Kostyshak <skostysh at lyx.org>
Date:   Fri Mar 20 09:12:47 2020 -0400

    Remove redundant font loader code
    
    Setting the family to "family" is already attempted above. A
    different case was adapted to be this duplicate case at d4ff3a29,
    but it should have just been removed.
    
    Spotted by Enrico.
---
 src/frontends/qt/GuiFontLoader.cpp |    9 ---------
 1 files changed, 0 insertions(+), 9 deletions(-)

diff --git a/src/frontends/qt/GuiFontLoader.cpp b/src/frontends/qt/GuiFontLoader.cpp
index 69dba8d..cf73292 100644
--- a/src/frontends/qt/GuiFontLoader.cpp
+++ b/src/frontends/qt/GuiFontLoader.cpp
@@ -203,15 +203,6 @@ QFont symbolFont(QString const & family, bool * ok)
 		return font;
 	}
 
-	LYXERR(Debug::FONT, "Trying " << family << " ... ");
-	font.setFamily(family);
-
-	if (isChosenFont(font, family, QString())) {
-		LYXERR(Debug::FONT, "raw version!");
-		*ok = true;
-		return font;
-	}
-
 	LYXERR(Debug::FONT, " FAILED :-(");
 	*ok = false;
 	return font;


More information about the lyx-cvs mailing list