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
next prev parent 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).