Skip to content

Conversation

@marcin-osowski
Copy link

I plan to extend it more, but I'd like to send out this chunk of work first, to check that I'm doing this right.

Changes:

  1. Updated URLs to Apple documentation.
  2. Added docs
  3. Implemented the matrix field
  4. Added duration in seconds
  5. Added two new atoms (there's a lot more in the spec)
    • Media header (mdhd)
    • Handler reference (hdlr)

I plan to extend it more, but I'd like to send out this chunk of work
first, to check that I'm doing this right.

Changes:
  1) Updated URLs to Apple documentation.
  2) Added docs
  3) Implemented the matrix field
  4) Added duration in seconds
  5) Added two new atoms (there's a lot more in the spec)
      - Media header (mdhd)
      - Handler reference (hdlr)
@marcin-osowski
Copy link
Author

Friendly ping.

@marcin-osowski
Copy link
Author

Thanks for the improvements, they look good to me.

I don't fully understand the syntax in d8fce6b1fceb0695b1bb0106764e9d2927dd335c, though, but I guess it's all good.

@generalmimon
Copy link
Member

I don't fully understand the syntax in d8fce6b1fceb0695b1bb0106764e9d2927dd335c, though, but I guess it's all good.

Most of it (the doc key) is just human-readable documentation with some Markdown formatting (more precisely CommonMark) - the syntax there is not important, the content is.

The value instances are described in the User Guide, but I guess you aren't asking about them, given that you used them too.

@marcin-osowski
Copy link
Author

I see, I was puzzled by for example -orig-id, but I see that those are just keys which names start with -: https://doc.kaitai.io/user_guide.html.

Looks good to me!

Copy link
Member

@generalmimon generalmimon left a comment

Choose a reason for hiding this comment

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

I have a few small suggestions:

marcin-osowski and others added 2 commits November 1, 2023 20:54
Co-authored-by: Petr Pučil <petr.pucil@seznam.cz>
Suggestions by: Petr Pučil <petr.pucil@seznam.cz>
@marcin-osowski
Copy link
Author

Thanks for the suggestions, and the very detailed research! I updated the pull request, I hope I didn't miss any of your proposals.

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.

2 participants