Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for pausing/resuming the recording #210

Merged
merged 1 commit into from
Dec 27, 2022
Merged

Conversation

chenxiaolong
Copy link
Owner

This commit implements basic pause/resume functionality via a notification action in BCR's persistent notification. Pausing is implemented by keeping the AudioRecord active, but discarding the data instead of passing it to the encoder. Pausing/resuming does not happen immediately, but after the processing of one buffer (which should be under 50ms on most devices). It's fast enough to be imperceptible.

Issue: #198

@chenxiaolong chenxiaolong self-assigned this Dec 27, 2022
This commit implements basic pause/resume functionality via a
notification action in BCR's persistent notification. Pausing is
implemented by keeping the AudioRecord active, but discarding the data
instead of passing it to the encoder. Pausing/resuming does not happen
immediately, but after the processing of one buffer (which should be
under 50ms on most devices). It's fast enough to be imperceptible.

Issue: #198

Signed-off-by: Andrew Gunnerson <chillermillerlong@hotmail.com>
@chenxiaolong chenxiaolong merged commit c8fbcfc into master Dec 27, 2022
@chenxiaolong chenxiaolong deleted the pause branch December 27, 2022 23:52
chenxiaolong added a commit that referenced this pull request Dec 27, 2022
Signed-off-by: Andrew Gunnerson <chillermillerlong@hotmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant