comparison src/xterm.c @ 32938:4b9c0e6751e7

(x_draw_glyph_string): Add a workaround so that fonts that specify a negative underline position can still use underlines.

32937:dc58e218a878

	{
	  unsigned long dy, h;

	  if (!XGetFontProperty (s->font, XA_UNDERLINE_THICKNESS, &h))
	    h = 1;
	  if (!XGetFontProperty (s->font, XA_UNDERLINE_POSITION, &dy))
	    dy = s->height - h;
      
	  if (s->face->underline_defaulted_p)
	    XFillRectangle (s->display, s->window, s->gc, s->x, s->y + dy,
			    s->width, h);

32938:4b9c0e6751e7

	{
	  unsigned long dy, h;

	  if (!XGetFontProperty (s->font, XA_UNDERLINE_THICKNESS, &h))
	    h = 1;
	  if (!XGetFontProperty (s->font, XA_UNDERLINE_POSITION, &dy)
	      /* If the font specifies a negative underline position,
		 we'll get a huge positive number, because dy is
		 unsigned.  This comparison is a workaround that just
		 ignores the font's underline position in that case.  XXX */
	      || dy > s->height)
	    dy = s->height - h;
      
	  if (s->face->underline_defaulted_p)
	    XFillRectangle (s->display, s->window, s->gc, s->x, s->y + dy,
			    s->width, h);