From a7032ede38a1428eb89fdbb34158b7ef77945c77 Mon Sep 17 00:00:00 2001 From: aquilescanta Date: Thu, 27 Jul 2017 06:20:24 -0700 Subject: [PATCH] Change copyRenditionsList parameters names Also instantiate the resulting list with a predicted size to minimize list resizing. ------------- Created by MOE: https://github.com/google/moe MOE_MIGRATED_REVID=163332285 --- .../source/hls/playlist/HlsMasterPlaylist.java | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/library/hls/src/main/java/com/google/android/exoplayer2/source/hls/playlist/HlsMasterPlaylist.java b/library/hls/src/main/java/com/google/android/exoplayer2/source/hls/playlist/HlsMasterPlaylist.java index 0b237e75e7..04192def9d 100644 --- a/library/hls/src/main/java/com/google/android/exoplayer2/source/hls/playlist/HlsMasterPlaylist.java +++ b/library/hls/src/main/java/com/google/android/exoplayer2/source/hls/playlist/HlsMasterPlaylist.java @@ -118,8 +118,8 @@ public final class HlsMasterPlaylist extends HlsPlaylist { * urls. */ public HlsMasterPlaylist copy(List renditionUrls) { - return new HlsMasterPlaylist(baseUri, tags, copyRenditionList(variants, renditionUrls), - copyRenditionList(audios, renditionUrls), copyRenditionList(subtitles, renditionUrls), + return new HlsMasterPlaylist(baseUri, tags, copyRenditionsList(variants, renditionUrls), + copyRenditionsList(audios, renditionUrls), copyRenditionsList(subtitles, renditionUrls), muxedAudioFormat, muxedCaptionFormats); } @@ -136,15 +136,15 @@ public final class HlsMasterPlaylist extends HlsPlaylist { emptyList, null, null); } - private static List copyRenditionList(List variants, List variantUrls) { - List copyVariants = new ArrayList<>(); - for (int i = 0; i < variants.size(); i++) { - HlsUrl variant = variants.get(i); - if (variantUrls.contains(variant.url)) { - copyVariants.add(variant); + private static List copyRenditionsList(List renditions, List urls) { + List copiedRenditions = new ArrayList<>(urls.size()); + for (int i = 0; i < renditions.size(); i++) { + HlsUrl rendition = renditions.get(i); + if (urls.contains(rendition.url)) { + copiedRenditions.add(rendition); } } - return copyVariants; + return copiedRenditions; } }