Skip to content
This repository has been archived by the owner on Aug 18, 2023. It is now read-only.

fieldalignment: struct with X pointer bytes could be Y (govet) #127

Closed
atc0005 opened this issue Apr 8, 2021 · 0 comments · Fixed by #129
Closed

fieldalignment: struct with X pointer bytes could be Y (govet) #127

atc0005 opened this issue Apr 8, 2021 · 0 comments · Fixed by #129
Assignees
Labels
bug Something isn't working linting
Milestone

Comments

@atc0005
Copy link
Owner

atc0005 commented Apr 8, 2021

Linting issues exposed by the changes in atc0005/go-ci#287 and atc0005/go-ci#288:

config/config.go:251:13: fieldalignment: struct with 112 pointer bytes could be 72 (govet)
type Config struct {
            ^
routes/routes.go:18:12: fieldalignment: struct with 80 pointer bytes could be 64 (govet)
type Route struct {
           ^
cmd/bounce/decode-json-body.go:24:23: fieldalignment: struct with 16 pointer bytes could be 8 (govet)
type malformedRequest struct {
                      ^
cmd/bounce/notify.go:16:19: fieldalignment: struct with 32 pointer bytes could be 24 (govet)
type NotifyResult struct {
                  ^
cmd/bounce/notify.go:34:18: fieldalignment: struct with 32 pointer bytes could be 24 (govet)
type NotifyQueue struct {
                 ^
@atc0005 atc0005 added bug Something isn't working linting labels Apr 8, 2021
@atc0005 atc0005 added this to the Next Release milestone Apr 8, 2021
@atc0005 atc0005 self-assigned this Apr 8, 2021
atc0005 added a commit that referenced this issue Apr 8, 2021
Change field order of `Config` struct to reduce allocation
requirements and pass `fieldalignment` linting check.

refs GH-127
atc0005 added a commit that referenced this issue Apr 8, 2021
Change field order of `Config` struct to reduce allocation
requirements and pass `fieldalignment` linting check.

refs GH-127
atc0005 added a commit that referenced this issue Apr 8, 2021
Change field order of `Config` struct to reduce allocation
requirements and pass `fieldalignment` linting check.

refs GH-127
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working linting
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant