diff options
| author | Karl Heuer | 1995-03-24 00:35:46 +0000 |
|---|---|---|
| committer | Karl Heuer | 1995-03-24 00:35:46 +0000 |
| commit | 4d641a1562a8691a3c52da680d997063d2fbf655 (patch) | |
| tree | 3948db66ed81e45d675d95644b42136572384584 /src | |
| parent | fa7a4531e740348160ba41cc8655ae40fad206d9 (diff) | |
| download | emacs-4d641a1562a8691a3c52da680d997063d2fbf655.tar.gz emacs-4d641a1562a8691a3c52da680d997063d2fbf655.zip | |
(redisplay): Make sure pause is set before used.
Delete unused vars.
(redisplay_window): Make sure update_mode_line is set before used.
If startp might not have been set, use w->start directly.
(try_window_id): In the easy case, find cursor pos before returning.
Diffstat (limited to 'src')
| -rw-r--r-- | src/xdisp.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/xdisp.c b/src/xdisp.c index 04955ad1fa9..7c6c2d8b54d 100644 --- a/src/xdisp.c +++ b/src/xdisp.c | |||
| @@ -745,7 +745,6 @@ redisplay () | |||
| 745 | int all_windows; | 745 | int all_windows; |
| 746 | register int tlbufpos, tlendpos; | 746 | register int tlbufpos, tlendpos; |
| 747 | struct position pos; | 747 | struct position pos; |
| 748 | extern int input_pending; | ||
| 749 | 748 | ||
| 750 | if (noninteractive) | 749 | if (noninteractive) |
| 751 | return; | 750 | return; |
| @@ -864,7 +863,6 @@ redisplay () | |||
| 864 | int left = XFASTINT (w->left); | 863 | int left = XFASTINT (w->left); |
| 865 | int *charstart_next_line | 864 | int *charstart_next_line |
| 866 | = FRAME_CURRENT_GLYPHS (XFRAME (WINDOW_FRAME (w)))->charstarts[this_line_vpos + 1]; | 865 | = FRAME_CURRENT_GLYPHS (XFRAME (WINDOW_FRAME (w)))->charstarts[this_line_vpos + 1]; |
| 867 | int i; | ||
| 868 | int adjust; | 866 | int adjust; |
| 869 | 867 | ||
| 870 | if (Z - tlendpos == ZV) | 868 | if (Z - tlendpos == ZV) |
| @@ -1013,6 +1011,8 @@ update: | |||
| 1013 | { | 1011 | { |
| 1014 | if (FRAME_VISIBLE_P (selected_frame)) | 1012 | if (FRAME_VISIBLE_P (selected_frame)) |
| 1015 | pause = update_frame (selected_frame, 0, 0); | 1013 | pause = update_frame (selected_frame, 0, 0); |
| 1014 | else | ||
| 1015 | pause = 0; | ||
| 1016 | 1016 | ||
| 1017 | /* We may have called echo_area_display at the top of this | 1017 | /* We may have called echo_area_display at the top of this |
| 1018 | function. If the echo area is on another frame, that may | 1018 | function. If the echo area is on another frame, that may |
| @@ -1268,6 +1268,7 @@ redisplay_window (window, just_this_one) | |||
| 1268 | abort (); | 1268 | abort (); |
| 1269 | 1269 | ||
| 1270 | height = window_internal_height (w); | 1270 | height = window_internal_height (w); |
| 1271 | update_mode_line = (!NILP (w->update_mode_line) || update_mode_lines); | ||
| 1271 | 1272 | ||
| 1272 | if (MINI_WINDOW_P (w)) | 1273 | if (MINI_WINDOW_P (w)) |
| 1273 | { | 1274 | { |
| @@ -1294,8 +1295,6 @@ redisplay_window (window, just_this_one) | |||
| 1294 | } | 1295 | } |
| 1295 | } | 1296 | } |
| 1296 | 1297 | ||
| 1297 | update_mode_line = (!NILP (w->update_mode_line) || update_mode_lines); | ||
| 1298 | |||
| 1299 | /* Otherwise set up data on this window; select its buffer and point value */ | 1298 | /* Otherwise set up data on this window; select its buffer and point value */ |
| 1300 | 1299 | ||
| 1301 | if (update_mode_line) | 1300 | if (update_mode_line) |
| @@ -1617,7 +1616,7 @@ done: | |||
| 1617 | || (w == XWINDOW (minibuf_window) && ! echo_area_glyphs)) | 1616 | || (w == XWINDOW (minibuf_window) && ! echo_area_glyphs)) |
| 1618 | { | 1617 | { |
| 1619 | whole = ZV - BEGV; | 1618 | whole = ZV - BEGV; |
| 1620 | start = startp - BEGV; | 1619 | start = marker_position (w->start) - BEGV; |
| 1621 | /* I don't think this is guaranteed to be right. For the | 1620 | /* I don't think this is guaranteed to be right. For the |
| 1622 | moment, we'll pretend it is. */ | 1621 | moment, we'll pretend it is. */ |
| 1623 | end = (Z - XINT (w->window_end_pos)) - BEGV; | 1622 | end = (Z - XINT (w->window_end_pos)) - BEGV; |
| @@ -1771,7 +1770,7 @@ try_window_id (window) | |||
| 1771 | width, hscroll, pos_tab_offset (w, start), w); | 1770 | width, hscroll, pos_tab_offset (w, start), w); |
| 1772 | XSETFASTINT (w->window_end_vpos, height); | 1771 | XSETFASTINT (w->window_end_vpos, height); |
| 1773 | XSETFASTINT (w->window_end_pos, Z - bp.bufpos); | 1772 | XSETFASTINT (w->window_end_pos, Z - bp.bufpos); |
| 1774 | return 1; | 1773 | goto findpoint; |
| 1775 | } | 1774 | } |
| 1776 | return 0; | 1775 | return 0; |
| 1777 | } | 1776 | } |
| @@ -2078,6 +2077,7 @@ try_window_id (window) | |||
| 2078 | /* If point was not in a line that was displayed, find it */ | 2077 | /* If point was not in a line that was displayed, find it */ |
| 2079 | if (cursor_vpos < 0) | 2078 | if (cursor_vpos < 0) |
| 2080 | { | 2079 | { |
| 2080 | findpoint: | ||
| 2081 | val = *compute_motion (start, 0, lmargin, PT, 10000, 10000, | 2081 | val = *compute_motion (start, 0, lmargin, PT, 10000, 10000, |
| 2082 | width, hscroll, pos_tab_offset (w, start), w); | 2082 | width, hscroll, pos_tab_offset (w, start), w); |
| 2083 | /* Admit failure if point is off frame now */ | 2083 | /* Admit failure if point is off frame now */ |