patch 8.0.0867: job and channel in a dict value not quoted

Problem:    When using a job or channel value as a dict value, when turning it
            into a string the quotes are missing.
Solution:   Add quotes to the job and channel values. (Yasuhiro Matsumoto,
            closes #1930)

diff --git a/src/eval.c b/src/eval.c
index c35def0b27465592d8944fd5c67860262c186ff6..b8e606443646d4c36070054fa2bcc78f29e89dec 100644 (file)
--- a/src/eval.c
+++ b/src/eval.c
@@ -5683,9 +5683,9 @@ get_var_special_name(int nr)
  * If the memory is allocated "tofree" is set to it, otherwise NULL.
  * "numbuf" is used for a number.
  * When "copyID" is not NULL replace recursive lists and dicts with "...".
- * When both "echo_style" and "dict_val" are FALSE, put quotes around stings as
- * "string()", otherwise does not put quotes around strings, as ":echo"
- * displays values.
+ * When both "echo_style" and "composite_val" are FALSE, put quotes around
+ * stings as "string()", otherwise does not put quotes around strings, as
+ * ":echo" displays values.
  * When "restore_copyID" is FALSE, repeated items in dictionaries and lists
  * are replaced with "...".
  * May return NULL.
@@ -5698,7 +5698,7 @@ echo_string_core(
     int                copyID,
     int                echo_style,
     int                restore_copyID,
-    int                dict_val)
+    int                composite_val)
 {
     static int recurse = 0;
     char_u     *r = NULL;
@@ -5721,10 +5721,12 @@ echo_string_core(
     switch (tv->v_type)
     {
        case VAR_STRING:
-           if (echo_style && !dict_val)
+           if (echo_style && !composite_val)
            {
                *tofree = NULL;
-               r = get_tv_string_buf(tv, numbuf);
+               r = tv->vval.v_string;
+               if (r == NULL)
+                   r = (char_u *)"";
            }
            else
            {
@@ -5841,10 +5843,19 @@ echo_string_core(
 
        case VAR_NUMBER:
        case VAR_UNKNOWN:
+           *tofree = NULL;
+           r = get_tv_string_buf(tv, numbuf);
+           break;
+
        case VAR_JOB:
        case VAR_CHANNEL:
            *tofree = NULL;
            r = get_tv_string_buf(tv, numbuf);
+           if (composite_val)
+           {
+               *tofree = string_quote(r, FALSE);
+               r = *tofree;
+           }
            break;
 
        case VAR_FLOAT:

diff --git a/src/list.c b/src/list.c
index 2fccbae76459d5cd02445a964a57f0392cea6c65..b593f7171926140d29a21ac25c8669b8dd1be03f 100644 (file)
--- a/src/list.c
+++ b/src/list.c
@@ -740,7 +740,7 @@ list_join_inner(
     for (item = l->lv_first; item != NULL && !got_int; item = item->li_next)
     {
        s = echo_string_core(&item->li_tv, &tofree, numbuf, copyID,
-                                          echo_style, restore_copyID, FALSE);
+                                     echo_style, restore_copyID, !echo_style);
        if (s == NULL)
            return FAIL;
 

diff --git a/src/testdir/test_terminal.vim b/src/testdir/test_terminal.vim
index 98bc75843e9b4ceae85c5d19d53d244ef1411f34..8c14fa4d034bc674f8578b7a5c7796e747706dd7 100644 (file)
--- a/src/testdir/test_terminal.vim
+++ b/src/testdir/test_terminal.vim
@@ -18,6 +18,9 @@ func Run_shell_in_terminal()
   let g:job = term_getjob(buf)
   call assert_equal(v:t_job, type(g:job))
 
+  let string = string({'job': term_getjob(buf)})
+  call assert_match("{'job': 'process \\d\\+ run'}", string)
+
   return buf
 endfunc
 

diff --git a/src/version.c b/src/version.c
index 3595c1af57555f62092b590a7d2bcd5d107e28b0..d3b3215fc065b8afc1e805d18f5deb3b6ebaa1c9 100644 (file)
--- a/src/version.c
+++ b/src/version.c
@@ -769,6 +769,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    867,
 /**/
     866,
 /**/