Changes in uspace/app/edit/edit.c [3052ff4:743e17b] 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;
 
@@ -88 +94 @@
 static pane_t pane;
 
-static ipcarg_t scr_rows, scr_columns;
+static int scr_rows, scr_columns;
 
 #define ROW_BUF_SIZE 4096
@@ -130 +136 @@
 
         pane.rows = scr_rows - 1;
+        pane.columns = scr_columns;
         pane.sh_row = 1;
+        pane.sh_column = 1;
 
         /* Start with an empty sheet. */
@@ -139 +147 @@
         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) {
@@ -206 +215 @@
         case KC_ENTER:
                 insert_char('\n');
-                pane.rflags |= REDRAW_TEXT;
                 caret_update();
                 break;
@@ -235 +243 @@
         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;
@@ -246 +252 @@
                 if (ev->c >= 32 || ev->c == '\t') {
                         insert_char(ev->c);
-                        pane.rflags |= REDRAW_ROW;
                         caret_update();
                 }
@@ -362 +367 @@
 {
         int sh_rows, rows;
-        int i;
-        unsigned j;
+        int i, j;
 
         sheet_get_num_rows(&doc.sh, &sh_rows);
@@ -403 +407 @@
 static void pane_row_range_display(int r0, int r1)
 {
-        int width;
         int i, j, fill;
         spt_t rb, re, dep;
@@ -416 +419 @@
         console_goto(con, 0, 0);
         for (i = r0; i < r1; ++i) {
-                sheet_get_row_width(&doc.sh, pane.sh_row + i, &width);
-
-                /* Determine row starting point. */
-                rbc.row = pane.sh_row + i; rbc.column = 1;
+                /* Starting point for row display */
+                rbc.row = pane.sh_row + i;
+                rbc.column = pane.sh_column;
                 sheet_get_cell_pt(&doc.sh, &rbc, dir_before, &rb);
 
-                /* Determine row ending point. */
-                rec.row = pane.sh_row + i; rec.column = width + 1;
+                /* Ending point for row display */
+                rec.row = pane.sh_row + i;
+                rec.column = pane.sh_column + pane.columns;
                 sheet_get_cell_pt(&doc.sh, &rec, dir_before, &re);
 
@@ -452 +455 @@
                 /* Fill until the end of display area. */
 
-                if (str_length(row_buf) < scr_columns)
+                if (str_length(row_buf) < (unsigned) scr_columns)
                         fill = scr_columns - str_length(row_buf);
                 else
@@ -495 +498 @@
 
         spt_get_coord(&caret_pt, &coord);
-        console_goto(con, coord.column - 1, coord.row - pane.sh_row);
+        console_goto(con, coord.column - pane.sh_column,
+            coord.row - pane.sh_row);
 }
 
@@ -512 +516 @@
 
         (void) sheet_insert(&doc.sh, &pt, dir_before, cbuf);
+
+        pane.rflags |= REDRAW_ROW;
+        if (c == '\n')
+                pane.rflags |= REDRAW_TEXT;
 }
 
@@ -527 +535 @@
 
         (void) sheet_delete(&doc.sh, &sp, &ep);
+
+        pane.rflags |= REDRAW_ROW;
+        if (coord.column < 1)
+                pane.rflags |= REDRAW_TEXT;
 }
 
@@ -533 +545 @@
 {
         spt_t sp, ep;
-        coord_t coord;
+        coord_t sc, ec;
 
         tag_get_pt(&pane.caret_pos, &sp);
-        spt_get_coord(&sp, &coord);
-
-        sheet_get_cell_pt(&doc.sh, &coord, dir_after, &ep);
+        spt_get_coord(&sp, &sc);
+
+        sheet_get_cell_pt(&doc.sh, &sc, dir_after, &ep);
+        spt_get_coord(&ep, &ec);
 
         (void) sheet_delete(&doc.sh, &sp, &ep);
+
+        pane.rflags |= REDRAW_ROW;
+        if (ec.row != sc.row)
+                pane.rflags |= REDRAW_TEXT;
 }
 
@@ -556 +573 @@
         spt_get_coord(&pt, &coord);
 
-        /* Scroll pane as necessary. */
+        /* Scroll pane vertically. */
 
         if (coord.row < pane.sh_row) {
@@ -562 +579 @@
                 pane.rflags |= REDRAW_TEXT;
         }
+
         if (coord.row > pane.sh_row + pane.rows - 1) {
                 pane.sh_row = coord.row - pane.rows + 1;
@@ -567 +585 @@
         }
 
+        /* 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);
-
 }
 
@@ -583 +612 @@
         coord_t coord;
         int num_rows;
+        bool pure_vertical;
 
         tag_get_pt(&pane.caret_pos, &pt);
@@ -595 +625 @@
                 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;
 
         /*
@@ -604 +639 @@
         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();
 }