Re: [HarfBuzz] Disable uniscribe backend

2011-08-24 Thread Behdad Esfahbod
On 08/20/11 14:47, Grigori Goronzy wrote:
 Hi,
 
 the Uniscribe backend doesn't work on Windows XP and unnecessarily

Is that because we use the OpenType version of ScriptShape, etc?  In that case
I think I should switch to a GetProcAddress() method which is common practice
on Windows then.


 bloats HarfBuzz on Windows. According to commit logs it only is for
 testing purposes anyway, so can we have an option to disable it at build
 time?

Yes, we will have for everything.  I just haven't got to building the
configure system properly yet...


 Grigori
 
 PS: what happened to the 0.8 release and the promises of API stability?

I've been on vacation last week and this week, but I'm getting some hacking
done.  I'm integrating Graphite today.  It was necessary to integrate at least
a couple other backends to kick the last quirks out of the API.  So that's for
the API stability.  As for the 0.8 release, well, I just have been lazy :).
___
HarfBuzz mailing list
HarfBuzz@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/harfbuzz


[HarfBuzz] Disable uniscribe backend

2011-08-20 Thread Grigori Goronzy
Hi,

the Uniscribe backend doesn't work on Windows XP and unnecessarily
bloats HarfBuzz on Windows. According to commit logs it only is for
testing purposes anyway, so can we have an option to disable it at build
time?

Grigori

PS: what happened to the 0.8 release and the promises of API stability?
___
HarfBuzz mailing list
HarfBuzz@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/harfbuzz


Re: [HarfBuzz] Disable uniscribe backend

2011-08-20 Thread Grigori Goronzy
Hi again,

here's a patch that adds a simple configure switch to select Uniscribe.
I disabled Uniscribe support by default.

Similar switches for ICU and glib might be handy as well.

Grigori
From 7f0b6832cc17e76012d249c7ccb536da59aa35a3 Mon Sep 17 00:00:00 2001
From: Grigori Goronzy greg@blackbox
Date: Sat, 20 Aug 2011 18:34:23 +0200
Subject: [PATCH 1/2] Add Uniscribe configure toggle, disabled by default

---
 configure.ac |7 ++-
 1 files changed, 6 insertions(+), 1 deletions(-)

diff --git a/configure.ac b/configure.ac
index f42fb3c..82b5df1 100644
--- a/configure.ac
+++ b/configure.ac
@@ -71,6 +71,9 @@ if test x$GCC = xyes; then
 	esac
 fi
 
+AC_ARG_ENABLE([uniscribe], AS_HELP_STRING([--enable-uniscribe],
+[enable uniscribe backend @:@default=no@:@]))
+
 dnl ==
 
 PKG_CHECK_MODULES(GLIB, glib-2.0 = 2.16, have_glib=true, have_glib=false)
@@ -157,7 +160,9 @@ AM_CONDITIONAL(HAVE_OT, $have_ot)
 
 dnl ===
 
-AC_CHECK_HEADERS(usp10.h windows.h, have_uniscribe=true, have_uniscribe=false)
+if test x$enable_uniscribe = xyes; then
+AC_CHECK_HEADERS(usp10.h windowsx.h, have_uniscribe=true, have_uniscribe=false)
+fi
 if $have_uniscribe; then
 	UNISCRIBE_CFLAGS=
 	UNISCRIBE_LIBS=-lusp10 -lgdi32
-- 
1.7.2.3

___
HarfBuzz mailing list
HarfBuzz@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/harfbuzz