# HG changeset patch
# User Bram Moolenaar <Bram@vim.org>
# Date 1580249703 -3600
# Node ID 173b99509038fa5cd6fec7739baa64a759d50a97
# Parent  7bf13c862e201e1ce319683e8be43d2e94d5e6fd
patch 8.2.0170: Coverity warning for ignoring return value

Commit: https://github.com/vim/vim/commit/a6d536829a2c3151f3d0faa0ecdc7b8230fb11ec
Author: Bram Moolenaar <Bram@vim.org>
Date:   Tue Jan 28 23:04:06 2020 +0100

    patch 8.2.0170: Coverity warning for ignoring return value

    Problem:    Coverity warning for ignoring return value.
    Solution:   Check the return value and return if failed.

diff --git a/src/version.c b/src/version.c
--- a/src/version.c
+++ b/src/version.c
@@ -743,6 +743,8 @@ static char *(features[]) =
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    170,
+/**/
     169,
 /**/
     168,
diff --git a/src/vim9compile.c b/src/vim9compile.c
--- a/src/vim9compile.c
+++ b/src/vim9compile.c
@@ -2917,7 +2917,8 @@ compile_expr1(char_u **arg,  cctx_T *cct
 
 	// evaluate the second expression; any type is accepted
 	*arg = skipwhite(p + 1);
-	compile_expr1(arg, cctx);
+	if (compile_expr1(arg, cctx) == FAIL)
+	    return FAIL;
 
 	// remember the type and drop it
 	--stack->ga_len;
@@ -2942,7 +2943,8 @@ compile_expr1(char_u **arg,  cctx_T *cct
 
 	// evaluate the third expression
 	*arg = skipwhite(p + 1);
-	compile_expr1(arg, cctx);
+	if (compile_expr1(arg, cctx) == FAIL)
+	    return FAIL;
 
 	// If the types differ, the result has a more generic type.
 	type2 = ((type_T **)stack->ga_data)[stack->ga_len - 1];
@@ -3265,6 +3267,7 @@ compile_assignment(char_u *arg, exarg_T 
 	if (*op != '=')
 	{
 	    if (option)
+		// TODO: check the option exists
 		generate_LOAD(cctx, ISN_LOADOPT, 0, name + 1, type);
 	    else if (global)
 		generate_LOAD(cctx, ISN_LOADG, 0, name + 2, type);