Skip to content

Add support for parsing and generating FrameRate attribute#10

Open
cnguyenWurl wants to merge 3 commits intomasterfrom
MCP_4699_Add_Support_For_FrameRate
Open

Add support for parsing and generating FrameRate attribute#10
cnguyenWurl wants to merge 3 commits intomasterfrom
MCP_4699_Add_Support_For_FrameRate

Conversation

@cnguyenWurl
Copy link
Copy Markdown

Add support for FR attribute to both reader & writer. Added new unit tests.

@cnguyenWurl cnguyenWurl self-assigned this Sep 12, 2019
@cnguyenWurl cnguyenWurl marked this pull request as ready for review September 12, 2019 20:59
Copy link
Copy Markdown

@devangvdesai devangvdesai left a comment

Choose a reason for hiding this comment

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

@cnguyenWurl - rest looks good.

Comment thread reader.go Outdated
Comment thread reader.go
Copy link
Copy Markdown

@devangvdesai devangvdesai left a comment

Choose a reason for hiding this comment

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

looks good

Copy link
Copy Markdown

@liorwurl liorwurl left a comment

Choose a reason for hiding this comment

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

Looks good to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants