Ticket #2229: mc-null_viewer.patch

src/viewer/dialogs.c

@@ -280 +280 @@
             default:
                 break;
             }
-            *offset = mcview_bol (view, *offset);
+            *offset = mcview_bol (view, *offset, 0);
         }
     }
 

src/viewer/internal.h

@@ -290 +290 @@
 void mcview_select_encoding (mcview_t * view);
 void mcview_set_codeset (mcview_t * view);
 void mcview_show_error (mcview_t * view, const char *error);
-off_t mcview_bol (mcview_t * view, off_t current);
-off_t mcview_eol (mcview_t * view, off_t current);
+off_t mcview_bol (mcview_t * view, off_t current, off_t limit);
+off_t mcview_eol (mcview_t * view, off_t current, off_t limit);
 char *mcview_get_title (const Dlg_head * h, size_t len);
 gboolean mcview_lock_file (mcview_t * view);
 gboolean mcview_unlock_file (mcview_t * view);

src/viewer/lib.c

@@ -96 +96 @@
 mcview_toggle_wrap_mode (mcview_t * view)
 {
     if (view->text_wrap_mode)
-        view->dpy_start = mcview_bol (view, view->dpy_start);
+        view->dpy_start = mcview_bol (view, view->dpy_start, 0);
     view->text_wrap_mode = !view->text_wrap_mode;
     view->dpy_bbar_dirty = TRUE;
     view->dirty++;
@@ -339 +339 @@
 /* returns index of the first char in the line */
 /* it is constant for all line characters */
 off_t
-mcview_bol (mcview_t * view, off_t current)
+mcview_bol (mcview_t * view, off_t current, off_t limit)
 {
     int c;
     off_t filesize;
@@ -357 +357 @@
         if (c == '\r')
             current--;
     }
-    while (current > 0)
+    while (current > 0 && current >= limit)
     {
         if (!mcview_get_byte (view, current - 1, &c))
             break;
@@ -373 +373 @@
 /* returns index of last char on line + width EOL */
 /* mcview_eol of the current line == mcview_bol next line */
 off_t
-mcview_eol (mcview_t * view, off_t current)
+mcview_eol (mcview_t * view, off_t current, off_t limit)
 {
     int c, prev_ch = 0;
     off_t filesize;
@@ -382 +382 @@
         return 0;
     if (current >= filesize)
         return filesize;
-    while (current < filesize)
+    while (current < filesize && current < limit)
     {
         if (!mcview_get_byte (view, current, &c))
             break;

src/viewer/mcviewer.c

@@ -414 +414 @@
         canon_fname = vfs_canon (view->filename);
         load_file_position (canon_fname, &line, &col, &new_offset);
         new_offset = min (new_offset, mcview_get_filesize (view));
-        view->dpy_start = mcview_bol (view, new_offset);
+        view->dpy_start = mcview_bol (view, new_offset, 0);
         g_free (canon_fname);
     }
     else if (start_line > 0)

src/viewer/move.c

@@ -108 +108 @@
         off_t i;
         for (i = 0; i < lines; i++)
         {
-            off_t cur_bol;
-            cur_bol = new_offset = mcview_bol (view, view->dpy_start);
-            if (new_offset > 0)
-                new_offset--;
-            new_offset = mcview_bol (view, new_offset);
-            if (new_offset < 0)
-                new_offset = 0;
+            if (view->dpy_start==0) break;
             if (view->text_wrap_mode)
             {
-                size_t last_row_length = (view->dpy_start - new_offset) % view->data_area.width;
-                if (last_row_length != 0 && cur_bol == view->dpy_start)
-                    new_offset = max (new_offset, (off_t) (view->dpy_start - last_row_length));
-                else
-                    new_offset = max (new_offset, view->dpy_start - view->data_area.width);
+                new_offset = mcview_bol (view, view->dpy_start, view->dpy_start - 1);   // check if dpy_start==BOL or not (then new_offset=dpy_start-1, no need to check more)
+            if (new_offset == view->dpy_start) {
+                new_offset--;
+                new_offset = mcview_bol (view, new_offset, 0);
+                size_t last_row_length = (view->dpy_start - new_offset) % view->data_area.width;
+                if (last_row_length > 0)
+                        new_offset = view->dpy_start - last_row_length;         // if dpy_start==BOL in wrapped mode, find BOL of previous line and move down all but the last rows
+            } else      new_offset = view->dpy_start - view->data_area.width;   // if dpy_start!=BOL in wrapped mode, just move one row up; no need to check if > 0 as there is at least exactly one wrap between dpy_start and BOL
+                view->dpy_start = new_offset;
+            } else {    // if unwrapped -> current BOL equals dpy_start, just find BOL of previous line
+                new_offset = view->dpy_start;
+                new_offset--;
+                view->dpy_start = mcview_bol (view, new_offset, 0);
             }
-            view->dpy_start = new_offset;
         }
     }
     mcview_movement_fixups (view, TRUE);
@@ -160 +161 @@
         {
             while (lines-- > 0)
             {
-                new_offset = mcview_eol (view, view->dpy_end);
                 if (view->text_wrap_mode)
-                    new_offset = min (new_offset, view->dpy_end + view->data_area.width);
-                view->dpy_end = new_offset;
+                        view->dpy_end = mcview_eol (view, view->dpy_end, view->dpy_end + view->data_area.width);
+                else    view->dpy_end = mcview_eol (view, view->dpy_end, last_byte);
                 if(view->dpy_end>=last_byte) lines=0;
 
-                new_offset = mcview_eol (view, view->dpy_start);
                 if (view->text_wrap_mode)
-                    new_offset = min (new_offset, view->dpy_start + view->data_area.width);
+                        new_offset = mcview_eol (view, view->dpy_start, view->dpy_start + view->data_area.width);
+                else    new_offset = mcview_eol (view, view->dpy_start, last_byte);
                 if(new_offset<last_byte) view->dpy_start = new_offset;
             }
         }
@@ -177 +177 @@
 
             for (i = 0; i < lines && new_offset < last_byte; i++)
             {
-                new_offset = mcview_eol (view, view->dpy_start);
                 if (view->text_wrap_mode)
-                    new_offset = min (new_offset, view->dpy_start + view->data_area.width);
+                        new_offset = mcview_eol (view, view->dpy_start, view->dpy_start + view->data_area.width);
+                else    new_offset = mcview_eol (view, view->dpy_start, last_byte);
                 if(new_offset<last_byte) view->dpy_start = new_offset;
             }
         }
@@ -312 +312 @@
     }
     else if (!view->text_wrap_mode)
     {
-        view->dpy_start = mcview_bol (view, view->dpy_start);
+        view->dpy_start = mcview_bol (view, view->dpy_start, 0);
     }
     view->dpy_text_column = 0;
     mcview_movement_fixups (view, TRUE);
@@ -342 +342 @@
     else
     {
         off_t eol;
-        bol = mcview_bol (view, view->dpy_start);
-        eol = mcview_eol (view, view->dpy_start);
+        bol = mcview_bol (view, view->dpy_start, 0);
+        eol = mcview_eol (view, view->dpy_start, mcview_get_filesize (view));
         if (!view->utf8)
         {
             if (eol > bol)
@@ -465 +465 @@
     }
     else
     {
-        view->dpy_start = mcview_bol (view, offset);
+        view->dpy_start = mcview_bol (view, offset, 0);
     }
 
     mcview_scroll_to_cursor (view);