ViewVC Help
View File | Revision Log | Show Annotations | Download File
/cvs/rxvt-unicode/src/screen.C
(Generate patch)

Comparing rxvt-unicode/src/screen.C (file contents):
Revision 1.177 by root, Tue Dec 20 21:49:55 2005 UTC vs.
Revision 1.191 by root, Thu Dec 22 18:36:55 2005 UTC

… …
83#define ROWCOL_IN_ROW_AT_OR_AFTER(X, Y) \ 83#define ROWCOL_IN_ROW_AT_OR_AFTER(X, Y) \
84 ROW_AND_COL_IN_ROW_AT_OR_AFTER ((X).row, (X).col, (Y).row, (Y).col) 84 ROW_AND_COL_IN_ROW_AT_OR_AFTER ((X).row, (X).col, (Y).row, (Y).col)
85#define ROWCOL_IN_ROW_AT_OR_BEFORE(X, Y) \ 85#define ROWCOL_IN_ROW_AT_OR_BEFORE(X, Y) \
86 ROW_AND_COL_IN_ROW_AT_OR_BEFORE ((X).row, (X).col, (Y).row, (Y).col) 86 ROW_AND_COL_IN_ROW_AT_OR_BEFORE ((X).row, (X).col, (Y).row, (Y).col)
87 87
88#define LINENO(n) (((n) + term_start + total_rows) % total_rows)
89#define ROW(n) (save [LINENO (n)])
90
91/* 88/*
92 * CLEAR_ROWS : clear <num> rows starting from row <row> 89 * CLEAR_ROWS : clear <num> rows starting from row <row>
93 * CLEAR_CHARS: clear <num> chars starting from pixel position <x,y> 90 * CLEAR_CHARS: clear <num> chars starting from pixel position <x,y>
94 * ERASE_ROWS : set <num> rows starting from row <row> to the foreground colour 91 * ERASE_ROWS : set <num> rows starting from row <row> to the foreground colour
95 */ 92 */
… …
150} 147}
151 148
152/* ------------------------------------------------------------------------- * 149/* ------------------------------------------------------------------------- *
153 * SCREEN INITIALISATION * 150 * SCREEN INITIALISATION *
154 * ------------------------------------------------------------------------- */ 151 * ------------------------------------------------------------------------- */
152
155void 153void
156rxvt_term::scr_reset () 154rxvt_term::scr_reset ()
157{ 155{
158 unsigned int p, q;
159 int k;
160
161#if ENABLE_OVERLAY 156#if ENABLE_OVERLAY
162 scr_overlay_off (); 157 scr_overlay_off ();
163#endif 158#endif
164 159
165 view_start = 0; 160 view_start = 0;
… …
175 return; 170 return;
176 171
177 // we need at least two lines for wrapping to work correctly 172 // we need at least two lines for wrapping to work correctly
178 if (nrow + saveLines < 2) 173 if (nrow + saveLines < 2)
179 { 174 {
175 //TODO//FIXME
180 saveLines++; 176 saveLines++;
181 prev_nrow--; 177 prev_nrow--;
182 nsaved++; 178 nsaved++;
183 } 179 }
184 180
… …
188 total_rows = nrow + saveLines; 184 total_rows = nrow + saveLines;
189 185
190 screen.tscroll = 0; 186 screen.tscroll = 0;
191 screen.bscroll = nrow - 1; 187 screen.bscroll = nrow - 1;
192 188
193 if (!save) 189 if (!row_buf)
194 { 190 {
195 /* 191 /*
196 * first time called so just malloc everything: don't rely on realloc 192 * first time called so just malloc everything: don't rely on realloc
197 * Note: this is still needed so that all the scrollback lines are NULL 193 * Note: this is still needed so that all the scrollback lines are NULL
198 */ 194 */
… …
200 term_start = 0; 196 term_start = 0;
201 197
202 talloc = new rxvt_salloc (ncol * sizeof (text_t)); 198 talloc = new rxvt_salloc (ncol * sizeof (text_t));
203 ralloc = new rxvt_salloc (ncol * sizeof (rend_t)); 199 ralloc = new rxvt_salloc (ncol * sizeof (rend_t));
204 200
205 save = (line_t *)rxvt_calloc (total_rows, sizeof (line_t));
206 buf = (line_t *)rxvt_calloc (total_rows, sizeof (line_t)); 201 row_buf = (line_t *)rxvt_calloc (total_rows, sizeof (line_t));
202 temp_buf = (line_t *)rxvt_calloc (total_rows, sizeof (line_t));
207 drawn = (line_t *)rxvt_calloc (nrow, sizeof (line_t)); 203 drawn_buf = (line_t *)rxvt_calloc (nrow , sizeof (line_t));
208 swap_save = (line_t *)rxvt_calloc (nrow, sizeof (line_t)); 204 swap_buf = (line_t *)rxvt_calloc (nrow , sizeof (line_t));
209 205
210 for (p = nrow; p--; ) 206 for (int row = nrow; row--; )
211 { 207 {
212 scr_blank_screen_mem (ROW(p), DEFAULT_RSTYLE); 208 scr_blank_screen_mem (ROW (row), DEFAULT_RSTYLE);
213 scr_blank_screen_mem (swap_save[p], DEFAULT_RSTYLE); 209 scr_blank_screen_mem (swap_buf [row], DEFAULT_RSTYLE);
214 scr_blank_screen_mem (drawn[p], DEFAULT_RSTYLE); 210 scr_blank_screen_mem (drawn_buf[row], DEFAULT_RSTYLE);
215 } 211 }
216 212
217 memset (charsets, 'B', sizeof (charsets)); 213 memset (charsets, 'B', sizeof (charsets));
218 rstyle = DEFAULT_RSTYLE; 214 rstyle = DEFAULT_RSTYLE;
219 screen.flags = Screen_DefaultFlags; 215 screen.flags = Screen_DefaultFlags;
… …
243 { 239 {
244 /* 240 /*
245 * add or delete rows as appropriate 241 * add or delete rows as appropriate
246 */ 242 */
247 243
248 rxvt_salloc *old_ta; 244 rxvt_salloc *old_ta = talloc; talloc = new rxvt_salloc (ncol * sizeof (text_t));
249 rxvt_salloc *old_ra; 245 rxvt_salloc *old_ra = ralloc; ralloc = new rxvt_salloc (ncol * sizeof (rend_t));
250 246
247#if 0
251 if (nrow < prev_nrow) 248 if (nrow < prev_nrow)
252 { 249 {
253 /* delete rows */
254 k = min (nsaved, prev_nrow - nrow);
255 // k = max (0, - ( (nrow - 1) - r->screen.cur.row)); // mmc's http://maruska.dyndns.org/wiki/scrolling-bug //make configurable? //D TODO
256 scr_scroll_text (0, (int)prev_nrow - 1, k);
257
258 for (p = nrow; p < prev_nrow; p++) 250 for (int row = nrow; row < prev_nrow; row++)
259 { 251 {
260 lfree (ROW(p));
261 lfree (swap_save[p]); 252 lfree (swap_buf [row]);
262 lfree (drawn[p]); 253 lfree (drawn_buf[row]);
263 } 254 }
255 }
256#endif
264 257
265 /* we have fewer rows so fix up cursor position */ 258 drawn_buf = (line_t *)rxvt_realloc (drawn_buf, nrow * sizeof (line_t));
266 MIN_IT (screen.cur.row, (int32_t)nrow - 1); 259 temp_buf = (line_t *)rxvt_realloc (temp_buf , nrow * sizeof (line_t));
260 swap_buf = (line_t *)rxvt_realloc (swap_buf , nrow * sizeof (line_t));
267 261
268 scr_reset_realloc (); /* realloc _last_ */ 262 for (int row = min (nrow, prev_nrow); row--; )
263 {
264 lresize (drawn_buf[row]);
265 lresize (swap_buf [row]);
266 }
267
268 for (int row = prev_nrow; row < nrow; row++)
269 {
270 swap_buf [row].clear (); scr_blank_screen_mem (swap_buf [row], DEFAULT_RSTYLE);
271 drawn_buf[row].clear (); scr_blank_screen_mem (drawn_buf[row], DEFAULT_RSTYLE);
272 }
273
274 line_t *old_buf = row_buf; row_buf = (line_t *)rxvt_calloc (total_rows, sizeof (line_t));
269 } 275
270 else if (nrow > prev_nrow) 276 int p = MOD (term_start + prev_nrow, prev_total_rows); // previous row
271 { 277 int pend = MOD (term_start - nsaved , prev_total_rows);
272 /* add rows */ 278 int q = total_rows; // rewrapped row
273 scr_reset_realloc (); /* realloc _first_ */
274 279
275 int ocol = ncol; 280 if (nsaved)
276 ncol = prev_ncol; // save b/c scr_blank_screen_mem uses this 281 {
282 // re-wrap lines, this is rather ugly, possibly because I am too dumb
283 // to come up with a lean and mean algorithm.
277 284
278 k = min (nsaved, nrow - prev_nrow); 285 row_col_t ocur = screen.cur;
286 ocur.row = MOD (term_start + ocur.row, prev_total_rows);
279 287
280 for (p = prev_total_rows; p < total_rows; p++) 288 while (p != pend && q > 0)
281 save[p].clear ();
282
283 for (p = prev_total_rows; p < total_rows - k; p++)
284 scr_blank_screen_mem (save[p], DEFAULT_RSTYLE);
285
286 for (p = prev_nrow; p < nrow; p++)
287 { 289 {
288 swap_save[p].clear (); scr_blank_screen_mem (swap_save[p], DEFAULT_RSTYLE); 290 p = MOD (p - 1, prev_total_rows);
289 drawn[p].clear (); scr_blank_screen_mem (drawn[p], DEFAULT_RSTYLE);
290 }
291
292 if (k > 0)
293 {
294 scr_scroll_text (0, (int)nrow - 1, -k);
295 screen.cur.row += k;
296 screen.s_cur.row += k;
297 nsaved -= k;
298 }
299
300#ifdef DEBUG_STRICT 291#ifdef DEBUG_STRICT
301 assert (screen.cur.row < nrow); 292 assert (old_buf [MOD (p, prev_total_rows)].t);
302#else /* drive with your eyes closed */
303 MIN_IT (screen.cur.row, nrow - 1);
304#endif 293#endif
305 ncol = ocol; // save b/c scr_blank_screen_mem uses this
306 }
307 294
308 /* resize columns */
309 if (ncol != prev_ncol)
310 {
311 old_ta = talloc; talloc = new rxvt_salloc (ncol * sizeof (text_t));
312 old_ra = ralloc; ralloc = new rxvt_salloc (ncol * sizeof (rend_t));
313
314 for (p = total_rows; p--; )
315 lresize (save[p]);
316
317 for (p = nrow; p--; )
318 {
319 lresize (drawn[p]);
320 lresize (swap_save[p]);
321 }
322
323 MIN_IT (screen.cur.col, (int16_t)ncol - 1);
324
325 delete old_ta;
326 delete old_ra;
327 }
328
329#if 0
330 // re-wrap lines, this is rather ugly, possibly because I am too dumb
331 // to come up with a lean and mean algorithm.
332 rxvt_salloc *ta = new rxvt_salloc (ncol * sizeof (text_t));
333 rxvt_salloc *ra = new rxvt_salloc (ncol * sizeof (rend_t));
334
335 text_t **tp = (text_t **)rxvt_calloc (total_rows, sizeof (text_t *));
336 rend_t **rp = (rend_t **)rxvt_calloc (total_rows, sizeof (rend_t *));
337 tlen_t *tl = (tlen_t *) rxvt_calloc (total_rows, sizeof (tlen_t));
338
339 for (p = 0; p < prev_total_rows; p++) printf ("P %p %d\n", save[p].t, save[p].l);//D
340
341 p = prev_total_rows;
342 q = total_rows;
343
344 while (p > 0 && q > 0)
345 {
346 --p;
347
348 printf ("pq %d:%d\n", p, q);
349 if (save[p].t)
350 {
351 int llen = save[p].l; 295 int plines = 1;
296 int llen = old_buf [MOD (p, prev_total_rows)].l;
352 297
353 assert (llen >= 0); 298 while (p != pend && old_buf [MOD (p - 1, prev_total_rows)].is_longer ())
354
355 while (p && save[p - 1].l < 0)
356 { 299 {
300 p = MOD (p - 1, prev_total_rows);
301
357 --p; 302 plines++;
358 llen += prev_ncol; 303 llen += prev_ncol;
359 } 304 }
360 305
361 int qlines = llen / ncol + 1; 306 int qlines = max (0, (llen - 1) / ncol) + 1;
307
308 // drop partial lines completely
309 if (q < qlines)
310 break;
311
312 q -= qlines;
313
362 int lofs = 0; 314 int lofs = 0;
315 line_t *qline;
363 316
364 q -= qlines; 317 // re-assemble the full line by destination lines
365
366 int qrow = q;
367
368 printf ("QL %d llen %d\n", qlines, llen);//D
369 for (; qlines--; qrow++) 318 for (int qrow = q; qlines--; qrow++)
370 { 319 {
320 qline = row_buf + qrow;
321 lalloc (*qline);
322 qline->set_is_longer ();
323
371 if (qrow >= 0) 324 int qcol = 0;
325
326 // fill a single destination line
327 while (lofs < llen && qcol < ncol)
372 { 328 {
373 tp [qrow] = (text_t *)ta->alloc ();
374 rp [qrow] = (rend_t *)ra->alloc ();
375 tl [qrow] = LINE_CONT1;
376
377 int qcol = 0;
378
379 for (;;)
380 {
381 int prow = lofs / prev_ncol + p; 329 int prow = lofs / prev_ncol;
382 int pcol = lofs % prev_ncol; 330 int pcol = lofs % prev_ncol;
383 331
332 prow = MOD (p + prow, prev_total_rows);
333
334 // we only adjust the cursor _row_ and put it into
335 // the topmost line of "long line" it was in, as
336 // this seems to upset applications/shells/readline
337 // least.
338 if (prow == ocur.row)
339 screen.cur.row = q - (total_rows - nrow);
340
341 line_t &pline = old_buf [prow];
342
384 int len = min (min (prev_ncol - pcol, ncol - qcol), llen - lofs); 343 int len = min (min (prev_ncol - pcol, ncol - qcol), llen - lofs);
385 344
386 printf ("q %d lofs %d>%d len %d pq %d:%d p %d:%d q :%d\n", q, llen, lofs, len, prev_ncol, ncol, prow, pcol, qcol);
387
388 if (len <= 0)
389 {
390 tl [qrow] = qcol;
391
392 TODO
393 scr_blank_line (tp [qrow] + qcol, rp [qrow] + qcol,
394 ncol - qcol, DEFAULT_RSTYLE);
395
396 break;
397 }
398
399 assert (lofs < 1000); 345 assert (len);
346 assert (pline.t);
400 347
401 memcpy (tp [qrow] + qcol, save[prow].t + pcol, len * sizeof (text_t)); 348 memcpy (qline->t + qcol, pline.t + pcol, len * sizeof (text_t));
402 memcpy (rp [qrow] + qcol, save[prow].r + pcol, len * sizeof (rend_t)); 349 memcpy (qline->r + qcol, pline.r + pcol, len * sizeof (rend_t));
403 350
404 lofs += len; 351 lofs += len;
405 qcol += len; 352 qcol += len;
406
407 if (qcol == ncol)
408 break;
409 }
410 } 353 }
411 else
412 lofs += ncol;
413 } 354 }
355
356 qline->l = llen < ncol ? llen : MOD (llen - 1, ncol) + 1;
357 scr_blank_line (*qline, qline->l, ncol - qline->l, DEFAULT_RSTYLE);
414 } 358 }
359
360 term_start = total_rows - nrow;
361 view_start = 0;
362 nsaved = term_start - q;
363
364 // make sure all terminal lines exist
365 while (nsaved < 0)
366 scr_blank_screen_mem (ROW (-++nsaved), DEFAULT_RSTYLE);
367
368 }
369 else
415 } 370 {
371 // if no scrollback exists (yet), wing, instead of wrap
372
373 for (int row = min (nrow, prev_nrow); row--; )
374 {
375 line_t &pline = old_buf [MOD (term_start + row, prev_total_rows)];
376 line_t &qline = row_buf [row];
416 377
417 free (screen.text); screen.text = tp; 378 qline = pline;
418 free (screen.rend); screen.rend = rp; 379 lresize (qline);
419 free (screen.tlen); screen.tlen = tl; 380 }
420 381
421 for (p = 0; p < total_rows; p++) printf ("P %p %d\n", save[p].t, save[p].l);//D 382 for (int row = prev_nrow; row < nrow; row++)
383 {
384 row_buf [row].clear (); scr_blank_screen_mem (row_buf [row], DEFAULT_RSTYLE);
385 }
386
387 term_start = 0;
388 view_start = 0;
389 }
390
391#ifdef DEBUG_STRICT
392 for (int i = -nsaved; i < nrow; i++)
393 assert (ROW (i).t);
422#endif 394#endif
395
396 free (old_buf);
397 delete old_ta;
398 delete old_ra;
399
400 clamp_it (screen.cur.row, 0, nrow - 1);
401 clamp_it (screen.cur.col, 0, ncol - 1);
423 402
424 if (tabs) 403 if (tabs)
425 free (tabs); 404 free (tabs);
426 } 405 }
427 406
428 prev_nrow = nrow; 407 prev_nrow = nrow;
429 prev_ncol = ncol; 408 prev_ncol = ncol;
430 409
431 tabs = (char *)rxvt_malloc (ncol * sizeof (char)); 410 tabs = (char *)rxvt_malloc (ncol * sizeof (char));
432 411
433 for (p = 0; p < ncol; p++) 412 for (int col = ncol; col--; )
434 tabs[p] = (p % TABSIZE == 0) ? 1 : 0; 413 tabs [col] = col % TABSIZE == 0;
435 414
436 tt_winch (); 415 tt_winch ();
437}
438
439void
440rxvt_term::scr_reset_realloc ()
441{
442 swap_save = (line_t *) rxvt_realloc (swap_save , nrow * sizeof (line_t));
443 drawn = (line_t *) rxvt_realloc (drawn , nrow * sizeof (line_t));
444 buf = (line_t *) rxvt_realloc (buf , nrow * sizeof (line_t));
445 save = (line_t *) rxvt_realloc (save , total_rows * sizeof (line_t));
446} 416}
447 417
448/* ------------------------------------------------------------------------- */ 418/* ------------------------------------------------------------------------- */
449/* 419/*
450 * Free everything. That way malloc debugging can find leakage. 420 * Free everything. That way malloc debugging can find leakage.
… …
453rxvt_term::scr_release () 423rxvt_term::scr_release ()
454{ 424{
455 delete talloc; talloc = 0; 425 delete talloc; talloc = 0;
456 delete ralloc; ralloc = 0; 426 delete ralloc; ralloc = 0;
457 427
458 free (save); 428 free (row_buf);
459 free (swap_save); 429 free (swap_buf);
460 free (drawn); 430 free (drawn_buf);
461 free (buf); 431 free (temp_buf);
462 free (tabs); 432 free (tabs);
433
434 row_buf = 0; // signal that we freed all the arrays
463} 435}
464 436
465/* ------------------------------------------------------------------------- */ 437/* ------------------------------------------------------------------------- */
466/* 438/*
467 * Hard reset 439 * Hard reset
… …
518 set_font_style (); 490 set_font_style ();
519 break; 491 break;
520 } 492 }
521 493
522 /* boundary check in case screen size changed between SAVE and RESTORE */ 494 /* boundary check in case screen size changed between SAVE and RESTORE */
523 MIN_IT (s->cur.row, nrow - 1); 495 min_it (s->cur.row, nrow - 1);
524 MIN_IT (s->cur.col, ncol - 1); 496 min_it (s->cur.col, ncol - 1);
525#ifdef DEBUG_STRICT 497#ifdef DEBUG_STRICT
526 assert (s->cur.row >= 0); 498 assert (s->cur.row >= 0);
527 assert (s->cur.col >= 0); 499 assert (s->cur.col >= 0);
528#else /* drive with your eyes closed */
529 MAX_IT (s->cur.row, 0);
530 MAX_IT (s->cur.col, 0);
531#endif 500#endif
532} 501}
533 502
534/* ------------------------------------------------------------------------- */ 503/* ------------------------------------------------------------------------- */
535/* 504/*
… …
546 if (current_screen == scrn) 515 if (current_screen == scrn)
547 return scrn; 516 return scrn;
548 517
549 selection_check (2); /* check for boundary cross */ 518 selection_check (2); /* check for boundary cross */
550 519
551 SWAP_IT (scrn, current_screen, int); 520 int i = current_screen; current_screen = scrn; scrn = i;
552 521
553 SWAP_IT (screen.cur.row, swap.cur.row, int16_t); 522 ::swap (screen.cur.row, swap.cur.row);
554 SWAP_IT (screen.cur.col, swap.cur.col, int16_t); 523 ::swap (screen.cur.col, swap.cur.col);
555 MAX_IT (screen.cur.row, 0); 524
556 MIN_IT (screen.cur.row, (int32_t)prev_nrow - 1); 525 screen.cur.row = clamp (screen.cur.row, 0, prev_nrow - 1);
557 MAX_IT (screen.cur.col, 0); 526 screen.cur.col = clamp (screen.cur.col, 0, prev_ncol - 1);
558 MIN_IT (screen.cur.col, (int32_t)prev_ncol - 1);
559 527
560#if NSCREENS 528#if NSCREENS
561 if (options & Opt_secondaryScreen) 529 if (options & Opt_secondaryScreen)
562 { 530 {
563 num_scr = 0; 531 num_scr = 0;
564 532
565 for (int i = nrow; i--; ) 533 for (int i = prev_nrow; i--; )
566 SWAP_IT (ROW(i), swap_save[i], line_t); 534 ::swap (ROW(i), swap_buf [i]);
567 535
568 SWAP_IT (screen.charset, swap.charset, int16_t); 536 ::swap (screen.charset, swap.charset);
569 SWAP_IT (screen.flags, swap.flags, int); 537 ::swap (screen.flags, swap.flags);
570 screen.flags |= Screen_VisibleCursor; 538 screen.flags |= Screen_VisibleCursor;
571 swap.flags |= Screen_VisibleCursor; 539 swap.flags |= Screen_VisibleCursor;
572 } 540 }
573 else 541 else
574#endif 542#endif
… …
697 // use a simple and robust scrolling algorithm, this 665 // use a simple and robust scrolling algorithm, this
698 // part of scr_scroll_text is not time-critical. 666 // part of scr_scroll_text is not time-critical.
699 667
700 int rows = row2 - row1 + 1; 668 int rows = row2 - row1 + 1;
701 669
670 min_it (count, rows);
671
702 for (int row = 0; row < rows; row++) 672 for (int row = 0; row < rows; row++)
703 { 673 {
704 buf [row] = ROW(row1 + (row + count + rows) % rows); 674 temp_buf [row] = ROW(row1 + (row + count + rows) % rows);
705 675
706 if (!IN_RANGE_EXC (row + count, 0, rows)) 676 if (!IN_RANGE_EXC (row + count, 0, rows))
707 scr_blank_screen_mem (buf [row], rstyle); 677 scr_blank_screen_mem (temp_buf [row], rstyle);
708 } 678 }
709 679
710 for (int row = 0; row < rows; row++) 680 for (int row = 0; row < rows; row++)
711 ROW(row1 + row) = buf [row]; 681 ROW(row1 + row) = temp_buf [row];
712 } 682 }
713 683
714 return count; 684 return count;
715} 685}
716 686
… …
734 last_col = ncol; 704 last_col = ncol;
735 705
736 if (nlines > 0) 706 if (nlines > 0)
737 { 707 {
738 nlines += screen.cur.row - screen.bscroll; 708 nlines += screen.cur.row - screen.bscroll;
739 if ((nlines > 0) 709 if (nlines > 0
740 && (screen.tscroll == 0) 710 && screen.tscroll == 0
741 && (screen.bscroll == (nrow - 1))) 711 && screen.bscroll == (nrow - 1))
742 { 712 {
743 /* _at least_ this many lines need to be scrolled */ 713 /* _at least_ this many lines need to be scrolled */
744 scr_scroll_text (screen.tscroll, screen.bscroll, nlines); 714 scr_scroll_text (screen.tscroll, screen.bscroll, nlines);
745 screen.cur.row -= nlines; 715 screen.cur.row -= nlines;
746 } 716 }
747 } 717 }
748 718
749#ifdef DEBUG_STRICT 719#ifdef DEBUG_STRICT
750 assert (screen.cur.col < last_col); 720 assert (screen.cur.col < last_col);
751 assert ((screen.cur.row < nrow) 721 assert (screen.cur.row < nrow
752 && (screen.cur.row >= - (int32_t)nsaved)); 722 && screen.cur.row >= -nsaved);
753#else /* drive with your eyes closed */
754 MIN_IT (screen.cur.col, last_col - 1);
755 MIN_IT (screen.cur.row, (int32_t)nrow - 1);
756 MAX_IT (screen.cur.row, - (int32_t)nsaved);
757#endif 723#endif
758 row = screen.cur.row; 724 row = screen.cur.row;
759 725
760 checksel = selection.op && current_screen == selection.screen ? 1 : 0; 726 checksel = selection.op && current_screen == selection.screen ? 1 : 0;
761 727
… …
767 733
768 if (c < 0x20) 734 if (c < 0x20)
769 if (c == C0_LF) 735 if (c == C0_LF)
770 { 736 {
771 if (!line->is_longer ()) /* XXX: think about this */ 737 if (!line->is_longer ()) /* XXX: think about this */
772 MAX_IT (line->l, screen.cur.col); 738 max_it (line->l, screen.cur.col);
773 739
774 screen.flags &= ~Screen_WrapNext; 740 screen.flags &= ~Screen_WrapNext;
775 741
776 if (screen.cur.row == screen.bscroll) 742 if (screen.cur.row == screen.bscroll)
777 scr_scroll_text (screen.tscroll, screen.bscroll, 1); 743 scr_scroll_text (screen.tscroll, screen.bscroll, 1);
… …
782 continue; 748 continue;
783 } 749 }
784 else if (c == C0_CR) 750 else if (c == C0_CR)
785 { 751 {
786 if (!line->is_longer ()) /* XXX: think about this */ 752 if (!line->is_longer ()) /* XXX: think about this */
787 MAX_IT (line->l, screen.cur.col); 753 max_it (line->l, screen.cur.col);
788 754
789 screen.flags &= ~Screen_WrapNext; 755 screen.flags &= ~Screen_WrapNext;
790 screen.cur.col = 0; 756 screen.cur.col = 0;
791 continue; 757 continue;
792 } 758 }
… …
968#endif 934#endif
969 } 935 }
970 } 936 }
971 937
972 if (!line->is_longer ()) /* XXX: think about this */ 938 if (!line->is_longer ()) /* XXX: think about this */
973 MAX_IT (line->l, screen.cur.col); 939 max_it (line->l, screen.cur.col);
974 940
975#ifdef DEBUG_STRICT 941#ifdef DEBUG_STRICT
976 assert (screen.cur.row >= 0); 942 assert (screen.cur.row >= 0);
977#else /* drive with your eyes closed */
978 MAX_IT (screen.cur.row, 0);
979#endif 943#endif
980} 944}
981 945
982/* ------------------------------------------------------------------------- */ 946/* ------------------------------------------------------------------------- */
983/* 947/*
… …
1049 if (ht && options & Opt_pastableTabs) 1013 if (ht && options & Opt_pastableTabs)
1050 { 1014 {
1051 base_rend = SET_FONT (base_rend, 0); 1015 base_rend = SET_FONT (base_rend, 0);
1052 1016
1053 if (!l.is_longer ()) /* XXX: think about this */ 1017 if (!l.is_longer ()) /* XXX: think about this */
1054 MAX_IT (l.l, x); 1018 max_it (l.l, x);
1055 1019
1056 i = screen.cur.col; 1020 i = screen.cur.col;
1057 1021
1058 l.t[i] = '\t'; 1022 l.t[i] = '\t';
1059 l.r[i] = base_rend; 1023 l.r[i] = base_rend;
… …
1138{ 1102{
1139 want_refresh = 1; 1103 want_refresh = 1;
1140 ZERO_SCROLLBACK (); 1104 ZERO_SCROLLBACK ();
1141 1105
1142 screen.cur.col = relative & C_RELATIVE ? screen.cur.col + col : col; 1106 screen.cur.col = relative & C_RELATIVE ? screen.cur.col + col : col;
1143 MAX_IT (screen.cur.col, 0); 1107 clamp_it (screen.cur.col, 0, ncol - 1);
1144 MIN_IT (screen.cur.col, (int32_t)ncol - 1);
1145 1108
1146 screen.flags &= ~Screen_WrapNext; 1109 screen.flags &= ~Screen_WrapNext;
1147 1110
1148 if (relative & R_RELATIVE) 1111 if (relative & R_RELATIVE)
1149 { 1112 {
… …
1167 else 1130 else
1168 { 1131 {
1169 if (screen.flags & Screen_Relative) 1132 if (screen.flags & Screen_Relative)
1170 { /* relative origin mode */ 1133 { /* relative origin mode */
1171 screen.cur.row = row + screen.tscroll; 1134 screen.cur.row = row + screen.tscroll;
1172 MIN_IT (screen.cur.row, screen.bscroll); 1135 min_it (screen.cur.row, screen.bscroll);
1173 } 1136 }
1174 else 1137 else
1175 screen.cur.row = row; 1138 screen.cur.row = row;
1176 } 1139 }
1177 1140
1178 MAX_IT (screen.cur.row, 0); 1141 clamp_it (screen.cur.row, 0, nrow - 1);
1179 MIN_IT (screen.cur.row, (int32_t)nrow - 1);
1180} 1142}
1181 1143
1182/* ------------------------------------------------------------------------- */ 1144/* ------------------------------------------------------------------------- */
1183/* 1145/*
1184 * direction should be UP or DN 1146 * direction should be UP or DN
… …
1199 || (screen.cur.row == screen.tscroll && direction == DN)) 1161 || (screen.cur.row == screen.tscroll && direction == DN))
1200 scr_scroll_text (screen.tscroll, screen.bscroll, dirn); 1162 scr_scroll_text (screen.tscroll, screen.bscroll, dirn);
1201 else 1163 else
1202 screen.cur.row += dirn; 1164 screen.cur.row += dirn;
1203 1165
1204 MAX_IT (screen.cur.row, 0); 1166 clamp_it (screen.cur.row, 0, nrow - 1);
1205 MIN_IT (screen.cur.row, (int32_t)nrow - 1);
1206 selection_check (0); 1167 selection_check (0);
1207} 1168}
1208 1169
1209/* ------------------------------------------------------------------------- */ 1170/* ------------------------------------------------------------------------- */
1210/* 1171/*
… …
1228 switch (mode) 1189 switch (mode)
1229 { 1190 {
1230 case 0: /* erase to end of line */ 1191 case 0: /* erase to end of line */
1231 col = screen.cur.col; 1192 col = screen.cur.col;
1232 num = ncol - col; 1193 num = ncol - col;
1233 MIN_IT (ROW(row).l, (int16_t)col); 1194 min_it (ROW(row).l, col);
1234 if (ROWCOL_IN_ROW_AT_OR_AFTER (selection.beg, screen.cur) 1195 if (ROWCOL_IN_ROW_AT_OR_AFTER (selection.beg, screen.cur)
1235 || ROWCOL_IN_ROW_AT_OR_AFTER (selection.end, screen.cur)) 1196 || ROWCOL_IN_ROW_AT_OR_AFTER (selection.end, screen.cur))
1236 CLEAR_SELECTION (); 1197 CLEAR_SELECTION ();
1237 break; 1198 break;
1238 case 1: /* erase to beginning of line */ 1199 case 1: /* erase to beginning of line */
… …
1305 CLEAR_SELECTION (); 1266 CLEAR_SELECTION ();
1306 1267
1307 if (row >= nrow) /* Out Of Bounds */ 1268 if (row >= nrow) /* Out Of Bounds */
1308 return; 1269 return;
1309 1270
1310 MIN_IT (num, (nrow - row)); 1271 min_it (num, nrow - row);
1311 1272
1312 if (rstyle & (RS_RVid | RS_Uline)) 1273 if (rstyle & (RS_RVid | RS_Uline))
1313 ren = (rend_t) ~RS_None; 1274 ren = (rend_t) ~RS_None;
1314 else if (GET_BASEBG (rstyle) == Color_bg) 1275 else if (GET_BASEBG (rstyle) == Color_bg)
1315 { 1276 {
… …
1328 1289
1329 for (; num--; row++) 1290 for (; num--; row++)
1330 { 1291 {
1331 scr_blank_screen_mem (ROW (row), rstyle); 1292 scr_blank_screen_mem (ROW (row), rstyle);
1332 ROW (row).l = 0; 1293 ROW (row).l = 0;
1333 scr_blank_line (drawn[row], 0, ncol, ren); 1294 scr_blank_line (drawn_buf [row], 0, ncol, ren);
1334 } 1295 }
1335} 1296}
1336 1297
1337#if ENABLE_FRILLS 1298#if ENABLE_FRILLS
1338void 1299void
… …
1423 return; 1384 return;
1424 1385
1425 scr_do_wrap (); 1386 scr_do_wrap ();
1426 1387
1427 selection_check (1); 1388 selection_check (1);
1428 MIN_IT (count, (ncol - screen.cur.col)); 1389 min_it (count, ncol - screen.cur.col);
1429 1390
1430 row = screen.cur.row; 1391 row = screen.cur.row;
1431 1392
1432 line_t *line = &ROW(row); 1393 line_t *line = &ROW(row);
1433 1394
… …
1441 } 1402 }
1442 1403
1443 if (!line->is_longer ()) 1404 if (!line->is_longer ())
1444 { 1405 {
1445 line->l += count; 1406 line->l += count;
1446 MIN_IT (line->l, ncol); 1407 min_it (line->l, ncol);
1447 } 1408 }
1448 1409
1449 if (selection.op && current_screen == selection.screen 1410 if (selection.op && current_screen == selection.screen
1450 && ROWCOL_IN_ROW_AT_OR_AFTER (selection.beg, screen.cur)) 1411 && ROWCOL_IN_ROW_AT_OR_AFTER (selection.beg, screen.cur))
1451 { 1412 {
… …
1512 * XTERM_SEQ: Set region <top> - <bot> inclusive: ESC [ <top> ; <bot> r 1473 * XTERM_SEQ: Set region <top> - <bot> inclusive: ESC [ <top> ; <bot> r
1513 */ 1474 */
1514void 1475void
1515rxvt_term::scr_scroll_region (int top, int bot) 1476rxvt_term::scr_scroll_region (int top, int bot)
1516{ 1477{
1517 MAX_IT (top, 0); 1478 max_it (top, 0);
1518 MIN_IT (bot, (int)nrow - 1); 1479 min_it (bot, nrow - 1);
1519 1480
1520 if (top > bot) 1481 if (top > bot)
1521 return; 1482 return;
1522 1483
1523 screen.tscroll = top; 1484 screen.tscroll = top;
… …
1621 XGCValues gcvalue; 1582 XGCValues gcvalue;
1622 1583
1623 if (rvideo != mode) 1584 if (rvideo != mode)
1624 { 1585 {
1625 rvideo = mode; 1586 rvideo = mode;
1626 SWAP_IT (pix_colors[Color_fg], pix_colors[Color_bg], rxvt_color); 1587 ::swap (pix_colors[Color_fg], pix_colors[Color_bg]);
1627#if XPM_BACKGROUND 1588#if XPM_BACKGROUND
1628 if (bgPixmap.pixmap == None) 1589 if (bgPixmap.pixmap == None)
1629#endif 1590#endif
1630#if TRANSPARENT 1591#if TRANSPARENT
1631 if (! (options & Opt_transparent) || am_transparent == 0) 1592 if (! (options & Opt_transparent) || am_transparent == 0)
… …
1727 bool found = false; 1688 bool found = false;
1728 1689
1729 for (int i = 0; i < nrow; i++) 1690 for (int i = 0; i < nrow; i++)
1730 { 1691 {
1731 int col = 0; 1692 int col = 0;
1732 rend_t *drp = drawn[i].r; 1693 rend_t *drp = drawn_buf[i].r;
1733 1694
1734 for (; col < ncol; col++, drp++) 1695 for (; col < ncol; col++, drp++)
1735 if ((*drp & mask) == value) 1696 if ((*drp & mask) == value)
1736 { 1697 {
1737 found = true; 1698 found = true;
… …
1755rxvt_term::scr_expose (int x, int y, int ewidth, int eheight, bool refresh) 1716rxvt_term::scr_expose (int x, int y, int ewidth, int eheight, bool refresh)
1756{ 1717{
1757 int i; 1718 int i;
1758 row_col_t rc[RC_COUNT]; 1719 row_col_t rc[RC_COUNT];
1759 1720
1760 if (!drawn) /* sanity check */ 1721 if (!drawn_buf) /* sanity check */
1761 return; 1722 return;
1762 1723
1763#ifndef NO_SLOW_LINK_SUPPORT 1724#ifndef NO_SLOW_LINK_SUPPORT
1764 if (refresh_type == FAST_REFRESH && !display->is_local) 1725 if (refresh_type == FAST_REFRESH && !display->is_local)
1765 { 1726 {
1766 y = 0; 1727 y = 0;
1767 eheight = height; 1728 eheight = height;
1768 } 1729 }
1769#endif
1770
1771#ifdef DEBUG_STRICT
1772 x = max (x, 0);
1773 x = min (x, (int)width);
1774 y = max (y, 0);
1775 y = min (y, (int)height);
1776#endif 1730#endif
1777 1731
1778 /* round down */ 1732 /* round down */
1779 rc[PART_BEG].col = Pixel2Col (x); 1733 rc[PART_BEG].col = Pixel2Col (x);
1780 rc[PART_BEG].row = Pixel2Row (y); 1734 rc[PART_BEG].row = Pixel2Row (y);
… …
1783 rc[PART_END].row = Pixel2Row (y + eheight + fheight - 1); 1737 rc[PART_END].row = Pixel2Row (y + eheight + fheight - 1);
1784 1738
1785 /* sanity checks */ 1739 /* sanity checks */
1786 for (i = PART_BEG; i < RC_COUNT; i++) 1740 for (i = PART_BEG; i < RC_COUNT; i++)
1787 { 1741 {
1788 MIN_IT (rc[i].col, ncol - 1); 1742 min_it (rc[i].col, ncol - 1);
1789 MIN_IT (rc[i].row, nrow - 1); 1743 min_it (rc[i].row, nrow - 1);
1790 } 1744 }
1791 1745
1792 for (i = rc[PART_BEG].row; i <= rc[PART_END].row; i++) 1746 for (i = rc[PART_BEG].row; i <= rc[PART_END].row; i++)
1793 fill_text (&drawn[i].t[rc[PART_BEG].col], 0, rc[PART_END].col - rc[PART_BEG].col + 1); 1747 fill_text (&drawn_buf[i].t[rc[PART_BEG].col], 0, rc[PART_END].col - rc[PART_BEG].col + 1);
1794 1748
1795 if (refresh) 1749 if (refresh)
1796 scr_refresh (SLOW_REFRESH); 1750 scr_refresh (SLOW_REFRESH);
1797} 1751}
1798 1752
… …
1961{ 1915{
1962 unsigned char must_clear, /* use draw_string not draw_image_string */ 1916 unsigned char must_clear, /* use draw_string not draw_image_string */
1963 showcursor; /* show the cursor */ 1917 showcursor; /* show the cursor */
1964 int16_t col, row, /* column/row we're processing */ 1918 int16_t col, row, /* column/row we're processing */
1965 ocrow; /* old cursor row */ 1919 ocrow; /* old cursor row */
1966 int i, /* tmp */ 1920 int i; /* tmp */
1967 row_offset; /* basic offset in screen structure */
1968#ifndef NO_CURSORCOLOR 1921#ifndef NO_CURSORCOLOR
1969 rend_t cc1; /* store colours at cursor position (s) */ 1922 rend_t cc1; /* store colours at cursor position (s) */
1970#endif 1923#endif
1971 rend_t *crp; // cursor rendition pointer 1924 rend_t *crp; // cursor rendition pointer
1972 1925
… …
1978 /* 1931 /*
1979 * A: set up vars 1932 * A: set up vars
1980 */ 1933 */
1981 must_clear = 0; 1934 must_clear = 0;
1982 refresh_count = 0; 1935 refresh_count = 0;
1983
1984 row_offset = term_start - view_start;
1985 1936
1986#if XPM_BACKGROUND 1937#if XPM_BACKGROUND
1987 must_clear |= bgPixmap.pixmap != None; 1938 must_clear |= bgPixmap.pixmap != None;
1988#endif 1939#endif
1989#if TRANSPARENT 1940#if TRANSPARENT
… …
2059 if (screen.cur.row + view_start != ocrow 2010 if (screen.cur.row + view_start != ocrow
2060 || screen.cur.col != oldcursor.col) 2011 || screen.cur.col != oldcursor.col)
2061 { 2012 {
2062 if (ocrow < nrow 2013 if (ocrow < nrow
2063 && oldcursor.col < ncol) 2014 && oldcursor.col < ncol)
2064 drawn[ocrow].r[oldcursor.col] ^= (RS_RVid | RS_Uline); 2015 drawn_buf[ocrow].r[oldcursor.col] ^= (RS_RVid | RS_Uline);
2065 2016
2066 if (focus || !showcursor) 2017 if (focus || !showcursor)
2067 oldcursor.row = -1; 2018 oldcursor.row = -1;
2068 else 2019 else
2069 setoldcursor = 1; 2020 setoldcursor = 1;
… …
2107 row = i > 0 ? 0 : j - 1; 2058 row = i > 0 ? 0 : j - 1;
2108 for (; j-- >= 0; row += (i > 0 ? 1 : -1)) 2059 for (; j-- >= 0; row += (i > 0 ? 1 : -1))
2109 { 2060 {
2110 if (row + i >= 0 && row + i < nrow && row + i != ocrow) 2061 if (row + i >= 0 && row + i < nrow && row + i != ocrow)
2111 { 2062 {
2112 line_t s = save[(row + row_offset) % total_rows]; 2063 line_t s = ROW(row - view_start);
2113 line_t d = drawn[row]; 2064 line_t d = drawn_buf[row];
2114 line_t d2 = drawn[row + i]; 2065 line_t d2 = drawn_buf[row + i];
2115 2066
2116 for (nits = 0, col = ncol; col--; ) 2067 for (nits = 0, col = ncol; col--; )
2117 if (s.t[col] != d2.t[col] || s.r[col] != d2.r[col]) 2068 if (s.t[col] != d2.t[col] || s.r[col] != d2.r[col])
2118 nits--; 2069 nits--;
2119 else if (s.t[col] != d.t[col] || s.r[col] != d.r[col]) 2070 else if (s.t[col] != d.t[col] || s.r[col] != d.r[col])
… …
2137 2088
2138 if (len != -1) 2089 if (len != -1)
2139 { 2090 {
2140 /* also comes here at end if needed because of >= above */ 2091 /* also comes here at end if needed because of >= above */
2141 if (wlen < len) 2092 if (wlen < len)
2142 SWAP_IT (wlen, len, int); 2093 ::swap (wlen, len);
2143 2094
2144 XCopyArea (display->display, vt, vt, 2095 XCopyArea (display->display, vt, vt,
2145 gc, 0, Row2Pixel (len + i), 2096 gc, 0, Row2Pixel (len + i),
2146 (unsigned int)TermWin_TotalWidth (), 2097 (unsigned int)TermWin_TotalWidth (),
2147 (unsigned int)Height2Pixel (wlen - len + 1), 2098 (unsigned int)Height2Pixel (wlen - len + 1),
… …
2155 /* 2106 /*
2156 * E: main pass across every character 2107 * E: main pass across every character
2157 */ 2108 */
2158 for (row = 0; row < nrow; row++) 2109 for (row = 0; row < nrow; row++)
2159 { 2110 {
2160 text_t *stp = save[(row + row_offset) % total_rows].t; 2111 text_t *stp = ROW(row - view_start).t;
2161 rend_t *srp = save[(row + row_offset) % total_rows].r; 2112 rend_t *srp = ROW(row - view_start).r;
2162 text_t *dtp = drawn[row].t; 2113 text_t *dtp = drawn_buf[row].t;
2163 rend_t *drp = drawn[row].r; 2114 rend_t *drp = drawn_buf[row].r;
2164 2115
2165 /* 2116 /*
2166 * E2: OK, now the real pass 2117 * E2: OK, now the real pass
2167 */ 2118 */
2168 int ypixel = (int)Row2Pixel (row); 2119 int ypixel = (int)Row2Pixel (row);
… …
2279 fore = Color_UL; 2230 fore = Color_UL;
2280#endif 2231#endif
2281 2232
2282 if (invert) 2233 if (invert)
2283 { 2234 {
2284 SWAP_IT (fore, back, int); 2235 ::swap (fore, back);
2285 2236
2286#ifndef NO_BOLD_UNDERLINE_REVERSE 2237#ifndef NO_BOLD_UNDERLINE_REVERSE
2287 if (ISSET_PIXCOLOR (Color_RV)) 2238 if (ISSET_PIXCOLOR (Color_RV))
2288 back = Color_RV; 2239 back = Color_RV;
2289 2240
… …
2381 2332
2382 while (col && ROW(screen.cur.row).t[col] == NOCHAR) 2333 while (col && ROW(screen.cur.row).t[col] == NOCHAR)
2383 col--; 2334 col--;
2384 2335
2385 while (col + cursorwidth < ncol 2336 while (col + cursorwidth < ncol
2386 && drawn[oldcursor.row].t[col + cursorwidth] == NOCHAR) 2337 && drawn_buf[oldcursor.row].t[col + cursorwidth] == NOCHAR)
2387 cursorwidth++; 2338 cursorwidth++;
2388 2339
2389#ifndef NO_CURSORCOLOR 2340#ifndef NO_CURSORCOLOR
2390 if (ISSET_PIXCOLOR (Color_cursor)) 2341 if (ISSET_PIXCOLOR (Color_cursor))
2391 XSetForeground (display->display, gc, pix_colors[Color_cursor]); 2342 XSetForeground (display->display, gc, pix_colors[Color_cursor]);
… …
2423 2374
2424void 2375void
2425rxvt_term::scr_remap_chars () 2376rxvt_term::scr_remap_chars ()
2426{ 2377{
2427 for (int i = total_rows; i--; ) 2378 for (int i = total_rows; i--; )
2428 scr_remap_chars (save[i]); 2379 scr_remap_chars (row_buf [i]);
2429 2380
2430 for (int i = nrow; i--; ) 2381 for (int i = nrow; i--; )
2431 { 2382 {
2432 scr_remap_chars (drawn[i]); 2383 scr_remap_chars (drawn_buf [i]);
2433 scr_remap_chars (swap_save[i]); 2384 scr_remap_chars (swap_buf [i]);
2434 } 2385 }
2435} 2386}
2436 2387
2437void 2388void
2438rxvt_term::scr_recolour () 2389rxvt_term::scr_recolour ()
… …
2535 char r1[] = "\n"; 2486 char r1[] = "\n";
2536 2487
2537 for (row = saveLines - nsaved; 2488 for (row = saveLines - nsaved;
2538 row < saveLines + nrow - 1; row++) 2489 row < saveLines + nrow - 1; row++)
2539 { 2490 {
2540 width = save[row].l >= 0 ? save[row].l 2491 width = row_buf[row].l >= 0 ? row_buf[row].l
2541 : ncol; 2492 : ncol;
2542 for (towrite = width; towrite; towrite -= wrote) 2493 for (towrite = width; towrite; towrite -= wrote)
2543 { 2494 {
2544 wrote = write (fd, & (save[row].t[width - towrite]), 2495 wrote = write (fd, & (row_buf[row].t[width - towrite]),
2545 towrite); 2496 towrite);
2546 if (wrote < 0) 2497 if (wrote < 0)
2547 return; /* XXX: death, no report */ 2498 return; /* XXX: death, no report */
2548 } 2499 }
2549 if (save[row].l >= 0) 2500 if (row_buf[row].l >= 0)
2550 if (write (fd, r1, 1) <= 0) 2501 if (write (fd, r1, 1) <= 0)
2551 return; /* XXX: death, no report */ 2502 return; /* XXX: death, no report */
2552 } 2503 }
2553} 2504}
2554#endif 2505#endif
… …
3031 want_refresh = 1; 2982 want_refresh = 1;
3032 2983
3033 selection.mark.row = row - view_start; 2984 selection.mark.row = row - view_start;
3034 selection.mark.col = col; 2985 selection.mark.col = col;
3035 2986
3036 selection.mark.row = min (max (selection.mark.row, -nsaved), nrow - 1); 2987 selection.mark.row = clamp (selection.mark.row, -nsaved, nrow - 1);
3037 selection.mark.col = min (max (selection.mark.col, 0), ncol - 1); 2988 selection.mark.col = clamp (selection.mark.col, 0, ncol - 1);
3038 2989
3039 while (selection.mark.col > 0 2990 while (selection.mark.col > 0
3040 && ROW(selection.mark.row).t[selection.mark.col] == NOCHAR) 2991 && ROW(selection.mark.row).t[selection.mark.col] == NOCHAR)
3041 --selection.mark.col; 2992 --selection.mark.col;
3042 2993
… …
3147 * flag == 2 ==> button 3 motion 3098 * flag == 2 ==> button 3 motion
3148 */ 3099 */
3149void 3100void
3150rxvt_term::selection_extend (int x, int y, int flag) 3101rxvt_term::selection_extend (int x, int y, int flag)
3151{ 3102{
3152 int col = min (max (Pixel2Col (x), 0), nrow - 1); 3103 int col = clamp (Pixel2Col (x), 0, ncol);
3153 int row = min (max (Pixel2Row (y), 0), ncol); 3104 int row = clamp (Pixel2Row (y), 0, nrow - 1);
3154 3105
3155 /* 3106 /*
3156 * If we're selecting characters (single click) then we must check first 3107 * If we're selecting characters (single click) then we must check first
3157 * if we are at the same place as the original mark. If we are then 3108 * if we are at the same place as the original mark. If we are then
3158 * select nothing. Otherwise, if we're to the right of the mark, you have to 3109 * select nothing. Otherwise, if we're to the right of the mark, you have to
… …
3371 } 3322 }
3372 } 3323 }
3373 3324
3374#if ENABLE_FRILLS 3325#if ENABLE_FRILLS
3375 if (selection.rect && selection.beg.col > selection.end.col) 3326 if (selection.rect && selection.beg.col > selection.end.col)
3376 SWAP_IT (selection.beg.col, selection.end.col, int); 3327 ::swap (selection.beg.col, selection.end.col);
3377#endif 3328#endif
3378} 3329}
3379 3330
3380#if ENABLE_FRILLS 3331#if ENABLE_FRILLS
3381void 3332void
… …
3579 */ 3530 */
3580void 3531void
3581rxvt_term::pixel_position (int *x, int *y) 3532rxvt_term::pixel_position (int *x, int *y)
3582{ 3533{
3583 *x = Pixel2Col (*x); 3534 *x = Pixel2Col (*x);
3584 /* MAX_IT (*x, 0); MIN_IT (*x, (int)ncol - 1); */ 3535 /* max_it (*x, 0); min_it (*x, (int)ncol - 1); */
3585 *y = Pixel2Row (*y); 3536 *y = Pixel2Row (*y);
3586 /* MAX_IT (*y, 0); MIN_IT (*y, (int)nrow - 1); */ 3537 /* max_it (*y, 0); min_it (*y, (int)nrow - 1); */
3587} 3538}
3588 3539
3589/* ------------------------------------------------------------------------- */ 3540/* ------------------------------------------------------------------------- */
3590#ifdef USE_XIM 3541#ifdef USE_XIM
3591void 3542void
… …
3613 3564
3614 if (x < 0) x = ncol - w; 3565 if (x < 0) x = ncol - w;
3615 if (y < 0) y = nrow - h; 3566 if (y < 0) y = nrow - h;
3616 3567
3617 // make space for border 3568 // make space for border
3618 w += 2; MIN_IT (w, ncol); 3569 w += 2; min_it (w, ncol);
3619 h += 2; MIN_IT (h, nrow); 3570 h += 2; min_it (h, nrow);
3620 3571
3621 x -= 1; MAX_IT (x, 0); 3572 x -= 1; clamp_it (x, 0, ncol - w);
3622 y -= 1; MAX_IT (y, 0); 3573 y -= 1; clamp_it (y, 0, nrow - h);
3623
3624 MIN_IT (x, ncol - w);
3625 MIN_IT (y, nrow - h);
3626 3574
3627 ov_x = x; ov_y = y; 3575 ov_x = x; ov_y = y;
3628 ov_w = w; ov_h = h; 3576 ov_w = w; ov_h = h;
3629 3577
3630 ov_text = new text_t *[h]; 3578 ov_text = new text_t *[h];

Diff Legend

– Removed lines
+ Added lines
< Changed lines
> Changed lines