Avoid duplicating the logic in Entry::get_selection

diff --git a/source/entry.cpp b/source/entry.cpp
index 19bb7cc48b9f038d86d95401941e5b30f71cbe02..fec27397eaf8a445f068e8edf19b2d6f0decb882 100644 (file)
--- a/source/entry.cpp
+++ b/source/entry.cpp
@@ -165,10 +165,8 @@ void Entry::rebuild_special(const Part &part)
                if(!text_part || !graphic || !graphic->get_texture())
                        return;
 
-               unsigned start = selection_pos;
-               unsigned end = edit_pos;
-               if(start>end)
-                       swap(start, end);
+               unsigned start, end;
+               get_selection(start, end);
 
                unsigned row, col;
                text.offset_to_coords(start, row, col);
@@ -386,14 +384,10 @@ void Entry::set_edit_position(unsigned ep, bool select)
 
 void Entry::erase_selection()
 {
-       if(!selection_active)
+       unsigned start, end;
+       if(!get_selection(start, end))
                return;
 
-       unsigned start = selection_pos;
-       unsigned end = edit_pos;
-       if(start>end)
-               swap(start, end);
-
        text.erase(start, end-start);
        set_edit_position(start, false);
 }