Support for io.EOF to C.AVERROR_EOF mapping (#73)

* Support for io.EOF to C.AVERROR_EOF mapping

* Requested changes
This commit is contained in:
Daniel Sullivan
2024-09-22 18:15:36 +09:00
committed by GitHub
parent 084f11b493
commit ff9ebd25ed
2 changed files with 45 additions and 27 deletions

View File

@@ -6,6 +6,7 @@ import "C"
import (
"errors"
"fmt"
"io"
"sync"
"unsafe"
)
@@ -241,6 +242,8 @@ func goAstiavIOContextReadFunc(opaque unsafe.Pointer, buf *C.uint8_t, bufSize C.
var e Error
if errors.As(err, &e) {
return C.int(e)
} else if errors.Is(err, io.EOF) {
return C.AVERROR_EOF
}
return C.AVERROR_UNKNOWN
}

View File

@@ -1,6 +1,7 @@
package astiav
import (
"io"
"os"
"path/filepath"
"testing"
@@ -9,34 +10,48 @@ import (
)
func TestIOContext(t *testing.T) {
var seeked bool
rb := []byte("read")
wb := []byte("write")
var written []byte
c, err := AllocIOContext(8, true, func(b []byte) (int, error) {
copy(b, rb)
return len(rb), nil
}, func(offset int64, whence int) (n int64, err error) {
seeked = true
return offset, nil
}, func(b []byte) (int, error) {
written = make([]byte, len(b))
copy(written, b)
return len(b), nil
t.Run("read write seek", func(t *testing.T) {
var seeked bool
rb := []byte("read")
wb := []byte("write")
var written []byte
c, err := AllocIOContext(8, true, func(b []byte) (int, error) {
copy(b, rb)
return len(rb), nil
}, func(offset int64, whence int) (n int64, err error) {
seeked = true
return offset, nil
}, func(b []byte) (int, error) {
written = make([]byte, len(b))
copy(written, b)
return len(b), nil
})
require.NoError(t, err)
defer c.Free()
b := make([]byte, 6)
n, err := c.Read(b)
require.NoError(t, err)
require.Equal(t, 4, n)
require.Equal(t, rb, b[:n])
_, err = c.Seek(2, 0)
require.NoError(t, err)
require.True(t, seeked)
c.Write(wb)
c.Flush()
require.Equal(t, wb, written)
})
t.Run("io.EOF is mapped to AVERROR_EOF when reading", func(t *testing.T) {
c, err := AllocIOContext(8, false, func(b []byte) (int, error) {
return 0, io.EOF
}, nil, nil)
require.NoError(t, err)
defer c.Free()
b := make([]byte, 100)
n, err := c.Read(b)
require.ErrorIs(t, err, ErrEof)
require.Equal(t, 0, n)
})
require.NoError(t, err)
defer c.Free()
b := make([]byte, 6)
n, err := c.Read(b)
require.NoError(t, err)
require.Equal(t, 4, n)
require.Equal(t, rb, b[:n])
_, err = c.Seek(2, 0)
require.NoError(t, err)
require.True(t, seeked)
c.Write(wb)
c.Flush()
require.Equal(t, wb, written)
}
func TestOpenIOContext(t *testing.T) {