Add ineffassign to the pre-commit hooks (#139)

* Add ineffassign to the pre-comiit hooks

* Fix travis.yml

* Only apply ineffassign to the /src directory

.travis.yml

@@ -21,6 +21,7 @@ install:
  - go get github.com/golang/lint/golint
  - go get github.com/fzipp/gocyclo
  - go get github.com/client9/misspell/...
+ - go get github.com/gordonklaus/ineffassign
 
 # Generate a self-signed X.509 certificate for TLS.
 before_script:

hooks/pre-commit

@@ -18,6 +18,7 @@ then
     exit 1
 fi
 
+ineffassign ./src/
 go tool vet --all --shadow ./src
 gocyclo -over 12 src/
 gb test

src/github.com/matrix-org/dendrite/common/test/client.go

@@ -136,12 +136,8 @@ func (r *Request) Run(label string, timeout time.Duration, serverCmdChan chan er
 	// - the server to exit with an error (error sent on serverCmdChan)
 	// - our test timeout to expire
 	// We don't need to clean up since the main() function handles that in the event we panic
-	var testPassed bool
 	select {
 	case <-time.After(timeout):
-		if testPassed {
-			break
-		}
 		fmt.Printf("==TESTING== %v TIMEOUT\n", label)
 		if reqErr := r.LastErr.Get(); reqErr != nil {
 			fmt.Println("Last /sync request error:")
@@ -157,7 +153,6 @@ func (r *Request) Run(label string, timeout time.Duration, serverCmdChan chan er
 			panic(err)
 		}
 	case <-done:
-		testPassed = true
 		fmt.Printf("==TESTING== %v PASSED\n", label)
 	}
 }