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

Fix FlacStreamMetadata compile failure

Browse Source 
Comment concatenation was accidentally not added previously, which
results in a testing failure.
This commit is contained in:
OxygenCobalt 2022-01-17 10:34:49 -07:00
parent fc08f866ce
commit a4682cbff5
No known key found for this signature in database
GPG Key ID: 37DBE3621FE9AD47
1 changed files with 17 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
library/extractor/src/main/java/com/google/android/exoplayer2/extractor/FlacStreamMetadata.java
View File

@ -143,8 +143,7 @@ public final class FlacStreamMetadata {
bitsPerSample,
totalSamples,
/* seekTable= */ null,
new Metadata(pictureFrames)
.copyWithAppendedEntriesFrom(VorbisUtil.parseVorbisComments(vorbisComments)));
concatenateVorbisMetadata(vorbisComments, pictureFrames));
}
private FlacStreamMetadata(
@ -250,6 +249,22 @@ public final class FlacStreamMetadata {
return metadata == null ? other : metadata.copyWithAppendedEntriesFrom(other);
}
/** Returns */
@Nullable
private static Metadata concatenateVorbisMetadata(
List<String> vorbisComments,
List<PictureFrame> pictureFrames
) {
@Nullable Metadata parsedVorbisComments = VorbisUtil.parseVorbisComments(vorbisComments);
if (parsedVorbisComments == null && pictureFrames.isEmpty()) {
return null;
}
return new Metadata(pictureFrames)
.copyWithAppendedEntriesFrom(parsedVorbisComments);
}
/** Returns a copy of {@code this} with the seek table replaced by the one given. */
public FlacStreamMetadata copyWithSeekTable(@Nullable SeekTable seekTable) {
return new FlacStreamMetadata(