| To: vim_dev@googlegroups.com |
| Subject: Patch 7.3.162 |
| Fcc: outbox |
| From: Bram Moolenaar <Bram@moolenaar.net> |
| Mime-Version: 1.0 |
| Content-Type: text/plain; charset=UTF-8 |
| Content-Transfer-Encoding: 8bit |
| |
| |
| Patch 7.3.162 |
| Problem: No error message when assigning to a list with an index out of |
| range. (Yukihiro Nakadaira) |
| Solution: Add the error message. |
| Files: src/eval.c |
| |
| |
| |
| |
| |
| *** 2794,2799 **** |
| --- 2794,2801 ---- |
| { |
| if (lp->ll_range && !lp->ll_empty2) |
| clear_tv(&var2); |
| + if (!quiet) |
| + EMSGN(_(e_listidx), lp->ll_n1); |
| return NULL; |
| } |
| |
| |
| *** 2811,2817 **** |
| --- 2813,2823 ---- |
| { |
| ni = list_find(lp->ll_list, lp->ll_n2); |
| if (ni == NULL) |
| + { |
| + if (!quiet) |
| + EMSGN(_(e_listidx), lp->ll_n2); |
| return NULL; |
| + } |
| lp->ll_n2 = list_idx_of_item(lp->ll_list, ni); |
| } |
| |
| |
| *** 2819,2825 **** |
| --- 2825,2835 ---- |
| if (lp->ll_n1 < 0) |
| lp->ll_n1 = list_idx_of_item(lp->ll_list, lp->ll_li); |
| if (lp->ll_n2 < lp->ll_n1) |
| + { |
| + if (!quiet) |
| + EMSGN(_(e_listidx), lp->ll_n2); |
| return NULL; |
| + } |
| } |
| |
| lp->ll_tv = &lp->ll_li->li_tv; |
| |
| |
| |
| *** 716,717 **** |
| --- 716,719 ---- |
| { /* Add new patch number below this line */ |
| + /**/ |
| + 162, |
| /**/ |
| |
| -- |
| Far back in the mists of ancient time, in the great and glorious days of the |
| former Galactic Empire, life was wild, rich and largely tax free. |
| Mighty starships plied their way between exotic suns, seeking adventure and |
| reward among the furthest reaches of Galactic space. In those days, spirits |
| were brave, the stakes were high, men were real men, women were real women |
| and small furry creatures from Alpha Centauri were real small furry creatures |
| from Alpha Centauri. And all dared to brave unknown terrors, to do mighty |
| deeds, to boldly split infinitives that no man had split before -- and thus |
| was the Empire forged. |
| -- Douglas Adams, "The Hitchhiker's Guide to the Galaxy" |
| |
| /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net \\\ |
| /// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\ |
| \\\ an exciting new programming language -- http://www.Zimbu.org /// |
| \\\ help me help AIDS victims -- http://ICCF-Holland.org /// |