Review and drop old frame resize hack.

* frame.h (struct frame): Remove force_flush_display_p.
* dispnew.c (update_frame): Adjust user and don't call
flush_frame here.  The comment has said that there was an issues
with redisplaying fringes, but I don't see any differences with
and without this hack.  Hopefully we can continue without it.
* xdisp.c (clear_garbaged_frames): Adjust user and do not clear
current frame matrices twice if resized_p is set.
This commit is contained in:
Dmitry Antipov 2013-09-09 20:47:43 +04:00
parent 71bd1a0084
commit 0750a308d0
4 changed files with 14 additions and 19 deletions

View file

@ -1,3 +1,14 @@
2013-09-09 Dmitry Antipov <dmantipov@yandex.ru>
Review and drop old frame resize hack.
* frame.h (struct frame): Remove force_flush_display_p.
* dispnew.c (update_frame): Adjust user and don't call
flush_frame here. The comment has said that there was an issues
with redisplaying fringes, but I don't see any differences with
and without this hack. Hopefully we can continue without it.
* xdisp.c (clear_garbaged_frames): Adjust user and do not clear
current frame matrices twice if resized_p is set.
2013-09-09 Dmitry Antipov <dmantipov@yandex.ru>
Do not populate pure Xism x_sync to other ports.

View file

@ -3061,16 +3061,6 @@ update_frame (struct frame *f, bool force_p, bool inhibit_hairy_id_p)
/* Update windows. */
paused_p = update_window_tree (root_window, force_p);
update_end (f);
/* This flush is a performance bottleneck under X,
and it doesn't seem to be necessary anyway (in general).
It is necessary when resizing the window with the mouse, or
at least the fringes are not redrawn in a timely manner. ++kfs */
if (f->force_flush_display_p)
{
flush_frame (f);
f->force_flush_display_p = 0;
}
}
else
{

View file

@ -185,10 +185,6 @@ struct frame
Clear the frame in clear_garbaged_frames if set. */
unsigned resized_p : 1;
/* Set to non-zero in when we want for force a flush_display in
update_frame, usually after resizing the frame. */
unsigned force_flush_display_p : 1;
/* Set to non-zero if the default face for the frame has been
realized. Reset to zero whenever the default face changes.
Used to see the difference between a font change and face change. */

View file

@ -10767,11 +10767,9 @@ clear_garbaged_frames (void)
if (FRAME_VISIBLE_P (f) && FRAME_GARBAGED_P (f))
{
if (f->resized_p)
{
redraw_frame (f);
f->force_flush_display_p = 1;
}
clear_current_matrices (f);
redraw_frame (f);
else
clear_current_matrices (f);
changed_count++;
f->garbaged = 0;
f->resized_p = 0;