-
Notifications
You must be signed in to change notification settings - Fork 109
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
Live waveform #1659
Live waveform #1659
Conversation
- Fix live waveform hight - Fix concurrent modification exception when sharing levels - Tidy up width of live waveform
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
…/element-x-android into langleyd/live_waveform
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #1659 +/- ##
===========================================
- Coverage 63.15% 63.08% -0.08%
===========================================
Files 1228 1223 -5
Lines 31680 31450 -230
Branches 6524 6453 -71
===========================================
- Hits 20009 19841 -168
+ Misses 8672 8634 -38
+ Partials 2999 2975 -24
☔ View full report in Codecov by Sentry. |
...snapshots/images/ui_S_t[l.textcomposer_null_TextComposerVoice-D-4_4_null,NEXUS_5,1.0,en].png
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGMT, just some nits
...t/android/features/messages/impl/voicemessages/composer/VoiceMessageComposerStateProvider.kt
Outdated
Show resolved
Hide resolved
.../textcomposer/impl/src/main/kotlin/io/element/android/libraries/textcomposer/TextComposer.kt
Outdated
Show resolved
Hide resolved
@@ -67,6 +69,7 @@ class VoiceRecorderImpl @Inject constructor( | |||
private var audioReader: AudioReader? = null | |||
private var recordingJob: Job? = null | |||
private val levels: MutableList<Float> = mutableListOf() | |||
private val lock = Mutex() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was there some concurrency thing that led to crashes before?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes a concurrency modification exception when calling toPersisentList() at a later point, as I was returning a list that was being modified, so some safety measures.
…/element-x-android into langleyd/live_waveform
Kudos, SonarCloud Quality Gate passed! |
Type of change
Content
Show a live waveform when the user is recording a voice message
Screenshots / GIFs
1000000006.mp4