diff options
author | Dmitry Saprykin <saprykin.dmitry@gmail.com> | 2014-07-24 17:58:39 +0200 |
---|---|---|
committer | Johannes Schlüter <johannes@php.net> | 2014-07-24 17:58:39 +0200 |
commit | d491b2f916d061666d9ff1cb5bdc484961b82db0 (patch) | |
tree | 11e9d316355078cc6d76130ee9be08847ad55b16 | |
parent | 71d3a69425449972f4efdf7228c6f7e49e090755 (diff) | |
download | php-git-d491b2f916d061666d9ff1cb5bdc484961b82db0.tar.gz |
Fix bug #67496 Save command history when exiting interactive shell with control-c
-rw-r--r-- | NEWS | 2 | ||||
-rw-r--r-- | ext/readline/readline_cli.c | 12 |
2 files changed, 13 insertions, 1 deletions
@@ -5,6 +5,8 @@ PHP NEWS - Readline: . Fixed bug #55496 (Interactive mode doesn't force a newline before the prompt). (Bob, Johannes) + . Fixed bug #67496 (Save command history when exiting interactive shell + with control-c). (Dmitry Saprykin, Johannes) ?? ??? 2014, PHP 5.4.31 diff --git a/ext/readline/readline_cli.c b/ext/readline/readline_cli.c index 9655777775..9c27acb193 100644 --- a/ext/readline/readline_cli.c +++ b/ext/readline/readline_cli.c @@ -590,6 +590,7 @@ static int readline_shell_run(TSRMLS_D) /* {{{ */ char *code = emalloc(size); char *prompt = cli_get_prompt("php", '>' TSRMLS_CC); char *history_file; + int history_lines_to_write = 0; if (PG(auto_prepend_file) && PG(auto_prepend_file)[0]) { zend_file_handle *prepend_file_p; @@ -654,6 +655,7 @@ static int readline_shell_run(TSRMLS_D) /* {{{ */ if (*line) { add_history(line); + history_lines_to_write += 1; } free(line); @@ -663,6 +665,15 @@ static int readline_shell_run(TSRMLS_D) /* {{{ */ continue; } + if (history_lines_to_write) { +#if HAVE_LIBEDIT + write_history(history_file); +#else + append_history(history_lines_to_write, history_file); +#endif + history_lines_to_write = 0; + } + zend_try { zend_eval_stringl(code, pos, NULL, "php shell code" TSRMLS_CC); } zend_end_try(); @@ -684,7 +695,6 @@ static int readline_shell_run(TSRMLS_D) /* {{{ */ php_last_char = '\0'; } - write_history(history_file); free(history_file); efree(code); efree(prompt); |