Hi, On Tue, Jun 14, 2022 at 10:33 AM Christ van Willegen <[email protected]> wrote: > > Op di 14 jun. 2022 14:43 schreef Bram Moolenaar <[email protected]>: >> >> >> Patch 8.2.5089 >> Problem: Some functions return a different value on failure. >> Solution: Initialize the return value earlier. (Yegappan Lakshmanan, >> closes #10568); >> >> + if (rettv_list_alloc(rettv) == FAIL) >> + return; > > ... > >> + if (rettv_list_alloc(rettv) == FAIL) >> + return; >> + > > ... > >> + if (rettv_list_alloc(rettv) != OK) >> + return; >> + > > > Any specific reason why the first compares use == FAIL and the third != OK ? >
No. Both the checks should do the same. I see that both types of checks are used in many places in the source currently. - Yegappan -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For more information, visit http://www.vim.org/maillist.php --- You received this message because you are subscribed to the Google Groups "vim_dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/vim_dev/CAAW7x7kfWmiSZyPAPsUxGzhXZpnzE7x15eyyDYB8rp3S-p4CLA%40mail.gmail.com.
