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.

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

[CoreMedia] Update bindings to Xcode 12 Beta 4 #9307

Merged
merged 1 commit into from
Aug 7, 2020

Conversation

dalexsoto
Copy link
Member

No description provided.

@dalexsoto dalexsoto added the note-highlight Worth calling out specifically in release notes label Aug 6, 2020
@dalexsoto dalexsoto added this to the xcode12 milestone Aug 6, 2020
@@ -244,6 +244,8 @@ public enum CMVideoCodecType : uint
Mpeg4Video = 0x6d703476,
Mpeg2Video = 0x6d703276,
Mpeg1Video = 0x6d703176,
[iOS (14,0), TV (14,0), Watch (7,0), Mac (11,0)]
Copy link
Contributor

Choose a reason for hiding this comment

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

Correct me if I'm getting this wrong for this case! Don't we usually add the newest enums at the end of the list?

Copy link
Member Author

Choose a reason for hiding this comment

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

You can but since these enums have an explicit value you can put them wherever you want, I like to keep the same order as their native counterpart so it is easier to compare whenever you come back to them.

@monojenkins
Copy link
Collaborator

Build success
Build succeeded
API Diff (from stable)
ℹ️ API Diff (from PR only) (please review changes)
Generator Diff (no change)
Test run succeeded

@dalexsoto dalexsoto merged commit 8b2c1c8 into xamarin:xcode12 Aug 7, 2020
@dalexsoto dalexsoto deleted the xcode12-b4-coremedia branch August 7, 2020 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
note-highlight Worth calling out specifically in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants