[terminal] replace set_active_vt with activate_vt

Since a terminal already knows which VT number it is, there's no
need to accept a VT number for switching to this VT.

diff --git a/src/libply-splash-core/ply-terminal.c b/src/libply-splash-core/ply-terminal.c
index ed086575d7b9ecbaf4117efd5790078003eadaee..7725eb4f07d90572c15aa2df3201c0bbef77c020 100644 (file)
--- a/src/libply-splash-core/ply-terminal.c
+++ b/src/libply-splash-core/ply-terminal.c
@@ -695,23 +695,32 @@ ply_terminal_get_active_vt (ply_terminal_t *terminal)
   return terminal->active_vt;
 }
 
+static bool
+set_active_vt (ply_terminal_t *terminal,
+               int             vt_number)
+{
+  if (ioctl (terminal->fd, VT_ACTIVATE, vt_number) < 0)
+    return false;
+
+  terminal->next_active_vt = vt_number;
+
+  return true;
+}
+
 bool
-ply_terminal_set_active_vt (ply_terminal_t *terminal,
-                            int             vt_number)
+ply_terminal_activate_vt (ply_terminal_t *terminal)
 {
   assert (terminal != NULL);
 
-  if (vt_number <= 0)
+  if (terminal->vt_number <= 0)
     return false;
 
-  if (vt_number == terminal->active_vt)
+  if (terminal->vt_number == terminal->active_vt)
     return true;
 
-  if (ioctl (terminal->fd, VT_ACTIVATE, vt_number) < 0)
+  if (!set_active_vt (terminal, terminal->vt_number))
     return false;
 
-  terminal->next_active_vt = vt_number;
-
   return true;
 }
 

diff --git a/src/libply-splash-core/ply-terminal.h b/src/libply-splash-core/ply-terminal.h
index 4b9ef5a40c38591685abc3efc40adf9616eed21e..2c9c4f84ed32acba519a4b7922d2dba65765d596 100644 (file)
--- a/src/libply-splash-core/ply-terminal.h
+++ b/src/libply-splash-core/ply-terminal.h
@@ -90,8 +90,7 @@ void ply_terminal_ignore_mode_changes (ply_terminal_t *terminal,
 
 int ply_terminal_get_vt_number (ply_terminal_t *terminal);
 int ply_terminal_get_active_vt (ply_terminal_t *terminal);
-bool ply_terminal_set_active_vt (ply_terminal_t *terminal,
-                                 int             vt_number);
+bool ply_terminal_activate_vt (ply_terminal_t *terminal);
 
 void ply_terminal_watch_for_active_vt_change (ply_terminal_t *terminal,
                                               ply_terminal_active_vt_changed_handler_t active_vt_changed_handler,

diff --git a/src/main.c b/src/main.c
index 76ba53a8c7cecb40f522575a256049672df60b13..35c287f522c02a86d5b702925660f885f704ab91 100644 (file)
--- a/src/main.c
+++ b/src/main.c
@@ -1021,8 +1021,7 @@ add_display_and_keyboard_for_terminal (state_t    *state,
 
   state->terminal = terminal;
 
-  ply_terminal_set_active_vt (state->terminal,
-                              ply_terminal_get_vt_number (state->terminal));
+  ply_terminal_activate_vt (state->terminal);
 
   keyboard = ply_keyboard_new_for_terminal (state->terminal);
   display = ply_text_display_new (state->terminal);

diff --git a/src/plugins/renderers/drm/plugin.c b/src/plugins/renderers/drm/plugin.c
index 48e9ab0c3367f9ee5e19efdbdd5d69274775aa08..549955d7a47411c24db48a374cdc7c1c7255ae6a 100644 (file)
--- a/src/plugins/renderers/drm/plugin.c
+++ b/src/plugins/renderers/drm/plugin.c
@@ -806,8 +806,7 @@ map_to_device (ply_renderer_backend_t *backend)
       node = next_node;
     }
 
-  ply_terminal_set_active_vt (backend->terminal,
-                              ply_terminal_get_vt_number (backend->terminal));
+  ply_terminal_activate_vt (backend->terminal);
 
   return head_mapped;
 }

diff --git a/src/plugins/renderers/frame-buffer/plugin.c b/src/plugins/renderers/frame-buffer/plugin.c
index ff2d1dcec4827823e98c9b4922f541cae993216b..1c0e34b869a628938742e053efb70709ad808e3c 100644 (file)
--- a/src/plugins/renderers/frame-buffer/plugin.c
+++ b/src/plugins/renderers/frame-buffer/plugin.c
@@ -512,8 +512,7 @@ map_to_device (ply_renderer_backend_t *backend)
   if (head->map_address == MAP_FAILED)
     return false;
 
-  ply_terminal_set_active_vt (backend->terminal,
-                              ply_terminal_get_vt_number (backend->terminal));
+  ply_terminal_activate_vt (backend->terminal);
 
   return true;
 }