Change slightly how x/X works

src/context.hh

@@ -125,6 +125,19 @@ public:
             m_jump_list.push(selections());
     }
 
+    void enter_or_keep_line_editing() const {
+        m_keep_line_mode = true;
+    }
+
+    bool is_line_editing() const {
+        return m_line_mode;
+    }
+
+    void post_movement_logic() const {
+        m_line_mode = m_keep_line_mode;
+        m_keep_line_mode = false;
+    }
+
     template<typename Func>
     void set_last_select(Func&& last_select) { m_last_select = std::forward<Func>(last_select); }
 
@@ -135,6 +148,8 @@ private:
     void end_edition();
     int m_edition_level = 0;
     size_t m_edition_timestamp = 0;
+    mutable bool m_line_mode = false;
+    mutable bool m_keep_line_mode = false;
 
     friend struct ScopedEdition;
 

src/input_handler.cc

@@ -311,12 +311,14 @@ class Normal : public InputMode
                 m_params = { 0, 0 };
 
                 command->func(context(), params);
+                context().post_movement_logic();
             }
         }
 
         context().hooks().run_hook(Hook::NormalKey, key_to_str(key), context());
         if (enabled() and not transient) // The hook might have changed mode
             m_idle_timer.set_next_date(Clock::now() + get_idle_timeout(context()));
+
     }
 
     DisplayLine mode_line() const override

src/normal.cc

@@ -2001,7 +2001,12 @@ class Repeated
     void operator() (Context& context, NormalParams params)
     {
         ScopedEdition edition(context);
-        do { m_func(context, {0, params.reg}); } while(--params.count > 0);
+        do {
+            m_func(context, {0, params.reg});
+            if (params.count > 1) {
+                context.post_movement_logic();
+            }
+        } while(--params.count > 0);
     }
 private:
     T m_func;
@@ -2011,7 +2016,12 @@ template<void (*func)(Context&, NormalParams)>
 void repeated(Context& context, NormalParams params)
 {
     ScopedEdition edition(context);
-    do { func(context, {0, params.reg}); } while(--params.count > 0);
+    do {
+        func(context, {0, params.reg});
+        if (params.count > 1) {
+            context.post_movement_logic();
+        }
+    } while(--params.count > 0);
 }
 
 template<typename Type, Direction direction, SelectMode mode = SelectMode::Replace>

src/selectors.cc

@@ -175,11 +175,16 @@ select_line(const Context& context, const Selection& selection)
 {
     auto& buffer = context.buffer();
     auto line = selection.cursor().line;
-    // Next line if line fully selected
-    if (selection.anchor() <= BufferCoord{line, 0_byte} and
-        selection.cursor() == BufferCoord{line, buffer[line].length() - 1} and
-        line != buffer.line_count() - 1)
+
+    if (context.is_line_editing()) {
         ++line;
+    }
+
+    context.enter_or_keep_line_editing();
+
+    if (line >= buffer.line_count()) {
+        return Optional<Selection>();
+    }
     return target_eol({{line, 0_byte}, {line, buffer[line].length() - 1}});
 }
 
@@ -811,6 +816,8 @@ select_argument(const Context& context, const Selection& selection,
 Optional<Selection>
 select_lines(const Context& context, const Selection& selection)
 {
+    context.enter_or_keep_line_editing();
+
     auto& buffer = context.buffer();
     BufferCoord anchor = selection.anchor();
     BufferCoord cursor  = selection.cursor();
@@ -832,6 +839,9 @@ trim_partial_lines(const Context& context, const Selection& selection)
     BufferCoord& to_line_start = anchor <= cursor ? anchor : cursor;
     BufferCoord& to_line_end = anchor <= cursor ? cursor : anchor;
 
+
+    context.enter_or_keep_line_editing();
+
     if (to_line_start.column != 0)
         to_line_start = to_line_start.line+1;
     if (to_line_end.column != buffer[to_line_end.line].length()-1)