status-im
/
mobile
mirror of
2
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

audio/al: don't define MakeContextCurrent on the pointer 

It's a common requirement to call MakeContextCurrent with a nil value
before destroying a context. Nil value as a reciever is not as user-
friendly as a standalone MakeContextCurrent.

Change-Id: I537ec0330657ef134f97a3c3b0d32793012eee86
Reviewed-on: https://go-review.googlesource.com/8500
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
This commit is contained in:
Burcu Dogan 2015-04-06 15:57:04 -04:00
parent c2bf90dc60
commit 6544820032
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
audio/al/alc.go
View File

@ -52,7 +52,7 @@ func (d *Device) CreateContext(attrs []int32) *Context {
// MakeContextCurrent makes a context current. The context available
// process-wide, you don't need to lock the current OS thread to
// access the current context.
func (c *Context) MakeContextCurrent() bool {
func MakeContextCurrent(c *Context) bool {
return alcMakeContextCurrent(c.ptr)
}

2
audio/audio.go
View File

@ -120,7 +120,7 @@ func NewPlayer(src io.ReadSeeker, format Format, samplesPerSecond int64) (*Playe
if device.d == nil {
device.d = al.Open("")
c := device.d.CreateContext(nil)
if !c.MakeContextCurrent() {
if !al.MakeContextCurrent(c) {
return nil, fmt.Errorf("audio: cannot initiate a new player")
}
}