GODRIVER-3455 Remove unused EncodeValue methods

Author: prestonvasquez

bson/array_codec.go

@@ -15,16 +15,6 @@ import (
 // arrayCodec is the Codec used for bsoncore.Array values.
 type arrayCodec struct{}
 
-// EncodeValue is the ValueEncoder for bsoncore.Array values.
-func (ac *arrayCodec) EncodeValue(_ EncodeContext, vw ValueWriter, val any) error {
-	arr, ok := val.(bsoncore.Array)
-	if !ok {
-		return ValueEncoderError{Name: "CoreArrayEncodeValue", Types: []reflect.Type{tCoreArray}, Received: reflect.ValueOf(val)}
-	}
-
-	return copyArrayFromBytes(vw, arr)
-}
-
 // DecodeValue is the ValueDecoder for bsoncore.Array values.
 func (ac *arrayCodec) DecodeValue(_ DecodeContext, vr ValueReader, val reflect.Value) error {
 	if !val.CanSet() || val.Type() != tCoreArray {

bson/default_value_encoders_test.go

@@ -1791,7 +1791,7 @@ func TestDefaultValueEncoders(t *testing.T) {
 	t.Run("EmptyInterfaceEncodeValue/nil", func(t *testing.T) {
 		val := reflect.New(tEmpty).Elem()
 		llvrw := new(valueReaderWriter)
-		err := (&emptyInterfaceCodec{}).EncodeValue(EncodeContext{Registry: newTestRegistry()}, llvrw, val)
+		err := emptyInterfaceValue(EncodeContext{Registry: newTestRegistry()}, llvrw, val)
 		noerr(t, err)
 		if llvrw.invoked != writeNull {
 			t.Errorf("Incorrect method called. got %v; want %v", llvrw.invoked, writeNull)
@@ -1802,10 +1802,10 @@ func TestDefaultValueEncoders(t *testing.T) {
 		val := reflect.New(tEmpty).Elem()
 		val.Set(reflect.ValueOf(int64(1234567890)))
 		llvrw := new(valueReaderWriter)
-		got := (&emptyInterfaceCodec{}).EncodeValue(EncodeContext{Registry: newTestRegistry()}, llvrw, val)
+		err := emptyInterfaceValue(EncodeContext{Registry: newTestRegistry()}, llvrw, val)
 		want := errNoEncoder{Type: tInt64}
-		if !assert.CompareErrors(got, want) {
-			t.Errorf("Did not receive expected error. got %v; want %v", got, want)
+		if !assert.CompareErrors(err, want) {
+			t.Errorf("Did not receive expected error. got %v; want %v", err, want)
 		}
 	})
 }

bson/empty_interface_codec.go

@@ -17,28 +17,6 @@ type emptyInterfaceCodec struct {
 	decodeBinaryAsSlice bool
 }
 
-// Assert that emptyInterfaceCodec satisfies the typeDecoder interface, which allows it
-// to be used by collection type decoders (e.g. map, slice, etc) to set individual values in a
-// collection.
-var _ typeDecoder = &emptyInterfaceCodec{}
-
-// EncodeValue is the ValueEncoderFunc for interface{}.
-func (eic *emptyInterfaceCodec) EncodeValue(ec EncodeContext, vw ValueWriter, val reflect.Value) error {
-	if !val.IsValid() || val.Type() != tEmpty {
-		return ValueEncoderError{Name: "EmptyInterfaceEncodeValue", Types: []reflect.Type{tEmpty}, Received: val}
-	}
-
-	if val.IsNil() {
-		return vw.WriteNull()
-	}
-	encoder, err := ec.LookupEncoder(val.Elem().Type())
-	if err != nil {
-		return err
-	}
-
-	return encoder.EncodeValue(ec, vw, val.Elem())
-}
-
 func (eic *emptyInterfaceCodec) getEmptyInterfaceDecodeType(dc DecodeContext, valueType Type) (reflect.Type, error) {
 	isDocument := valueType == Type(0) || valueType == TypeEmbeddedDocument
 	if isDocument {

bson/time_codec.go

@@ -97,14 +97,3 @@ func (tc *timeCodec) DecodeValue(dc DecodeContext, vr ValueReader, val reflect.V
 	val.Set(elem)
 	return nil
 }
-
-// EncodeValue is the ValueEncoderFunc for time.TIme.
-func (tc *timeCodec) EncodeValue(_ EncodeContext, vw ValueWriter, val any) error {
-	timeVal, ok := val.(time.Time)
-	if !ok {
-		return ValueEncoderError{Name: "TimeEncodeValue", Types: []reflect.Type{tTime}, Received: reflect.ValueOf(val)}
-	}
-
-	dt := NewDateTimeFromTime(timeVal)
-	return vw.WriteDateTime(int64(dt))
-}