audio: Use bigger buffer size for mobile phones

This commit is contained in:
Hajime Hoshi 2017-12-31 01:28:08 +09:00
parent de1a22d9f3
commit df62b16270
3 changed files with 47 additions and 4 deletions

View File

@ -230,10 +230,7 @@ func (c *Context) loop() {
// e.g. a variable for JVM on Android might not be set.
<-initCh
// This is a heuristic decision of audio buffer size.
// On most environments, 4096 [bytes] is enough but there are some known environment that is too short (e.g. Windows on Parallels).
bufferSize := 8192
p, err := oto.NewPlayer(c.sampleRate, channelNum, bytesPerSample, bufferSize)
p, err := oto.NewPlayer(c.sampleRate, channelNum, bytesPerSample, bufferSize())
if err != nil {
audiobinding.SetError(err)
return

25
audio/buffersize.go Normal file
View File

@ -0,0 +1,25 @@
// Copyright 2017 The Ebiten Authors
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
// +build !android
// +build !ios
package audio
// bufferSize returns a heuristic decision of audio buffer size.
func bufferSize() int {
// On most desktop environments, 4096 [bytes] is enough
// but there are some known environment that is too short (e.g. Windows on Parallels).
return 8192
}

View File

@ -0,0 +1,21 @@
// Copyright 2017 The Ebiten Authors
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
// +build android ios
package audio
func bufferSize() int {
return 12288
}