google / google/oboe

Update the full guide to mention that `getFramesRead|Written` are available when using OpenSLES

Open
#1,444 0 comments 0 reactions 1 assignee Claimed by @mmoczkowski View on GitHub
Dominant language
C++
Stars
4.1k
Forks
639
Avg merge
2d 23h
Merged PRs (30d)
8

Description

### Discussed in https://github.com/google/oboe/discussions/1434

Originally posted by **YouheiLee** November 8, 2021
At the last of doc "Full Guide To Oboe" said that:
getFramesRead(), getFramesWritten(), getTimestamp() will return Result::ErrorUnimplemented for OpenSLES streams, But I found that in AudioStream.cpp, there has a implementation of getFramesRead():

```
int64_t AudioStream::getFramesRead() {
updateFramesRead();
return mFramesRead;
}
```

and in AudioOutputStreamOpenSLES.cpp, function updateFramesRead are also be implemented.
```
void AudioOutputStreamOpenSLES::updateFramesRead() {
if (usingFIFO()) {
AudioStreamBuffered::updateFramesRead();
} else {
mFramesRead = getFramesProcessedByServer();
}
}
```

does it means those three functions(getFramesRead, getFramesWritten, getTimestamp) is also available in OpenSLES streams before OS version android O?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.