xterm/bug-246573-tentative-patch.diff
Stephan Kulow 14c2cb85b9 Accepting request 85817 from home:pcerny:factory
- Patch #275 - 2011/09/11
  * add ash, zsh to known shells for resize.
  * modify resize to reuse the logic from xterm which determines
    the actual logon-user's shell if $SHELL is not set.
  * revert the unsetenv("SHELL") added in patch #272.
  * an unsetenv("SHELL") added in patch #272 to help ensure that
    luit would get the user's shell consistently did not work as
    expected for cases where multiple names are in the
    password-file for a given uid. That was because changes in
    patch #157 to handle this situation did not take into account
    that repeated calls to getpwnam and getpwuid return a pointer
    to the same static buffer. Fixed the older logic to work as
    intended, by ensuring that the passwd-data from each call is
    stored separately (report by Paul Keusemann).
  * adjust ifdef's for putenv and unsetenv in case only one of
    those is provided on a given platform.
  * correct comparison used in ExposeContains macro from patch
    #274 changes, to handle window-dragging (patch by Todd
    Eigenschink).

- Patch #274 - 2011/09/05
  * portability fixes for cygwin: do not define SVR4, and work
    around nonstandard header location.
  * ifdef'd use of unsetenv from patch #273 changes to work with
    Solaris 9 (report by Waldemar Rachwal).
  * modify logic for XtAppPending to merge adjacent Expose and
    ConfigureNotify events which are redundant (report by Edward
    McGuire).
  * fix an unneeded warning message when -r option is given.
  * remove a few redundant entries from table used for helping

OBS-URL: https://build.opensuse.org/request/show/85817
OBS-URL: https://build.opensuse.org/package/show/X11:terminals/xterm?expand=0&rev=25
2011-10-12 07:21:46 +00:00

90 lines
3.2 KiB
Diff

--- xterm-275/fontutils.c 2011-09-11 15:22:34.000000000 +0200
+++ xterm-275/fontutils.c 2011-09-29 16:49:57.899419068 +0200
@@ -631,81 +631,33 @@ is_fixed_font(XFontStruct * fs)
static int
is_double_width_font(XFontStruct * fs)
{
return ((2 * fs->min_bounds.width) == fs->max_bounds.width);
}
#else
#define is_double_width_font(fs) 0
#endif
#if OPT_WIDE_CHARS && OPT_RENDERFONT && defined(HAVE_TYPE_FCCHAR32)
-#define HALF_WIDTH_TEST_STRING "1234567890"
-
-/* '1234567890' in Chinese characters in UTF-8 */
-#define FULL_WIDTH_TEST_STRING "\xe4\xb8\x80\xe4\xba\x8c\xe4\xb8\x89" \
- "\xe5\x9b\x9b\xe4\xba\x94" \
- "\xef\xa7\x91\xe4\xb8\x83\xe5\x85\xab" \
- "\xe4\xb9\x9d\xef\xa6\xb2"
-/* '1234567890' in Korean script in UTF-8 */
-#define FULL_WIDTH_TEST_STRING2 "\xec\x9d\xbc\xec\x9d\xb4\xec\x82\xbc" \
- "\xec\x82\xac\xec\x98\xa4" \
- "\xec\x9c\xa1\xec\xb9\xa0\xed\x8c\x94" \
- "\xea\xb5\xac\xec\x98\x81"
-
-#define HALF_WIDTH_CHAR1 0x0031 /* '1' */
-#define HALF_WIDTH_CHAR2 0x0057 /* 'W' */
#define FULL_WIDTH_CHAR1 0x4E00 /* CJK Ideograph 'number one' */
#define FULL_WIDTH_CHAR2 0xAC00 /* Korean script syllable 'Ka' */
static Bool
is_double_width_font_xft(Display * dpy, XftFont * font)
{
- XGlyphInfo gi1, gi2;
- FcChar32 c1 = HALF_WIDTH_CHAR1, c2 = HALF_WIDTH_CHAR2;
- String fwstr = FULL_WIDTH_TEST_STRING;
- String hwstr = HALF_WIDTH_TEST_STRING;
-
/* Some Korean fonts don't have Chinese characters at all. */
- if (!XftCharExists(dpy, font, FULL_WIDTH_CHAR1)) {
- if (!XftCharExists(dpy, font, FULL_WIDTH_CHAR2))
- return False; /* Not a CJK font */
- else /* a Korean font without CJK Ideographs */
- fwstr = FULL_WIDTH_TEST_STRING2;
- }
-
- XftTextExtents32(dpy, font, &c1, 1, &gi1);
- XftTextExtents32(dpy, font, &c2, 1, &gi2);
- if (gi1.xOff != gi2.xOff) /* Not a fixed-width font */
- return False;
-
- XftTextExtentsUtf8(dpy,
- font,
- (_Xconst FcChar8 *) hwstr,
- (int) strlen(hwstr),
- &gi1);
- XftTextExtentsUtf8(dpy,
- font,
- (_Xconst FcChar8 *) fwstr,
- (int) strlen(fwstr),
- &gi2);
-
- /*
- * fontconfig and Xft prior to 2.2(?) set the width of half-width
- * characters identical to that of full-width character in CJK double-width
- * (bi-width / monospace) font even though the former is half as wide as
- * the latter. This was fixed sometime before the release of fontconfig
- * 2.2 in early 2003. See
- * http://bugzilla.mozilla.org/show_bug.cgi?id=196312
- * In the meantime, we have to check both possibilities.
- */
- return ((2 * gi1.xOff == gi2.xOff) || (gi1.xOff == gi2.xOff));
+ if (XftCharExists(dpy, font, FULL_WIDTH_CHAR1) ||
+ XftCharExists(dpy, font, FULL_WIDTH_CHAR2))
+ return True; /* CJK font */
+ else
+ return False; /* Not a CJK font */
}
#else
#define is_double_width_font_xft(dpy, xftfont) 0
#endif
#define EmptyFont(fs) (fs != 0 \
&& ((fs)->ascent + (fs)->descent == 0 \
|| (fs)->max_bounds.width == 0))
#define FontSize(fs) (((fs)->ascent + (fs)->descent) \