Skip to content

Comments

update from origin#1861

Closed
vy12021 wants to merge 225 commits intogoogle:release-v2from
vy12021:master
Closed

update from origin#1861
vy12021 wants to merge 225 commits intogoogle:release-v2from
vy12021:master

Conversation

@vy12021
Copy link

@vy12021 vy12021 commented Sep 26, 2016

No description provided.

ojw28 and others added 30 commits February 9, 2015 18:04
- Bump version.
- Remove AC3 renderer. It's not ready.
Correctly reset ADTSreader state
- This change:

1. Extracts HlsExtractor interface from TsExtractor.
2. Adds AdtsExtractor for AAC/ADTS streams, which turned out to be
   really easy.

Selection of the ADTS extractor relies on seeing the .aac extension.
This is at least guaranteed not to break anything that works already
(since no-one is going to be using .aac as the extension for something
that's not elementary AAC/ADTS).

Issue: #209
Some extractor implementations underneath MediaExtractor require a seekTo
call after tracks are selected to ensure samples are read from the correct
position. De-duplicating logic was preventing this from happening in some
cases, causing issues like:

#301

Note that seeking all tracks a side effect of track selection sucks if
you already have one or more tracks selected, because it introduces
discontinuities to the already selected tracks. However, in general, it
*is* necessary to specify the position for the track being selected,
because the underlying extractor doesn't have enough information to know
where to start reading from. It can't determine this based on the read
positions of the already selected tracks, because the samples in these
tracks might be very sparse with respect to time.

I think a more optimal fix would be to change the SampleExtractor
interface to receive the current position as an argument to selectTrack.
For our own extractors, we'd seek the newly selected track to that
position, whilst the already enabled tracks would be left in their
current positions (if possible). For FrameworkSampleExtractor we'd
still have no choice but to call seekTo on the extractor to seek all
of the tracks. This solution ends up being more complex though, because:

- The SampleExtractor then needs a way of telling DefaultSampleSource
  which tracks were actually seeked, so that the pendingDiscontinuities
  flags can be set correctly.
- It's a weird API that requires the "current playback position to seek
  only the track being enabled"

So it may not be worth it! I think this fix is definitely good for now,
in any case.

Issue: #301
Previous regular expression for extracting codec information was wrong,
given a line that defines a variant it added information from “CODEC=“
text to the end of the line (including also information about
RESOLUTION or alternate rendition groups as part of the CODEC field).

This is not causing a functional problem (at least known by me)
although is making codecs field storing information that is not related
with the codec.
- Data needs to be unescaped before it's passed to SeiReader.
- SeiReader should loop over potentially multiple child messages.
- I also changed the sample passed to the EIA-608 renderer so that
  it's the entire sei message payload. The first 8 bytes are
  unnecessary, but it seems nicer conceptually to do it this way.

Issue: #295
1. Clear prefixFlags when a NAL unit is found.
2. continueBuffering should return true if loading is finished.
Minor release to enhance DASH mpd parsing
Fixes issue where an NPE occurs when RollingSampleBuffer is created
with a null Allocator.
drhill and others added 26 commits March 29, 2016 09:05
UdpDataSource: allow to setup socket timeout, change default timeout from infinite to 8000 milliseconds
This method is needed by FlacExtractor to release native resources.
-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=118769990
-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=118778155
-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=118921673
-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=118924287
-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=118935299
-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=118935398
-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=118936240
-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=118941196
When there is not enough bytes to read a word Libflac
keeps the left bytes in the read buffer and does a read
with a reduced length by 1 to 3 bytes. This results to
reading 8191, 1, 8191, 1, 8191... bytes.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=119048705
Fourcc vc1 metadata parse support
Add 16 bit PCM audio track detected including MS ACM for >2 channels
@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@ojw28 ojw28 closed this Sep 26, 2016
@google google locked and limited conversation to collaborators Jun 28, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.