From fa18d7229f02e532de1f3413449174f68bbb894d Mon Sep 17 00:00:00 2001
From: Michael McLoughlin <mmcloughlin@gmail.com>
Date: Wed, 5 Dec 2018 00:27:42 -0800
Subject: [PATCH] address some lint

---
 ast.go           |  5 ++---
 pass/alloc.go    |  4 +++-
 printer.go       |  7 +++----
 reg/types.go     | 16 ++++------------
 script/bootstrap |  5 +++++
 5 files changed, 17 insertions(+), 20 deletions(-)
 create mode 100755 script/bootstrap

diff --git a/ast.go b/ast.go
index 5f2b9ef..84fa26d 100644
--- a/ast.go
+++ b/ast.go
@@ -109,9 +109,8 @@ func NewFile() *File {
 
 // Function represents an assembly function.
 type Function struct {
-	name   string
-	params []Parameter
-	Nodes  []Node
+	name  string
+	Nodes []Node
 
 	// LabelTarget maps from label name to the following instruction.
 	LabelTarget map[Label]*Instruction
diff --git a/pass/alloc.go b/pass/alloc.go
index 5dbf0ec..ab88b9f 100644
--- a/pass/alloc.go
+++ b/pass/alloc.go
@@ -53,7 +53,9 @@ func (a *Allocator) AddInterference(x, y reg.Register) {
 
 func (a *Allocator) Allocate() (reg.Allocation, error) {
 	for a.remaining() > 0 {
-		a.update()
+		if err := a.update(); err != nil {
+			return nil, err
+		}
 
 		v := a.mostrestricted()
 		if err := a.alloc(v); err != nil {
diff --git a/printer.go b/printer.go
index ad9c2e4..1eaf6ae 100644
--- a/printer.go
+++ b/printer.go
@@ -16,10 +16,9 @@ type Printer interface {
 }
 
 type GoPrinter struct {
-	w         io.Writer
-	by        string // generated by
-	copyright []string
-	err       error
+	w   io.Writer
+	by  string // generated by
+	err error
 }
 
 func NewGoPrinter(w io.Writer) *GoPrinter {
diff --git a/reg/types.go b/reg/types.go
index eb16cc5..7a227c7 100644
--- a/reg/types.go
+++ b/reg/types.go
@@ -43,11 +43,7 @@ func (f *Family) Virtual(id VID, s Size) Virtual {
 
 // Registers returns the registers in this family.
 func (f *Family) Registers() []Physical {
-	rs := make([]Physical, 0, len(f.registers))
-	for _, r := range f.registers {
-		rs = append(rs, r)
-	}
-	return rs
+	return append([]Physical(nil), f.registers...)
 }
 
 // Set returns the set of registers in the family.
@@ -59,10 +55,6 @@ func (f *Family) Set() Set {
 	return s
 }
 
-type private interface {
-	private()
-}
-
 type (
 	ID  uint64
 	VID uint16
@@ -74,7 +66,7 @@ type Register interface {
 	Kind() Kind
 	Bytes() uint
 	Asm() string
-	private
+	register()
 }
 
 type Virtual interface {
@@ -116,7 +108,7 @@ func (v virtual) Asm() string {
 	return fmt.Sprintf("<virtual:%v:%v:%v>", v.id, v.Kind(), v.Bytes())
 }
 
-func (v virtual) private() {}
+func (v virtual) register() {}
 
 type Physical interface {
 	PhysicalID() PID
@@ -143,7 +135,7 @@ func (r register) PhysicalID() PID { return r.id }
 func (r register) ID() ID          { return (ID(r.Mask()) << 16) | ID(r.id) }
 func (r register) Kind() Kind      { return r.kind }
 func (r register) Asm() string     { return r.name }
-func (r register) private()        {}
+func (r register) register()       {}
 
 type Spec uint16
 
diff --git a/script/bootstrap b/script/bootstrap
new file mode 100755
index 0000000..07bff55
--- /dev/null
+++ b/script/bootstrap
@@ -0,0 +1,5 @@
+#!/bin/bash -ex
+
+# Install golangci-lint
+golangci_lint_version='v1.12.3'
+curl -sfL https://install.goreleaser.com/github.com/golangci/golangci-lint.sh | sh -s -- -b $GOPATH/bin ${golangci_lint_version}