gl and gh now go to first or last character of the line

append mode is supported through G key

src/main.cc

@@ -322,11 +322,16 @@ void do_insert(Window& window, IncrementalInserter::Mode mode)
     window.clear_selections();
 }
 
+template<bool append>
 void do_go(Window& window, int count)
 {
-    BufferIterator target;
     if (count != 0)
-        target = window.buffer().iterator_at({count, 0});
+    {
+        BufferIterator target =
+            window.buffer().iterator_at(BufferCoord(count, 0));
+
+        window.move_cursor_to(window.line_and_column_at(target));
+    }
     else
     {
         char c = getch();
@@ -334,28 +339,29 @@ void do_go(Window& window, int count)
         {
         case 'g':
         case 't':
-            target = window.buffer().iterator_at({0,0});
+        {
+            BufferIterator target =
+                window.buffer().iterator_at(BufferCoord(0,0));
+            window.move_cursor_to(window.line_and_column_at(target));
             break;
+        }
         case 'l':
-            target = window.iterator_at(window.cursor_position());
+        case 'L':
-            while (not target.is_end() and *target != '\n')
+            window.select(select_to_eol, append);
-                ++target;
-            --target;
             break;
         case 'h':
-            target = window.iterator_at(window.cursor_position());
+        case 'H':
-            while (not target.is_begin() and *target != '\n')
+            window.select(select_to_eol_reverse, append);
-                --target;
-            ++target;
             break;
         case 'b':
-            target = window.buffer().iterator_at(
+        {
-                {window.buffer().line_count() - 1, 0});
+            BufferIterator target = window.buffer().iterator_at(
-            break;
+                BufferCoord(window.buffer().line_count() - 1, 0));
-            break;
-        }
-    }
             window.move_cursor_to(window.line_and_column_at(target));
+            break;
+        }
+        }
+    }
 }
 
 Window* current_window;
@@ -516,7 +522,8 @@ std::unordered_map<char, std::function<void (Window& window, int count)>> keymap
     { 'o', [](Window& window, int count) { do_insert(window, IncrementalInserter::Mode::OpenLineBelow); } },
     { 'O', [](Window& window, int count) { do_insert(window, IncrementalInserter::Mode::OpenLineAbove); } },
 
-    { 'g', do_go },
+    { 'g', do_go<false> },
+    { 'G', do_go<true> },
 
     { 'y', do_yank },
     { 'p', do_paste<true> },

src/selectors.cc

@@ -223,4 +223,18 @@ Selection select_to_reverse(const BufferIterator& cursor, char c, int count, boo
     return Selection(cursor, inclusive ? end : end+1);
 }
 
+Selection select_to_eol(const BufferIterator& cursor)
+{
+    BufferIterator end = cursor + 1;
+    skip_while(end, [](char cur) { return not is_eol(cur); });
+    return Selection(cursor, end-1);
+}
+
+Selection select_to_eol_reverse(const BufferIterator& cursor)
+{
+    BufferIterator end = cursor - 1;
+    skip_while_reverse(end, [](char cur) { return not is_eol(cur); });
+    return Selection(cursor, end.is_begin() ? end : end+1);
+}
+
 }

src/selectors.hh

@@ -16,6 +16,8 @@ Selection select_matching(const BufferIterator& cursor);
 Selection select_to(const BufferIterator& cursor, char c, int count, bool inclusive);
 Selection select_to_reverse(const BufferIterator& cursor, char c, int count, bool inclusive);
 
+Selection select_to_eol(const BufferIterator& cursor);
+Selection select_to_eol_reverse(const BufferIterator& cursor);
 }
 
 #endif // selectors_hh_INCLUDED