Merge pull request #2150 from profanity-im/fix/2148
fix: resolve issues with async editor
This commit is contained in:
@@ -139,6 +139,8 @@ prof_run(gchar* log_level, gchar* account_name, gchar* config_file, gchar* log_f
|
|||||||
session_process_events();
|
session_process_events();
|
||||||
iq_autoping_check();
|
iq_autoping_check();
|
||||||
ui_update();
|
ui_update();
|
||||||
|
while (g_main_context_iteration(NULL, FALSE))
|
||||||
|
;
|
||||||
#ifdef HAVE_GTK
|
#ifdef HAVE_GTK
|
||||||
tray_update();
|
tray_update();
|
||||||
#endif
|
#endif
|
||||||
|
|||||||
@@ -14,6 +14,7 @@
|
|||||||
#include <errno.h>
|
#include <errno.h>
|
||||||
#include <sys/wait.h>
|
#include <sys/wait.h>
|
||||||
#include <unistd.h>
|
#include <unistd.h>
|
||||||
|
#include <signal.h>
|
||||||
|
|
||||||
#include "config/files.h"
|
#include "config/files.h"
|
||||||
#include "config/preferences.h"
|
#include "config/preferences.h"
|
||||||
@@ -128,6 +129,12 @@ launch_editor(gchar* initial_content, void (*callback)(gchar* content, void* dat
|
|||||||
return TRUE;
|
return TRUE;
|
||||||
} else if (pid == 0) {
|
} else if (pid == 0) {
|
||||||
// Child process: Inherits TTY from parent
|
// Child process: Inherits TTY from parent
|
||||||
|
|
||||||
|
// Reset signal handlers that profanity sets so the editor doesn't inherit them
|
||||||
|
signal(SIGINT, SIG_DFL);
|
||||||
|
signal(SIGTSTP, SIG_DFL);
|
||||||
|
signal(SIGPIPE, SIG_DFL);
|
||||||
|
|
||||||
if (editor_argv && editor_argv[0]) {
|
if (editor_argv && editor_argv[0]) {
|
||||||
execvp(editor_argv[0], editor_argv);
|
execvp(editor_argv[0], editor_argv);
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -179,6 +179,7 @@ ui_reset_idle_time(void)
|
|||||||
void
|
void
|
||||||
ui_suspend(void)
|
ui_suspend(void)
|
||||||
{
|
{
|
||||||
|
inp_suspend();
|
||||||
endwin();
|
endwin();
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -186,6 +187,7 @@ void
|
|||||||
ui_resume(void)
|
ui_resume(void)
|
||||||
{
|
{
|
||||||
refresh();
|
refresh();
|
||||||
|
inp_resume();
|
||||||
}
|
}
|
||||||
|
|
||||||
void
|
void
|
||||||
|
|||||||
@@ -65,6 +65,7 @@ static fd_set fds;
|
|||||||
static int r;
|
static int r;
|
||||||
static char* inp_line = NULL;
|
static char* inp_line = NULL;
|
||||||
static gboolean get_password = FALSE;
|
static gboolean get_password = FALSE;
|
||||||
|
static gboolean is_suspended = FALSE;
|
||||||
|
|
||||||
static void _inp_win_update_virtual(void);
|
static void _inp_win_update_virtual(void);
|
||||||
static int _inp_edited(const wint_t ch);
|
static int _inp_edited(const wint_t ch);
|
||||||
@@ -172,7 +173,7 @@ char*
|
|||||||
inp_readline(void)
|
inp_readline(void)
|
||||||
{
|
{
|
||||||
// UI is suspended
|
// UI is suspended
|
||||||
if (isendwin()) {
|
if (is_suspended || isendwin()) {
|
||||||
g_usleep(100000); // 100ms
|
g_usleep(100000); // 100ms
|
||||||
return NULL;
|
return NULL;
|
||||||
}
|
}
|
||||||
@@ -272,6 +273,22 @@ inp_close(void)
|
|||||||
discard = NULL;
|
discard = NULL;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
void
|
||||||
|
inp_suspend(void)
|
||||||
|
{
|
||||||
|
is_suspended = TRUE;
|
||||||
|
rl_callback_handler_remove();
|
||||||
|
rl_deprep_terminal();
|
||||||
|
}
|
||||||
|
|
||||||
|
void
|
||||||
|
inp_resume(void)
|
||||||
|
{
|
||||||
|
is_suspended = FALSE;
|
||||||
|
rl_callback_handler_install(NULL, _inp_rl_linehandler);
|
||||||
|
rl_prep_terminal(0);
|
||||||
|
}
|
||||||
|
|
||||||
char*
|
char*
|
||||||
inp_get_line(void)
|
inp_get_line(void)
|
||||||
{
|
{
|
||||||
|
|||||||
@@ -18,6 +18,8 @@ void create_input_window(void);
|
|||||||
void inp_close(void);
|
void inp_close(void);
|
||||||
void inp_win_resize(void);
|
void inp_win_resize(void);
|
||||||
void inp_put_back(void);
|
void inp_put_back(void);
|
||||||
|
void inp_suspend(void);
|
||||||
|
void inp_resume(void);
|
||||||
char* inp_get_password(void);
|
char* inp_get_password(void);
|
||||||
char* inp_get_line(void);
|
char* inp_get_line(void);
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user