Skip to content
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.

Sign up for GitHub

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

Jump to bottom

stss atom parsing #380

Merged
merged 21 commits into from
Mar 29, 2021
Merged

stss atom parsing #380

merged 21 commits into from
Mar 29, 2021

Conversation

nklhtv
Copy link
Contributor

@nklhtv nklhtv commented Mar 16, 2021 •
edited
Loading

parse stss atom implemented

@nklhtv
parse edts box implemented
bcf2ad4
@nklhtv
test for edts parsing implemented
e3645e1
@nklhtv
fix elst version comment
ec46c1a
@nklhtv
fix mediaTime type
1925ec0
@nklhtv
elst entries renamed to edits
f15e019
@nklhtv
use fractional value for media rate in tests
cb5057f
@nklhtv
parse elst entries based on elst version
6742c89
@nklhtv
parse 64 bit nums without using BigInt
3a26bb6
@nklhtv
test case added for elst atom with version 1
a608bd1
@nklhtv
Merge branch 'main' of github.com:videojs/mux.js
e73d850
@nklhtv
parse ctts atom implemented
f8a9d86
@nklhtv
test ctts atom parsing
99b57a2
@nklhtv
use only uint for ctts offsets
b47b0da
@nklhtv
parse stss atom implemented
45dea4a
@nklhtv
test stss atom parsing
c80dfdf
@nklhtv
parse version 1 of ctts implemented
93d8160
@nklhtv
test version 1 parsing of ctts
d8e6d08
@nklhtv
Merge branch 'main' of github.com:nklhrstv/mux.js into stss-atom
44b9dbf
@nklhtv
fix moov size in tests
6fa5181
@nklhtv
Merge pull request #1 from nklhrstv/stss-atom
dac67fe 
Stss atom parsing
@nklhtv
Merge branch 'main' of github.com:videojs/mux.js
6a5249f
brandonocasey
brandonocasey approved these changes Mar 16, 2021
View reviewed changes
Copy link
Contributor

@brandonocasey brandonocasey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

verified this against the spec and it looks like what I would expect.

@nklhtv
Copy link
Contributor Author

nklhtv commented Mar 19, 2021

When should i exprect to get it merged?

@brandonocasey brandonocasey merged commit 305eb4f into videojs:main Mar 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brandonocasey brandonocasey brandonocasey approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@nklhtv @brandonocasey