comparison src/eval.c @ 25153:610e15e8778d v8.2.3113

patch 8.2.3113: no error when for loop variable shadows script variable Commit: https://github.com/vim/vim/commit/442b29c968eff1c8fec1a96418aa47be9cba62ea Author: Bram Moolenaar <Bram@vim.org> Date: Mon Jul 5 22:23:00 2021 +0200 patch 8.2.3113: no error when for loop variable shadows script variable Problem: No error when for loop variable shadows script variable. Solution: Check for the error. (closes https://github.com/vim/vim/issues/8512)
author Bram Moolenaar <Bram@vim.org>
date Mon, 05 Jul 2021 22:30:03 +0200
parents 18b31f0a4bb5
children e495f40e4b07
comparison
equal deleted inserted replaced
25152:7da8d1cf23f3 25153:610e15e8778d
1775 next_for_item(void *fi_void, char_u *arg) 1775 next_for_item(void *fi_void, char_u *arg)
1776 { 1776 {
1777 forinfo_T *fi = (forinfo_T *)fi_void; 1777 forinfo_T *fi = (forinfo_T *)fi_void;
1778 int result; 1778 int result;
1779 int flag = ASSIGN_FOR_LOOP | (in_vim9script() 1779 int flag = ASSIGN_FOR_LOOP | (in_vim9script()
1780 ? (ASSIGN_FINAL | ASSIGN_DECL | ASSIGN_NO_MEMBER_TYPE) 1780 ? (ASSIGN_FINAL
1781 // first round: error if variable exists
1782 | (fi->fi_bi == 0 ? 0 : ASSIGN_DECL)
1783 | ASSIGN_NO_MEMBER_TYPE)
1781 : 0); 1784 : 0);
1782 listitem_T *item; 1785 listitem_T *item;
1783 1786
1784 if (fi->fi_blob != NULL) 1787 if (fi->fi_blob != NULL)
1785 { 1788 {
1805 return FALSE; 1808 return FALSE;
1806 tv.v_type = VAR_STRING; 1809 tv.v_type = VAR_STRING;
1807 tv.v_lock = VAR_FIXED; 1810 tv.v_lock = VAR_FIXED;
1808 tv.vval.v_string = vim_strnsave(fi->fi_string + fi->fi_byte_idx, len); 1811 tv.vval.v_string = vim_strnsave(fi->fi_string + fi->fi_byte_idx, len);
1809 fi->fi_byte_idx += len; 1812 fi->fi_byte_idx += len;
1813 ++fi->fi_bi;
1810 result = ex_let_vars(arg, &tv, TRUE, fi->fi_semicolon, 1814 result = ex_let_vars(arg, &tv, TRUE, fi->fi_semicolon,
1811 fi->fi_varcount, flag, NULL) == OK; 1815 fi->fi_varcount, flag, NULL) == OK;
1812 vim_free(tv.vval.v_string); 1816 vim_free(tv.vval.v_string);
1813 return result; 1817 return result;
1814 } 1818 }
1817 if (item == NULL) 1821 if (item == NULL)
1818 result = FALSE; 1822 result = FALSE;
1819 else 1823 else
1820 { 1824 {
1821 fi->fi_lw.lw_item = item->li_next; 1825 fi->fi_lw.lw_item = item->li_next;
1826 ++fi->fi_bi;
1822 result = (ex_let_vars(arg, &item->li_tv, TRUE, fi->fi_semicolon, 1827 result = (ex_let_vars(arg, &item->li_tv, TRUE, fi->fi_semicolon,
1823 fi->fi_varcount, flag, NULL) == OK); 1828 fi->fi_varcount, flag, NULL) == OK);
1824 } 1829 }
1825 return result; 1830 return result;
1826 } 1831 }