# HG changeset patch
# User Bram Moolenaar <Bram@vim.org>
# Date 1631562304 -7200
# Node ID 5ef704de7709ade5a9328c1967ce1c934b659496
# Parent  7124fbe5db0464596362d2cfad0b2a77b21d7a3f
patch 8.2.3436: check for optional bool type has confusing return type

Commit: https://github.com/vim/vim/commit/a29856fcdc41541c607437629c1c1f45951100a4
Author: Bram Moolenaar <Bram@vim.org>
Date:   Mon Sep 13 21:36:27 2021 +0200

    patch 8.2.3436: check for optional bool type has confusing return type

    Problem:    Check for optional bool type has confusing return type.
    Solution:   Explicitly return OK.

diff --git a/src/typval.c b/src/typval.c
--- a/src/typval.c
+++ b/src/typval.c
@@ -459,13 +459,15 @@ check_for_bool_arg(typval_T *args, int i
 }
 
 /*
- * Check for an optional bool argument at 'idx'
+ * Check for an optional bool argument at 'idx'.
+ * Return FAIL if the type is wrong.
  */
     int
 check_for_opt_bool_arg(typval_T *args, int idx)
 {
-    return (args[idx].v_type == VAR_UNKNOWN
-	    || check_for_bool_arg(args, idx) != FAIL);
+    if (args[idx].v_type == VAR_UNKNOWN)
+	return OK;
+    return check_for_bool_arg(args, idx);
 }
 
 /*
diff --git a/src/version.c b/src/version.c
--- a/src/version.c
+++ b/src/version.c
@@ -756,6 +756,8 @@ static char *(features[]) =
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    3436,
+/**/
     3435,
 /**/
     3434,