commit 2acfbed9dbea990f129535de7ff3df360365130b
author Bram Moolenaar <Bram@vim.org> Fri Jul 01 23:14:02 2016 +0200
committer Bram Moolenaar <Bram@vim.org> Fri Jul 01 23:14:02 2016 +0200
tree a7a0be95e7ed13ae5450de39b948a1ef34d318c0
parent c5af40ae646ceda817eff93b4f9ba274f031bea6

patch 7.4.1979
Problem:    Getting value of binary option is wrong. (Kent Sibilev)
Solution:   Fix type cast.  Add a test.

src/option.c

diff --git a/src/option.c b/src/option.c
index 47bbe73..95f4c73 100644
--- a/src/option.c
+++ b/src/option.c
@@ -9102,7 +9102,7 @@
 	if ((int *)varp == &curbuf->b_changed)
 	    *numval = curbufIsChanged();
 	else
-	    *numval = (long) *(varnumber_T *)varp;
+	    *numval = (long) *(int *)varp;
     }
     return 1;
 }

src/testdir/test_expr.vim

diff --git a/src/testdir/test_expr.vim b/src/testdir/test_expr.vim
index f44e216..067dbf2 100644
--- a/src/testdir/test_expr.vim
+++ b/src/testdir/test_expr.vim
@@ -106,3 +106,26 @@
   " The failure is only visible using valgrind.
   call assert_fails('echo "\<C-">')
 endfunc
+
+func Test_option_value()
+  " boolean
+  set bri
+  call assert_equal(1, &bri)
+  set nobri
+  call assert_equal(0, &bri)
+
+  " number
+  set ts=1
+  call assert_equal(1, &ts)
+  set ts=8
+  call assert_equal(8, &ts)
+
+  " string
+  exe "set cedit=\<Esc>"
+  call assert_equal("\<Esc>", &cedit)
+  set cpo=
+  call assert_equal("", &cpo)
+  set cpo=abcdefgi
+  call assert_equal("abcdefgi", &cpo)
+  set cpo&vim
+endfunc

src/version.c

diff --git a/src/version.c b/src/version.c
index 1613a72..1e9672b 100644
--- a/src/version.c
+++ b/src/version.c
@@ -759,6 +759,8 @@
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    1979,
+/**/
     1978,
 /**/
     1977,