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

Clean up formatting and warnings #627

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Clean up formatting and warnings #627

wants to merge 5 commits into from

Commits on Dec 27, 2021

  1. Run gofmt -s on all source files.

    @freaksdotcom
    freaksdotcom committed Dec 27, 2021
    Configuration menu
    Copy the full SHA
    a72a676 View commit details
    Browse the repository at this point in the history

Commits on Dec 28, 2021

  1. Fix warnings from go vet

    @freaksdotcom
    freaksdotcom committed Dec 28, 2021
    Configuration menu
    Copy the full SHA
    5c4b0aa View commit details
    Browse the repository at this point in the history

  2. Only build entry_test on platforms with xattr support

    @freaksdotcom
    freaksdotcom committed Dec 28, 2021
    Configuration menu
    Copy the full SHA
    90fbfff View commit details
    Browse the repository at this point in the history

  3. Run go fix on source files

    @freaksdotcom
    freaksdotcom committed Dec 28, 2021
    Configuration menu
    Copy the full SHA
    04c8878 View commit details
    Browse the repository at this point in the history

  4. Change from time.Tick() to time.NewTicker() to avoid leaking memory. 

    There is no way to stop the ticker from `time.Tick()` and therefore no
way for it to be garbage collected.
    @freaksdotcom
    freaksdotcom committed Dec 28, 2021
    Configuration menu
    Copy the full SHA
    5e2ea56 View commit details
    Browse the repository at this point in the history