diff options
| author | luukvbaal <luukvbaal@gmail.com> | 2024-12-17 13:12:22 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-12-17 04:12:22 -0800 |
| commit | 6bf2a6fc5bb395b67c88cb26d332f882a106c7ab (patch) | |
| tree | 44259c3b4df40c5af3a4bacec308945f93365d6e /src/nvim/api/tabpage.c | |
| parent | b03e790cddd19b57fa91f4fbfcc30c28f3c173bf (diff) | |
| download | rneovim-6bf2a6fc5bb395b67c88cb26d332f882a106c7ab.tar.gz rneovim-6bf2a6fc5bb395b67c88cb26d332f882a106c7ab.tar.bz2 rneovim-6bf2a6fc5bb395b67c88cb26d332f882a106c7ab.zip | |
refactor(api): always use TRY_WRAP #31600
Problem: Two separate try/end wrappers, that only marginally differ by
restoring a few variables. Wrappers that don't restore
previous state are dangerous to use in "api-fast" functions.
Solution: Remove wrappers that don't restore the previous state.
Always use TRY_WRAP.
Diffstat (limited to 'src/nvim/api/tabpage.c')
| -rw-r--r-- | src/nvim/api/tabpage.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/src/nvim/api/tabpage.c b/src/nvim/api/tabpage.c index 56a3f1cf23..b4d519dc98 100644 --- a/src/nvim/api/tabpage.c +++ b/src/nvim/api/tabpage.c @@ -146,11 +146,9 @@ void nvim_tabpage_set_win(Tabpage tabpage, Window win, Error *err) } if (tp == curtab) { - try_start(); - win_goto(wp); - if (!try_end(err) && curwin != wp) { - api_set_error(err, kErrorTypeException, "Failed to switch to window %d", win); - } + TRY_WRAP(err, { + win_goto(wp); + }); } else if (tp->tp_curwin != wp) { tp->tp_prevwin = tp->tp_curwin; tp->tp_curwin = wp; |