Clearer function names for SetBitrateTo*()

There is also a MaxBitrate property; once implemented that would become
SetMaxBitrate(). Too confusing to also have SetBitrateMax(). The extra
To hopefully helps disambiguate.

diff --git a/encoder.go b/encoder.go
index ffd9b6cd7e6c58d98c458760e4d628a44acfee27..b7217b6d75b2adeedb81e77c65083ca4d59b5b3f 100644 (file)
--- a/encoder.go
+++ b/encoder.go
@@ -245,7 +245,7 @@ func (enc *Encoder) SetBitrate(bitrate int) error {
 }
 
 // SetBitrateAuto will allow the encoder to automatically set the bitrate
-func (enc *Encoder) SetBitrateAuto() error {
+func (enc *Encoder) SetBitrateToAuto() error {
        res := C.bridge_encoder_set_bitrate(enc.p, C.opus_int32(C.CONST_BITRATE_AUTO))
        if res != C.OPUS_OK {
                return Error(res)
@@ -255,7 +255,7 @@ func (enc *Encoder) SetBitrateAuto() error {
 
 // SetBitrateMax causes the encoder to use as much rate as it can. This can be
 // useful for controlling the rate by adjusting the output buffer size.
-func (enc *Encoder) SetBitrateMax() error {
+func (enc *Encoder) SetBitrateToMax() error {
        res := C.bridge_encoder_set_bitrate(enc.p, C.opus_int32(C.CONST_BITRATE_MAX))
        if res != C.OPUS_OK {
                return Error(res)

diff --git a/encoder_test.go b/encoder_test.go
index b1defe11dea6c461204c91067e811699cd804209..086d7899609534cb6e5d372dae0104109323cdfe 100644 (file)
--- a/encoder_test.go
+++ b/encoder_test.go
@@ -88,7 +88,7 @@ func TestEncoder_SetGetBitrate(t *testing.T) {
        }
 }
 
-func TestEncoder_SetBitrateAuto(t *testing.T) {
+func TestEncoder_SetBitrateToAuto(t *testing.T) {
        enc, err := NewEncoder(8000, 1, AppVoIP)
        if err != nil || enc == nil {
                t.Errorf("Error creating new encoder: %v", err)
@@ -108,7 +108,7 @@ func TestEncoder_SetBitrateAuto(t *testing.T) {
                t.Errorf("Unexpected bitrate. Got %d, but expected %d", br, bitrate)
        }
 
-       err = enc.SetBitrateAuto()
+       err = enc.SetBitrateToAuto()
        if err != nil {
                t.Error("Error setting Auto bitrate:", err)
        }
@@ -124,7 +124,7 @@ func TestEncoder_SetBitrateAuto(t *testing.T) {
        }
 }
 
-func TestEncoder_SetBitrateMax(t *testing.T) {
+func TestEncoder_SetBitrateToMax(t *testing.T) {
        enc, err := NewEncoder(8000, 1, AppVoIP)
        if err != nil || enc == nil {
                t.Errorf("Error creating new encoder: %v", err)
@@ -144,7 +144,7 @@ func TestEncoder_SetBitrateMax(t *testing.T) {
                t.Errorf("Unexpected bitrate. Got %d, but expected %d", br, bitrate)
        }
 
-       err = enc.SetBitrateMax()
+       err = enc.SetBitrateToMax()
        if err != nil {
                t.Error("Error setting Max bitrate:", err)
        }