diff options
| author | Karl Heuer | 1994-10-04 15:56:43 +0000 |
|---|---|---|
| committer | Karl Heuer | 1994-10-04 15:56:43 +0000 |
| commit | 94d92e9c1c721fb03c81563a46e47426aef539ef (patch) | |
| tree | 1db1e69ba1d97ed11ea010d90373219505bb85fb /src | |
| parent | f4e93c40596ab5bef01293848eb1d756e816511d (diff) | |
| download | emacs-94d92e9c1c721fb03c81563a46e47426aef539ef.tar.gz emacs-94d92e9c1c721fb03c81563a46e47426aef539ef.zip | |
(Fcurrent_column, Findent_to, Fcurrent_indentation, Fmove_to_column,
compute_motion, Fcompute_motion, vmotion): Don't use XFASTINT as an lvalue.
Diffstat (limited to 'src')
| -rw-r--r-- | src/indent.c | 32 |
1 files changed, 17 insertions, 15 deletions
diff --git a/src/indent.c b/src/indent.c index e9b9659d536..4dacd1eddff 100644 --- a/src/indent.c +++ b/src/indent.c | |||
| @@ -77,7 +77,7 @@ however, ^M is treated as end of line when `selective-display' is t.") | |||
| 77 | () | 77 | () |
| 78 | { | 78 | { |
| 79 | Lisp_Object temp; | 79 | Lisp_Object temp; |
| 80 | XFASTINT (temp) = current_column (); | 80 | XSETFASTINT (temp, current_column ()); |
| 81 | return temp; | 81 | return temp; |
| 82 | } | 82 | } |
| 83 | 83 | ||
| @@ -187,7 +187,7 @@ even if that goes past COLUMN; by default, MIN is zero.") | |||
| 187 | 187 | ||
| 188 | CHECK_NUMBER (col, 0); | 188 | CHECK_NUMBER (col, 0); |
| 189 | if (NILP (minimum)) | 189 | if (NILP (minimum)) |
| 190 | XFASTINT (minimum) = 0; | 190 | XSETFASTINT (minimum, 0); |
| 191 | CHECK_NUMBER (minimum, 1); | 191 | CHECK_NUMBER (minimum, 1); |
| 192 | 192 | ||
| 193 | fromcol = current_column (); | 193 | fromcol = current_column (); |
| @@ -202,7 +202,7 @@ even if that goes past COLUMN; by default, MIN is zero.") | |||
| 202 | if (indent_tabs_mode) | 202 | if (indent_tabs_mode) |
| 203 | { | 203 | { |
| 204 | Lisp_Object n; | 204 | Lisp_Object n; |
| 205 | XFASTINT (n) = mincol / tab_width - fromcol / tab_width; | 205 | XSETFASTINT (n, mincol / tab_width - fromcol / tab_width); |
| 206 | if (XFASTINT (n) != 0) | 206 | if (XFASTINT (n) != 0) |
| 207 | { | 207 | { |
| 208 | Finsert_char (make_number ('\t'), n, Qt); | 208 | Finsert_char (make_number ('\t'), n, Qt); |
| @@ -211,7 +211,7 @@ even if that goes past COLUMN; by default, MIN is zero.") | |||
| 211 | } | 211 | } |
| 212 | } | 212 | } |
| 213 | 213 | ||
| 214 | XFASTINT (col) = mincol - fromcol; | 214 | XSETFASTINT (col, mincol - fromcol); |
| 215 | Finsert_char (make_number (' '), col, Qt); | 215 | Finsert_char (make_number (' '), col, Qt); |
| 216 | 216 | ||
| 217 | last_known_column = mincol; | 217 | last_known_column = mincol; |
| @@ -231,7 +231,7 @@ following any initial whitespace.") | |||
| 231 | { | 231 | { |
| 232 | Lisp_Object val; | 232 | Lisp_Object val; |
| 233 | 233 | ||
| 234 | XFASTINT (val) = position_indentation (find_next_newline (point, -1)); | 234 | XSETFASTINT (val, position_indentation (find_next_newline (point, -1))); |
| 235 | return val; | 235 | return val; |
| 236 | } | 236 | } |
| 237 | 237 | ||
| @@ -376,7 +376,7 @@ and if COLUMN is in the middle of a tab character, change it to spaces.") | |||
| 376 | last_known_column_point = point; | 376 | last_known_column_point = point; |
| 377 | last_known_column_modified = MODIFF; | 377 | last_known_column_modified = MODIFF; |
| 378 | 378 | ||
| 379 | XFASTINT (val) = col; | 379 | XSETFASTINT (val, col); |
| 380 | return val; | 380 | return val; |
| 381 | } | 381 | } |
| 382 | 382 | ||
| @@ -478,7 +478,7 @@ compute_motion (from, fromvpos, fromhpos, to, tovpos, tohpos, width, hscroll, ta | |||
| 478 | the next property change */ | 478 | the next property change */ |
| 479 | while (pos == next_invisible && pos < to) | 479 | while (pos == next_invisible && pos < to) |
| 480 | { | 480 | { |
| 481 | XFASTINT (position) = pos; | 481 | XSETFASTINT (position, pos); |
| 482 | prop = Fget_char_property (position, | 482 | prop = Fget_char_property (position, |
| 483 | Qinvisible, | 483 | Qinvisible, |
| 484 | Fcurrent_buffer ()); | 484 | Fcurrent_buffer ()); |
| @@ -490,7 +490,7 @@ compute_motion (from, fromvpos, fromhpos, to, tovpos, tohpos, width, hscroll, ta | |||
| 490 | performance; nothing should go wrong if it is too small. */ | 490 | performance; nothing should go wrong if it is too small. */ |
| 491 | limit = Fnext_overlay_change (position); | 491 | limit = Fnext_overlay_change (position); |
| 492 | if (XFASTINT (limit) > pos + 100) | 492 | if (XFASTINT (limit) > pos + 100) |
| 493 | XFASTINT (limit) = pos + 100; | 493 | XSETFASTINT (limit, pos + 100); |
| 494 | end = Fnext_single_property_change (position, Qinvisible, | 494 | end = Fnext_single_property_change (position, Qinvisible, |
| 495 | Fcurrent_buffer (), limit); | 495 | Fcurrent_buffer (), limit); |
| 496 | if (INTEGERP (end)) | 496 | if (INTEGERP (end)) |
| @@ -697,7 +697,7 @@ DEFUN ("compute-motion", Fcompute_motion, Scompute_motion, 7, 7, 0, | |||
| 697 | XINT (width), hscroll, tab_offset, | 697 | XINT (width), hscroll, tab_offset, |
| 698 | XWINDOW (window)); | 698 | XWINDOW (window)); |
| 699 | 699 | ||
| 700 | XFASTINT (bufpos) = pos->bufpos; | 700 | XSETFASTINT (bufpos, pos->bufpos); |
| 701 | XSETINT (hpos, pos->hpos); | 701 | XSETINT (hpos, pos->hpos); |
| 702 | XSETINT (vpos, pos->vpos); | 702 | XSETINT (vpos, pos->vpos); |
| 703 | XSETINT (prevhpos, pos->prevhpos); | 703 | XSETINT (prevhpos, pos->prevhpos); |
| @@ -767,7 +767,7 @@ vmotion (from, vtarget, width, hscroll, window) | |||
| 767 | to determine hpos of starting point */ | 767 | to determine hpos of starting point */ |
| 768 | if (from > BEGV && FETCH_CHAR (from - 1) != '\n') | 768 | if (from > BEGV && FETCH_CHAR (from - 1) != '\n') |
| 769 | { | 769 | { |
| 770 | XFASTINT (prevline) = find_next_newline_no_quit (from, -1); | 770 | XSETFASTINT (prevline, find_next_newline_no_quit (from, -1)); |
| 771 | while (XFASTINT (prevline) > BEGV | 771 | while (XFASTINT (prevline) > BEGV |
| 772 | && ((selective > 0 | 772 | && ((selective > 0 |
| 773 | && indented_beyond_p (XFASTINT (prevline), selective)) | 773 | && indented_beyond_p (XFASTINT (prevline), selective)) |
| @@ -778,8 +778,9 @@ vmotion (from, vtarget, width, hscroll, window) | |||
| 778 | window)) | 778 | window)) |
| 779 | #endif | 779 | #endif |
| 780 | )) | 780 | )) |
| 781 | XFASTINT (prevline) | 781 | XSETFASTINT (prevline, |
| 782 | = find_next_newline_no_quit (XFASTINT (prevline) - 1, -1); | 782 | find_next_newline_no_quit (XFASTINT (prevline) - 1, |
| 783 | -1)); | ||
| 783 | pos = *compute_motion (XFASTINT (prevline), 0, | 784 | pos = *compute_motion (XFASTINT (prevline), 0, |
| 784 | lmargin + (XFASTINT (prevline) == 1 | 785 | lmargin + (XFASTINT (prevline) == 1 |
| 785 | ? start_hpos : 0), | 786 | ? start_hpos : 0), |
| @@ -804,11 +805,12 @@ vmotion (from, vtarget, width, hscroll, window) | |||
| 804 | 805 | ||
| 805 | while ((vpos > vtarget || first) && from > BEGV) | 806 | while ((vpos > vtarget || first) && from > BEGV) |
| 806 | { | 807 | { |
| 807 | XFASTINT (prevline) = from; | 808 | XSETFASTINT (prevline, from); |
| 808 | while (1) | 809 | while (1) |
| 809 | { | 810 | { |
| 810 | XFASTINT (prevline) | 811 | XSETFASTINT (prevline, |
| 811 | = find_next_newline_no_quit (XFASTINT (prevline) - 1, -1); | 812 | find_next_newline_no_quit (XFASTINT (prevline) - 1, |
| 813 | -1)); | ||
| 812 | if (XFASTINT (prevline) == BEGV | 814 | if (XFASTINT (prevline) == BEGV |
| 813 | || ((selective <= 0 | 815 | || ((selective <= 0 |
| 814 | || ! indented_beyond_p (XFASTINT (prevline), selective)) | 816 | || ! indented_beyond_p (XFASTINT (prevline), selective)) |