Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove redundant null check #8876

Merged
merged 1 commit into from Feb 24, 2021

Conversation

drewnoakes
Copy link
Member

Proposed changes

Removes a redundant null check.

All callers pass a non-null delegate. This code is now annotated, so future code will not pass null either.

Test methodology

  • Manual testing

✒️ I contribute this code under The Developer Certificate of Origin.

All callers pass a non-null delegate. This code is now annotated, so future code will not pass null either.
@ghost ghost assigned drewnoakes Feb 24, 2021
@codecov
Copy link

codecov bot commented Feb 24, 2021

Codecov Report

Merging #8876 (b03867c) into master (24ed8db) will increase coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #8876      +/-   ##
==========================================
+ Coverage   56.01%   56.04%   +0.02%     
==========================================
  Files         922      922              
  Lines       65945    65944       -1     
  Branches    12071    12070       -1     
==========================================
+ Hits        36942    36956      +14     
+ Misses      25989    25984       -5     
+ Partials     3014     3004      -10     
Flag Coverage Δ
production 43.27% <0.00%> (+0.03%) ⬆️
tests 94.83% <ø> (+0.01%) ⬆️

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

@drewnoakes drewnoakes merged commit b6905f1 into gitextensions:master Feb 24, 2021
@ghost ghost added this to the 3.6 milestone Feb 24, 2021
@drewnoakes drewnoakes deleted the remove-null-check branch February 24, 2021 11:14
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.

None yet

2 participants