summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDmitry Saprykin <saprykin.dmitry@gmail.com>2014-07-24 17:58:39 +0200
committerJohannes Schlüter <johannes@php.net>2014-07-24 17:58:39 +0200
commitd491b2f916d061666d9ff1cb5bdc484961b82db0 (patch)
tree11e9d316355078cc6d76130ee9be08847ad55b16
parent71d3a69425449972f4efdf7228c6f7e49e090755 (diff)
downloadphp-git-d491b2f916d061666d9ff1cb5bdc484961b82db0.tar.gz
Fix bug #67496 Save command history when exiting interactive shell with control-c
-rw-r--r--NEWS2
-rw-r--r--ext/readline/readline_cli.c12
2 files changed, 13 insertions, 1 deletions
diff --git a/NEWS b/NEWS
index c4592a3536..89d41ab384 100644
--- a/NEWS
+++ b/NEWS
@@ -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);