Hello community,
here is the log from the commit of package xorg-x11
checked in at Fri Jun 23 17:15:04 CEST 2006.
--------
--- xorg-x11/xorg-x11.changes 2006-06-21 22:06:00.000000000 +0200
+++ xorg-x11/xorg-x11.changes 2006-06-23 10:11:33.000000000 +0200
@@ -1,0 +2,7 @@
+Fri Jun 23 08:29:18 CEST 2006 - sndirsch@suse.de
+
+- libXfont-X11R7.0-1.0.0-no-ft-internals.patch:
+ * removes use of internal freetype stuff
+ * obsoletes p_freetype-crashfix.diff (Bug #97202)
+
+-------------------------------------------------------------------
Old:
----
p_freetype-crashfix.diff
New:
----
libXfont-X11R7.0-1.0.0-no-ft-internals.patch
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ xorg-x11.spec ++++++
--- /var/tmp/diff_new_pack.54tF6D/_old 2006-06-23 17:14:21.000000000 +0200
+++ /var/tmp/diff_new_pack.54tF6D/_new 2006-06-23 17:14:21.000000000 +0200
@@ -40,7 +40,7 @@
%endif
Autoreqprov: on
Version: 6.9.0
-Release: 62
+Release: 63
Summary: The basic X Window System package
Source: xorg-6.9.0.tar.bz2
Source1: xdm.tar.gz
@@ -70,7 +70,6 @@
Patch237: p_xlc-locale.diff
Patch238: p_sdk.diff
Patch276: p_xlib-threadsafe.diff
-Patch280: p_freetype-crashfix.diff
Patch282: p_new-locales.diff
Patch283: p_xauth.diff
Patch287: p_sr_CS-locales.diff
@@ -125,6 +124,7 @@
Patch366: pc_xf86-pci.diff
Patch367: p_xlib_skip_ext_env.diff
Patch368: p_xnest-ignore-getimage-errors.diff
+Patch369: libXfont-X11R7.0-1.0.0-no-ft-internals.patch
Patch400: p_bug96328.diff
BuildRoot: %{_tmppath}/%{name}-%{version}-build
@@ -1474,7 +1474,6 @@
pushd lib/X11
%patch276
popd
-%patch280
### disabled for now (Bug #70890)
#%patch232 -p0
%patch308 -p0 -b .mouse_misc
@@ -1565,6 +1564,9 @@
%patch367 -p0
popd
%patch368
+pushd lib/font
+%patch369 -p2
+popd
#
UTS_RELEASE=`uname -r`
UTS_MAJOR=${UTS_RELEASE%%%%.*}
@@ -1577,7 +1579,7 @@
RPM_OPT_FLAGS=$(echo $RPM_OPT_FLAGS | sed 's/-D_FORTIFY_SOURCE=2//g')
echo "RPM_OPT_FLAGS: $RPM_OPT_FLAGS"
cat << EOF > config/cf/host.def
-#define LinuxDistribution LinuxSUSE
+#define LinuxDistribution LinuxSuSE
#undef OSVendor
#define OSVendor SUSE
#undef DefaultOSName
@@ -2722,6 +2724,10 @@
%clean
%changelog -n xorg-x11
+* Fri Jun 23 2006 - sndirsch@suse.de
+- libXfont-X11R7.0-1.0.0-no-ft-internals.patch:
+ * removes use of internal freetype stuff
+ * obsoletes p_freetype-crashfix.diff (Bug #97202)
* Wed Jun 21 2006 - sndirsch@suse.de
- ps_glxproto.diff:
* adjusted to new defines of Mesa 6.5
++++++ FILES ++++++
--- xorg-x11/FILES 2006-06-14 06:24:33.000000000 +0200
+++ xorg-x11/FILES 2006-06-23 08:54:43.000000000 +0200
@@ -58,8 +58,6 @@
- panoramiXproto.h requrired for compiling xgi driver
p_xlib-threadsafe.diff [KEEP?]
- don't have the input filter wrapped by the display lock (Bug #55962)
-p_freetype-crashfix.diff [KEEP?]
-- fixes freetype related crash with freetype 2.1.10 (Bug #97202)
p_new-locales.diff [KEEP?]
- support for new indic/african locales (Bug #98262)
p_xauth.diff [KEEP?]
@@ -179,3 +177,5 @@
variables (Bug #167317)
p_xnest-ignore-getimage-errors.diff [KEEP?]
- ignores the X error on GetImage in Xnest (Bug #174228, X.Org Bug #4411)
+libXfont-X11R7.0-1.0.0-no-ft-internals.patch [KEEP?]
+- removes use of internal freetype stuff
++++++ libXfont-X11R7.0-1.0.0-no-ft-internals.patch ++++++
diff -ubrN libXfont-X11R7.0-1.0.0/src/FreeType/ftfuncs.c libXfont-X11R7.0-1.0.0-new/src/FreeType/ftfuncs.c
--- libXfont-X11R7.0-1.0.0/src/FreeType/ftfuncs.c 2005-10-23 20:32:05.000000000 +0200
+++ libXfont-X11R7.0-1.0.0-new/src/FreeType/ftfuncs.c 2006-05-17 13:50:17.527499000 +0200
@@ -54,10 +54,7 @@
#include FT_TYPE1_TABLES_H
#include FT_XFREE86_H
#include FT_BBOX_H
-#include FT_INTERNAL_TRUETYPE_TYPES_H
#include FT_TRUETYPE_TAGS_H
-#include FT_INTERNAL_SFNT_H
-#include FT_INTERNAL_STREAM_H
/*
* If you want to use FT_Outline_Get_CBox instead of
* FT_Outline_Get_BBox, define here.
@@ -123,6 +120,46 @@
};
+/* read 2-byte value from a SFNT table */
+static FT_UShort
+sfnt_get_ushort( FT_Face face,
+ FT_ULong table_tag,
+ FT_ULong table_offset )
+{
+ FT_Byte buff[2];
+ FT_ULong len = sizeof(buff);
+ FT_UShort result = 0;
+
+ if ( !FT_Load_Sfnt_Table( face, table_tag, table_offset, buff, &len ) );
+ result = (FT_UShort)( (buff[0] << 8) | buff[1] );
+
+ return result;
+}
+
+#define sfnt_get_short(f,t,o) ((FT_Short)sfnt_get_ushort((f),(t),(o)))
+
+
+#if 0
+static FT_ULong
+sfnt_get_ulong( FT_Face face,
+ FT_ULong table_tag,
+ FT_ULong table_offset )
+{
+ FT_Byte buff[4];
+ FT_ULong len = sizeof(buff);
+ FT_UShort result = 0;
+
+ if ( !FT_Load_Sfnt_Table( face, table_tag, table_offset, buff, &len );
+ result = ((FT_ULong)buff[0] << 24) | ((FT_ULong)buff[1] << 16) |
+ ((FT_ULong)buff[2] << 8) | (FT_ULong)buff[3];
+
+ return result;
+}
+#endif
+
+
+
+
static int ftypeInitP = 0; /* is the engine initialised? */
FT_Library ftypeLibrary;
@@ -211,6 +248,10 @@
if(maxp && maxp->maxContours == 0)
face->bitmap = 1;
}
+
+ face->num_hmetrics = (FT_UInt) sfnt_get_ushort( face->face,
+ TTAG_hhea, 34 );
+
/* Insert face in hashtable and return it */
face->next = faceTable[bucket];
faceTable[bucket] = face;
@@ -462,6 +503,29 @@
}
if( FT_IS_SFNT( face->face ) ) {
+#if 1
+ FT_F26Dot6 tt_char_width, tt_char_height, tt_dim_x, tt_dim_y;
+ FT_UInt nn;
+
+ instance->strike_index=0xFFFFU;
+
+ tt_char_width = (int)(trans->scale*(1<<6) + 0.5);
+ tt_char_height = (int)(trans->scale*(1<<6) + 0.5);
+
+ tt_dim_x = ( ( tt_char_width * trans->xres + (36+32*72) ) / 72 ) & -64;
+ tt_dim_y = ( ( tt_char_height * trans->yres + (36+32*72) ) / 72 ) & -64;
+
+ for ( nn = 0; nn < face->face->num_fixed_sizes; nn++ )
+ {
+ FT_Bitmap_Size* sz = &face->face->available_sizes[nn];
+
+ if ( tt_dim_x == sz->x_ppem && tt_dim_y == sz->y_ppem )
+ {
+ instance->strike_index = nn;
+ break;
+ }
+ }
+#else
/* See Set_Char_Sizes() in ttdriver.c */
FT_Error err;
TT_Face tt_face;
@@ -486,6 +550,7 @@
sfnt = (SFNT_Service)tt_face->sfnt;
err = sfnt->set_sbit_strike(tt_face,tt_x_ppem,tt_y_ppem,&instance->strike_index);
if ( err ) instance->strike_index=0xFFFFU;
+#endif
}
/* maintain a linked list of instances */
@@ -803,31 +868,36 @@
* parse the htmx field in TrueType font.
*/
-/* from src/truetype/ttgload.c */
static void
-tt_get_metrics( TT_HoriHeader* header,
+tt_get_metrics( FT_Face face,
FT_UInt idx,
+ FT_UInt num_hmetrics,
FT_Short* bearing,
FT_UShort* advance )
-/* Copyright 1996-2001, 2002 by */
-/* David Turner, Robert Wilhelm, and Werner Lemberg. */
{
- TT_LongMetrics longs_m;
- FT_UShort k = header->number_Of_HMetrics;
+ /* read the metrics directly from the horizontal header, we
+ * parse the SFNT table directly through the standard FreeType API.
+ * this works with any version of the library and doesn't need to
+ * peek at its internals. Maybe a bit less
+ */
+ FT_UInt count = num_hmetrics;
- if ( k == 0 ) {
- *bearing = *advance = 0;
- return;
+ if ( count == 0 )
+ {
+ *advance = 0;
+ *bearing = 0;
}
+ else if ( idx < count )
+ {
+ FT_ULong offset = idx*4;
- if ( idx < (FT_UInt)k ) {
- longs_m = (TT_LongMetrics )header->long_metrics + idx;
- *bearing = longs_m->bearing;
- *advance = longs_m->advance;
+ *advance = sfnt_get_ushort( face, TTAG_hmtx, offset );
+ *bearing = sfnt_get_short ( face, TTAG_hmtx, offset+2 );
}
- else {
- *bearing = ((TT_ShortMetrics*)header->short_metrics)[idx - k];
- *advance = ((TT_LongMetrics )header->long_metrics)[k - 1].advance;
+ else
+ {
+ *advance = sfnt_get_ushort( face, TTAG_hmtx, (count-1)*4 );
+ *bearing = sfnt_get_short ( face, TTAG_hmtx, (count+idx)*2 );
}
}
@@ -835,6 +905,7 @@
ft_get_very_lazy_bbox( FT_UInt index,
FT_Face face,
FT_Size size,
+ FT_UInt num_hmetrics,
double slant,
FT_Matrix *matrix,
FT_BBox *bbox,
@@ -842,14 +913,13 @@
FT_Long *vertAdvance)
{
if ( FT_IS_SFNT( face ) ) {
- TT_Face ttface = (TT_Face)face;
FT_Size_Metrics *smetrics = &size->metrics;
FT_Short leftBearing = 0;
FT_UShort advance = 0;
FT_Vector p0, p1, p2, p3;
/* horizontal */
- tt_get_metrics(&ttface->horizontal, index,
+ tt_get_metrics( face, index, num_hmetrics,
&leftBearing, &advance);
#if 0
@@ -910,7 +980,27 @@
FT_UShort glyph_index, FT_Glyph_Metrics *metrics_return,
int *sbitchk_incomplete_but_exist )
{
-#if (FREETYPE_VERSION >= 2001008)
+#if 1
+ if ( strike_index != 0xFFFFU && ft_face->available_sizes != NULL )
+ {
+ FT_Error error;
+ FT_Bitmap_Size* sz = &ft_face->available_sizes[strike_index];
+
+ error = FT_Set_Pixel_Sizes( ft_face, sz->x_ppem/64, sz->y_ppem/64 );
+ if ( !error )
+ {
+ error = FT_Load_Glyph( ft_face, glyph_index, FT_LOAD_SBITS_ONLY );
+ if ( !error )
+ {
+ if ( metrics_return != NULL )
+ *metrics_return = ft_face->glyph->metrics;
+
+ return 0;
+ }
+ }
+ }
+ return -1;
+#elif (FREETYPE_VERSION >= 2001008)
SFNT_Service sfnt;
TT_Face face;
FT_Error error;
@@ -1043,6 +1133,7 @@
if( bitmap_metrics == NULL ) {
if ( sbitchk_incomplete_but_exist==0 && (instance->ttcap.flags & TTCAP_IS_VERY_LAZY) ) {
if( ft_get_very_lazy_bbox( idx, face->face, instance->size,
+ face->num_hmetrics,
instance->ttcap.vl_slant,
&instance->transformation.matrix,
&bbox, &outline_hori_advance,
@@ -1221,6 +1312,7 @@
if( is_outline == 1 ){
if( correct ){
if( ft_get_very_lazy_bbox( idx, face->face, instance->size,
+ face->num_hmetrics,
instance->ttcap.vl_slant,
&instance->transformation.matrix,
&bbox, &outline_hori_advance,
diff -ubrN libXfont-X11R7.0-1.0.0/src/FreeType/ftfuncs.h libXfont-X11R7.0-1.0.0-new/src/FreeType/ftfuncs.h
--- libXfont-X11R7.0-1.0.0/src/FreeType/ftfuncs.h 2005-07-07 16:59:47.000000000 +0200
+++ libXfont-X11R7.0-1.0.0-new/src/FreeType/ftfuncs.h 2006-05-17 11:16:33.304210000 +0200
@@ -47,6 +47,7 @@
char *filename;
FT_Face face;
int bitmap;
+ FT_UInt num_hmetrics;
struct _FTInstance *instances;
struct _FTInstance *active_instance;
struct _FTFace *next; /* link to next face in bucket */
diff -ubrN libXfont-X11R7.0-1.0.0/src/FreeType/ftsystem.c libXfont-X11R7.0-1.0.0-new/src/FreeType/ftsystem.c
--- libXfont-X11R7.0-1.0.0/src/FreeType/ftsystem.c 2005-07-09 08:36:10.000000000 +0200
+++ libXfont-X11R7.0-1.0.0-new/src/FreeType/ftsystem.c 2006-05-17 13:58:07.177239000 +0200
@@ -35,7 +35,6 @@
#endif
#include