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

lib: fix errors found by vet #136

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

lib: fix errors found by vet #136

wants to merge 1 commit into from

Commits on Apr 19, 2018

  1. lib: fix errors found by vet 

    Vet found the following errors:
	lib/proxy.go:87: result of fmt.Errorf call not used
	lib/proxy.go:97: result of fmt.Errorf call not used
	lib/proxy.go:105: result of fmt.Errorf call not used

These are pointless calls because fmt.Errorf just creates an error
value; it is then discarded without reporting the error to the user.

This commit changes the fmt.Errorf calls to log.Printf calls so
the error is logged.
    @functionary
    functionary committed Apr 19, 2018
    Configuration menu
    Copy the full SHA
    8c40832 View commit details
    Browse the repository at this point in the history