unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: YAMAMOTO Mitsuharu <mituharu@math.s.chiba-u.ac.jp>
To: martin rudalics <rudalics@gmx.at>
Cc: 16306@debbugs.gnu.org
Subject: bug#16306: 24.3.50; font change creates blank column on Lucid
Date: Thu, 02 Jan 2014 14:37:49 +0900	[thread overview]
Message-ID: <wlob3vj64y.wl%mituharu@math.s.chiba-u.ac.jp> (raw)
In-Reply-To: <52C3F773.1030804@gmx.at>

>>>>> On Wed, 01 Jan 2014 12:09:39 +0100, martin rudalics <rudalics@gmx.at> said:

>> On my system, (640 16 16 2) before and (1120 28 16 2) after the
>> font change.

> 28 for the fringe width doesn't look right, this should be 30 since
> a character is 15 pixels wide with the new font.  So there could be
> a bug in compute_fringe_widths, a function I never touched.  But
> this still would not explain a difference of four pixels.

There is actually nothing wrong with the calculation of the fringe
width, because the character width for the new font (Courier-18) is 14
on my side (14 * 80 = 1120).

> Could you try starting with GDB in x_new_font and tell me which
> value of "width" is passed to x_set_window_size_1 and how the latter
> calculates "pixelwidth" which it passes to XResizeWindow?

The value of "width" is 1120 as of the entry to x_set_window_size_1,
and the value of "pixelwidth" is 1166 at the call to XResizeWindow.

It is the window manager "quartz-wm" that reduces the width from 1166
to 1162, probably because it respects the size hint that was set in
update_wm_hints (in widget.c).  There, the value of base_width/height
is calculated using the width/height of the frame *before* resizing.

> And please also try the following patch (as mentioned earlier I
> never understood that calculation and what the value "14" is
> supposed to accomplish).  For me it doesn't change anything - maybe
> it does for you.

It doesn't change the behavior for me, either.

				     YAMAMOTO Mitsuharu
				mituharu@math.s.chiba-u.ac.jp





  reply	other threads:[~2014-01-02  5:37 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-31  2:27 bug#16306: 24.3.50; font change creates blank column on Lucid YAMAMOTO Mitsuharu
2013-12-31  8:05 ` Eli Zaretskii
2013-12-31  9:48   ` YAMAMOTO Mitsuharu
2013-12-31  9:59     ` Eli Zaretskii
2013-12-31 11:33     ` martin rudalics
2013-12-31 11:51       ` YAMAMOTO Mitsuharu
2013-12-31 17:52         ` martin rudalics
2013-12-31 18:05           ` Jan Djärv
2013-12-31 18:58             ` martin rudalics
2014-01-01  4:51           ` YAMAMOTO Mitsuharu
2014-01-01 11:09             ` martin rudalics
2014-01-02  5:37               ` YAMAMOTO Mitsuharu [this message]
2014-01-02 18:13                 ` martin rudalics
2014-01-05  5:08             ` YAMAMOTO Mitsuharu
2014-01-05 10:38               ` martin rudalics
2019-11-02 16:15                 ` Lars Ingebrigtsen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=wlob3vj64y.wl%mituharu@math.s.chiba-u.ac.jp \
    --to=mituharu@math.s.chiba-u.ac.jp \
    --cc=16306@debbugs.gnu.org \
    --cc=rudalics@gmx.at \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).