loliball/media3
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove a redundant TODO in Util

Browse Source 
PiperOrigin-RevId: 601820851
This commit is contained in:
jbibik 2024-01-26 11:31:33 -08:00 committed by Copybara-Service
parent 2fc5590e7a
commit eb3173aa90
2 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
libraries/common/src/main/java/androidx/media3/common/util/Util.java
View File

@ -3338,11 +3338,10 @@ public final class Util {
* <p>For example: android.media.MediaCodec.error_1 or android.media.MediaDrm.error_neg_2.
*
* @param diagnosticsInfo A string from which to parse the error code.
* @return The parser error code, or 0 if an error code could not be parsed.
* @return The parsed error code, or 0 if an error code could not be parsed.
*/
@UnstableApi
public static int getErrorCodeFromPlatformDiagnosticsInfo(@Nullable String diagnosticsInfo) {
// TODO (internal b/192337376): Change 0 for ERROR_UNKNOWN once available.
if (diagnosticsInfo == null) {
return 0;
}
@ -3354,7 +3353,7 @@ public final class Util {
String digitsSection = strings[length - 1];
boolean isNegative = length >= 3 && "neg".equals(strings[length - 2]);
try {
int errorCode = Integer.parseInt(Assertions.checkNotNull(digitsSection));
int errorCode = Integer.parseInt(checkNotNull(digitsSection));
return isNegative ? -errorCode : errorCode;
} catch (NumberFormatException e) {
return 0;

1
libraries/common/src/test/java/androidx/media3/common/util/UtilTest.java
View File

@ -1258,7 +1258,6 @@ public class UtilTest {
@Test
public void getErrorCodeFromPlatformDiagnosticsInfo_withInvalidInput_returnsZero() {
// TODO (internal b/192337376): Change 0 for ERROR_UNKNOWN once available.
assertThat(Util.getErrorCodeFromPlatformDiagnosticsInfo("")).isEqualTo(0);
assertThat(Util.getErrorCodeFromPlatformDiagnosticsInfo("android.media.MediaDrm.empty"))
.isEqualTo(0);