Changes in uspace/app/edit/edit.c [743e17b:3052ff4] in mainline

File:

• uspace/app/edit/edit.c (modified) (21 diffs)

uspace/app/edit/edit.c

@@ -72 +72 @@
         /** Current position of the caret */
         tag_t caret_pos;
-
-        /** 
-         * Ideal column where the caret should try to get. This is used
-         * for maintaining the same column during vertical movement.
-         */
-        int ideal_column;
 } pane_t;
 
@@ -94 +88 @@
 static pane_t pane;
 
-static int scr_rows, scr_columns;
+static ipcarg_t scr_rows, scr_columns;
 
 #define ROW_BUF_SIZE 4096
@@ -136 +130 @@
 
         pane.rows = scr_rows - 1;
-        pane.columns = scr_columns;
         pane.sh_row = 1;
-        pane.sh_column = 1;
 
         /* Start with an empty sheet. */
@@ -147 +139 @@
         sheet_get_cell_pt(&doc.sh, &coord, dir_before, &pt);
         sheet_place_tag(&doc.sh, &pt, &pane.caret_pos);
-        pane.ideal_column = coord.column;
 
         if (argc == 2) {
@@ -215 +206 @@
         case KC_ENTER:
                 insert_char('\n');
+                pane.rflags |= REDRAW_TEXT;
                 caret_update();
                 break;
@@ -243 +235 @@
         case KC_BACKSPACE:
                 delete_char_before();
+                pane.rflags |= REDRAW_TEXT;
                 caret_update();
                 break;
         case KC_DELETE:
                 delete_char_after();
+                pane.rflags |= REDRAW_TEXT;
                 caret_update();
                 break;
@@ -252 +246 @@
                 if (ev->c >= 32 || ev->c == '\t') {
                         insert_char(ev->c);
+                        pane.rflags |= REDRAW_ROW;
                         caret_update();
                 }
@@ -367 +362 @@
 {
         int sh_rows, rows;
-        int i, j;
+        int i;
+        unsigned j;
 
         sheet_get_num_rows(&doc.sh, &sh_rows);
@@ -407 +403 @@
 static void pane_row_range_display(int r0, int r1)
 {
+        int width;
         int i, j, fill;
         spt_t rb, re, dep;
@@ -419 +416 @@
         console_goto(con, 0, 0);
         for (i = r0; i < r1; ++i) {
-                /* Starting point for row display */
-                rbc.row = pane.sh_row + i;
-                rbc.column = pane.sh_column;
+                sheet_get_row_width(&doc.sh, pane.sh_row + i, &width);
+
+                /* Determine row starting point. */
+                rbc.row = pane.sh_row + i; rbc.column = 1;
                 sheet_get_cell_pt(&doc.sh, &rbc, dir_before, &rb);
 
-                /* Ending point for row display */
-                rec.row = pane.sh_row + i;
-                rec.column = pane.sh_column + pane.columns;
+                /* Determine row ending point. */
+                rec.row = pane.sh_row + i; rec.column = width + 1;
                 sheet_get_cell_pt(&doc.sh, &rec, dir_before, &re);
 
@@ -455 +452 @@
                 /* Fill until the end of display area. */
 
-                if (str_length(row_buf) < (unsigned) scr_columns)
+                if (str_length(row_buf) < scr_columns)
                         fill = scr_columns - str_length(row_buf);
                 else
@@ -498 +495 @@
 
         spt_get_coord(&caret_pt, &coord);
-        console_goto(con, coord.column - pane.sh_column,
-            coord.row - pane.sh_row);
+        console_goto(con, coord.column - 1, coord.row - pane.sh_row);
 }
 
@@ -516 +512 @@
 
         (void) sheet_insert(&doc.sh, &pt, dir_before, cbuf);
-
-        pane.rflags |= REDRAW_ROW;
-        if (c == '\n')
-                pane.rflags |= REDRAW_TEXT;
 }
 
@@ -535 +527 @@
 
         (void) sheet_delete(&doc.sh, &sp, &ep);
-
-        pane.rflags |= REDRAW_ROW;
-        if (coord.column < 1)
-                pane.rflags |= REDRAW_TEXT;
 }
 
@@ -545 +533 @@
 {
         spt_t sp, ep;
-        coord_t sc, ec;
+        coord_t coord;
 
         tag_get_pt(&pane.caret_pos, &sp);
-        spt_get_coord(&sp, &sc);
-
-        sheet_get_cell_pt(&doc.sh, &sc, dir_after, &ep);
-        spt_get_coord(&ep, &ec);
+        spt_get_coord(&sp, &coord);
+
+        sheet_get_cell_pt(&doc.sh, &coord, dir_after, &ep);
 
         (void) sheet_delete(&doc.sh, &sp, &ep);
-
-        pane.rflags |= REDRAW_ROW;
-        if (ec.row != sc.row)
-                pane.rflags |= REDRAW_TEXT;
 }
 
@@ -573 +556 @@
         spt_get_coord(&pt, &coord);
 
-        /* Scroll pane vertically. */
+        /* Scroll pane as necessary. */
 
         if (coord.row < pane.sh_row) {
@@ -579 +562 @@
                 pane.rflags |= REDRAW_TEXT;
         }
-
         if (coord.row > pane.sh_row + pane.rows - 1) {
                 pane.sh_row = coord.row - pane.rows + 1;
@@ -585 +567 @@
         }
 
-        /* Scroll pane horizontally. */
-
-        if (coord.column < pane.sh_column) {
-                pane.sh_column = coord.column;
-                pane.rflags |= REDRAW_TEXT;
-        }
-
-        if (coord.column > pane.sh_column + pane.columns - 1) {
-                pane.sh_column = coord.column - pane.columns + 1;
-                pane.rflags |= REDRAW_TEXT;
-        }
-
         pane.rflags |= (REDRAW_CARET | REDRAW_STATUS);
+
 }
 
@@ -612 +583 @@
         coord_t coord;
         int num_rows;
-        bool pure_vertical;
 
         tag_get_pt(&pane.caret_pos, &pt);
@@ -625 +595 @@
                 if (coord.row > num_rows) coord.row = num_rows;
         }
-
-        /* For purely vertical movement try attaining @c ideal_column. */
-        pure_vertical = (dcolumn == 0 && align_dir == dir_before);
-        if (pure_vertical)
-                coord.column = pane.ideal_column;
 
         /*
@@ -639 +604 @@
         sheet_place_tag(&doc.sh, &pt, &pane.caret_pos);
 
-        /* For non-vertical movement set the new value for @c ideal_column. */
-        if (!pure_vertical) {
-                spt_get_coord(&pt, &coord);
-                pane.ideal_column = coord.column;
-        }
-
         caret_update();
 }