Issue 62039 - Don't use X fonts by default
Summary: Don't use X fonts by default
Status: CLOSED FIXED
Alias: None
Product: gsl
Classification: Code
Component: code (show other issues)
Version: OOo 2.0.2
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo 2.2
Assignee: eric.savary
QA Contact: issues@gsl
URL:
Keywords:
Depends on:
Blocks: 62485 62521
  Show dependency tree
 
Reported: 2006-02-14 11:13 UTC by kendy
Modified: 2010-06-18 14:50 UTC (History)
6 users (show)

See Also:
Issue Type: PATCH
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description kendy 2006-02-14 11:13:13 UTC 
The default of using/non-using X fonts should be changed - as discussed in 
issue 35641...  We currently use the following in ooo-build: 
 
--- vcl/unx/source/gdi/salgdi3.cxx      2006-02-14 11:44:42.000000000 +0100 
+++ vcl/unx/source/gdi/salgdi3.cxx      2006-02-14 12:01:55.155856532 +0100 
@@ -1264,7 +1264,7 @@ bool X11SalGraphics::AddTempDevFont( Imp 
 void X11SalGraphics::GetDevFontList( ImplDevFontList *pList ) 
 { 
     static const char* pEnableX11FontStr = 
getenv( "SAL_ENABLE_NATIVE_XFONTS" ); 
-    if( !pEnableX11FontStr || (pEnableX11FontStr[0] != '0') ) 
+    if( pEnableX11FontStr && (pEnableX11FontStr[0] == '1') ) 
     { 
         // announce X11 fonts 
         XlfdStorage* pX11FontList = GetDisplay()->GetXlfdList();
Comment 1 philipp.lohmann 2006-02-14 12:21:07 UTC 
pl->hdu: i think that's on plan for 3.0 anyway ?
Comment 2 hdu@apache.org 2006-02-14 12:24:56 UTC 
Yep. The default will be changed for target OOo3.
Comment 3 simos.bugzilla 2006-02-15 22:45:22 UTC 
Is the issue 57426 (greek fonts not showing properly in several menus) related
to this issue?
Comment 4 mmeeks 2006-02-16 09:40:13 UTC 
simos - easy to check: just 'export SAL_ENABLE_NATIVE_XFONTS=0' before you run
OO.o - if that fixes it: then yes :-)
Comment 5 stx123 2006-08-08 15:39:40 UTC 
Hi Herbert, you might want to reconsider the target.
Comment 6 hdu@apache.org 2006-08-09 08:09:46 UTC 
@ST: I know. I just feel very uncomfortable to switch off functionality in a
minor version, if this functionality worked just fine in the corresponding major
version. Please set a target and reassign the issue back to me.
Comment 7 stx123 2006-10-10 13:08:26 UTC 
2.1 may be a bit short notice. Would 2.2 work for you?
Comment 8 hdu@apache.org 2006-10-10 13:46:24 UTC 
will do
Comment 9 hdu@apache.org 2006-10-30 11:30:19 UTC 
Done in CWS vcl69. Good bye native X11 fonts!
Comment 10 hdu@apache.org 2006-11-22 16:09:30 UTC 
@ES: please verify in CWS vcl69
X11 fonts are no longer used by default. Regressions are not probable, but
please watch out.
Comment 11 eric.savary 2006-11-27 15:54:16 UTC 
Keywords 4 me:
xfontsel
pspfontcache
Comment 12 eric.savary 2007-01-18 12:31:34 UTC 
Ok in m199