Comment out unreachable TODO code

Makefile

@@ -98,9 +98,9 @@ lint:
 	@echo "Running golint"
 	-golint ./... | $(FILTER) | tee lint.log
 	@echo "Running go vet"
-	-go tool vet $(PKGS) 2>&1 | tee -a lint.log
+	-go tool vet $(PKGS) 2>&1 | grep -v Godeps | tee -a lint.log
 	@echo "Checking gofmt"
-	-[ $(GO_VERSION) != "go1.5" ] || gofmt -d . | tee -a lint.log
+	-[ $(GO_VERSION) != "go1.5" ] || gofmt -d `go list -f {{.Dir}} ./...` | tee -a lint.log
 	@[ ! -s lint.log ]
 
 thrift_example: thrift_gen

frame_test.go

@@ -140,10 +140,14 @@ func TestServiceCallReq(t *testing.T) {
 	// reqResWriter instead of callReq. We should instead handle that in
 	// callReq, which will allow tests to be sane.
 	return
+	/* go vet doesn't like unreachable code...
+
 	frame := NewFrame(MaxFramePayloadSize)
 	err := frame.write(&callReq{Service: "udr"})
 	require.NoError(t, err, "Error writing message to frame.")
 	assert.Equal(t, "udr", frame.Service(), "Failed to read service name from frame.")
+
+	*/
 }
 
 func TestServiceCallReqTerrible(t *testing.T) {