rm needless lock on responses

rm duplicate mark check in tests

epsilon_greedy.go

@@ -13,10 +13,8 @@ type epsilonHostPoolResponse struct {
 }
 
 func (r *epsilonHostPoolResponse) Mark(err error) {
-	r.Do(func() {
-		r.ended = time.Now()
-		doMark(err, r)
-	})
+	r.ended = time.Now()
+	r.standardHostPoolResponse.Mark(err)
 
 }
 

hostpool.go

@@ -28,7 +28,6 @@ type HostPoolResponse interface {
 
 type standardHostPoolResponse struct {
 	host string
-	sync.Once
 	pool HostPool
 }
 
@@ -94,12 +93,6 @@ func (r *standardHostPoolResponse) hostPool() HostPool {
 }
 
 func (r *standardHostPoolResponse) Mark(err error) {
-	r.Do(func() {
-		doMark(err, r)
-	})
-}
-
-func doMark(err error, r HostPoolResponse) {
 	if err == nil {
 		r.hostPool().markSuccess(r)
 	} else {

hostpool_test.go

@@ -32,8 +32,6 @@ func TestHostPool(t *testing.T) {
 	respC.Mark(nil)
 	// get again, and verify that it's still c
 	assert.Equal(t, p.Get().Host(), "c")
-	// now try to mark b as success; should fail because already marked
-	respB.Mark(nil)
 	assert.Equal(t, p.Get().Host(), "c") // would be b if it were not dead
 	// now restore a
 	respA = &standardHostPoolResponse{host: "a", pool: p}