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.

Sign up for GitHub

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

Jump to bottom

client: remove redundant/repeated code #8182

Merged
merged 1 commit into from
Dec 17, 2020
Merged

client: remove redundant/repeated code #8182

merged 1 commit into from
Dec 17, 2020

Commits on Dec 17, 2020

  1. client: remove redundant/repeated code 

    Follows up PR #4876 which removed the return on non-nil errors
to always return a non-nil REST response regardless of error.
The checks performed seem benign and the return result is the
same.
This change uses the reverse conditional refactoring method
to gut out nesting and many conditions.
Noticed while hunting through types.ParseABCILogs.

Fixes #8181
    @odeke-em
    odeke-em committed Dec 17, 2020
    Configuration menu
    Copy the full SHA
    6d1c65b View commit details
    Browse the repository at this point in the history