diff options
author | zeertzjq <zeertzjq@outlook.com> | 2023-01-19 09:51:59 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-01-19 09:51:59 +0800 |
commit | 45dc0f3454d826c42efdcd841953b5e7e613bc5e (patch) | |
tree | 3a41b3930feb90624040f5269146fceaab295366 /src/nvim/regexp.c | |
parent | f8669e8a181abad98229d97439a9cafe8c32fc06 (diff) | |
download | rneovim-45dc0f3454d826c42efdcd841953b5e7e613bc5e.tar.gz rneovim-45dc0f3454d826c42efdcd841953b5e7e613bc5e.tar.bz2 rneovim-45dc0f3454d826c42efdcd841953b5e7e613bc5e.zip |
vim-patch:9.0.1221: code is indented more than necessary (#21891)
Problem: Code is indented more than necessary.
Solution: Use an early return where it makes sense. (Yegappan Lakshmanan,
closes vim/vim#11833)
https://github.com/vim/vim/commit/f97a295ccaa9803367f3714cdefce4e2283c771d
Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
Diffstat (limited to 'src/nvim/regexp.c')
-rw-r--r-- | src/nvim/regexp.c | 44 |
1 files changed, 24 insertions, 20 deletions
diff --git a/src/nvim/regexp.c b/src/nvim/regexp.c index ab8e974b29..c2a7dd1011 100644 --- a/src/nvim/regexp.c +++ b/src/nvim/regexp.c @@ -1184,32 +1184,36 @@ static int prog_magic_wrong(void) // used (to increase speed). static void cleanup_subexpr(void) { - if (rex.need_clear_subexpr) { - if (REG_MULTI) { - // Use 0xff to set lnum to -1 - memset(rex.reg_startpos, 0xff, sizeof(lpos_T) * NSUBEXP); - memset(rex.reg_endpos, 0xff, sizeof(lpos_T) * NSUBEXP); - } else { - memset(rex.reg_startp, 0, sizeof(char *) * NSUBEXP); - memset(rex.reg_endp, 0, sizeof(char *) * NSUBEXP); - } - rex.need_clear_subexpr = false; + if (!rex.need_clear_subexpr) { + return; + } + + if (REG_MULTI) { + // Use 0xff to set lnum to -1 + memset(rex.reg_startpos, 0xff, sizeof(lpos_T) * NSUBEXP); + memset(rex.reg_endpos, 0xff, sizeof(lpos_T) * NSUBEXP); + } else { + memset(rex.reg_startp, 0, sizeof(char *) * NSUBEXP); + memset(rex.reg_endp, 0, sizeof(char *) * NSUBEXP); } + rex.need_clear_subexpr = false; } static void cleanup_zsubexpr(void) { - if (rex.need_clear_zsubexpr) { - if (REG_MULTI) { - // Use 0xff to set lnum to -1 - memset(reg_startzpos, 0xff, sizeof(lpos_T) * NSUBEXP); - memset(reg_endzpos, 0xff, sizeof(lpos_T) * NSUBEXP); - } else { - memset(reg_startzp, 0, sizeof(char *) * NSUBEXP); - memset(reg_endzp, 0, sizeof(char *) * NSUBEXP); - } - rex.need_clear_zsubexpr = false; + if (!rex.need_clear_zsubexpr) { + return; + } + + if (REG_MULTI) { + // Use 0xff to set lnum to -1 + memset(reg_startzpos, 0xff, sizeof(lpos_T) * NSUBEXP); + memset(reg_endzpos, 0xff, sizeof(lpos_T) * NSUBEXP); + } else { + memset(reg_startzp, 0, sizeof(char *) * NSUBEXP); + memset(reg_endzp, 0, sizeof(char *) * NSUBEXP); } + rex.need_clear_zsubexpr = false; } // Advance rex.lnum, rex.line and rex.input to the next line. |