Skip to content

Use early return to simplify code in a few files#11833

Closed
yegappan wants to merge 1 commit intovim:masterfrom
yegappan:lsp
Closed

Use early return to simplify code in a few files#11833
yegappan wants to merge 1 commit intovim:masterfrom
yegappan:lsp

Conversation

@yegappan
Copy link
Copy Markdown
Member

No description provided.

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 17, 2023

Codecov Report

Merging #11833 (a17f1aa) into master (f7d1c6e) will increase coverage by 0.01%.
The diff coverage is 78.34%.

@@            Coverage Diff             @@
##           master   #11833      +/-   ##
==========================================
+ Coverage   81.86%   81.87%   +0.01%     
==========================================
  Files         164      164              
  Lines      193022   193079      +57     
  Branches    43765    43765              
==========================================
+ Hits       158012   158079      +67     
+ Misses      22186    22177       -9     
+ Partials    12824    12823       -1     
Flag Coverage Δ
huge-clang-none 82.57% <78.16%> (+<0.01%) ⬆️
huge-gcc-none 53.71% <25.55%> (-0.02%) ⬇️
huge-gcc-testgui 52.18% <22.77%> (?)
huge-gcc-unittests 0.29% <0.00%> (-0.01%) ⬇️
linux 82.32% <78.34%> (+0.07%) ⬆️
mingw-x64-HUGE 76.47% <63.90%> (+<0.01%) ⬆️
mingw-x86-HUGE 76.92% <63.90%> (-0.03%) ⬇️
windows 78.04% <63.90%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/regexp_bt.c 85.98% <62.50%> (+0.01%) ⬆️
src/regexp_nfa.c 89.63% <73.52%> (+0.01%) ⬆️
src/register.c 84.87% <77.14%> (-0.10%) ⬇️
src/popupwin.c 86.27% <77.27%> (-0.50%) ⬇️
src/profiler.c 83.48% <81.25%> (-0.60%) ⬇️
src/quickfix.c 91.16% <82.47%> (-0.06%) ⬇️
src/regexp.c 86.17% <86.36%> (+0.03%) ⬆️
src/popupmenu.c 79.73% <88.23%> (-0.08%) ⬇️
src/beval.c 63.63% <0.00%> (-2.10%) ⬇️
... and 22 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@brammool brammool closed this in f97a295 Jan 18, 2023
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Jan 19, 2023
Problem:    Code is indented more than necessary.
Solution:   Use an early return where it makes sense. (Yegappan Lakshmanan,
            closes vim/vim#11833)

vim/vim@f97a295

Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
zeertzjq added a commit to neovim/neovim that referenced this pull request Jan 19, 2023
Problem:    Code is indented more than necessary.
Solution:   Use an early return where it makes sense. (Yegappan Lakshmanan,
            closes vim/vim#11833)

vim/vim@f97a295

Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
yesean pushed a commit to yesean/neovim that referenced this pull request Mar 25, 2023
Problem:    Code is indented more than necessary.
Solution:   Use an early return where it makes sense. (Yegappan Lakshmanan,
            closes vim/vim#11833)

vim/vim@f97a295

Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant