Skip to content

[mediaplayer] Update for iOS 10.3 beta 1 - #1627

Merged
VincentDondain merged 2 commits into
dotnet:xcode8.3from
VincentDondain:mediaplayer-b1
Feb 6, 2017
Merged

[mediaplayer] Update for iOS 10.3 beta 1#1627
VincentDondain merged 2 commits into
dotnet:xcode8.3from
VincentDondain:mediaplayer-b1

Conversation

@VincentDondain

Copy link
Copy Markdown
Contributor

No description provided.

@VincentDondain

Copy link
Copy Markdown
Contributor Author

For:

[Export ("insertQueueDescriptor:afterItem:")]
void InsertAfter (MPMusicPlayerQueueDescriptor queueDescriptor, [NullAllowed] MPMediaItem item);

Guess we could also go Insert but I find that it's not explicit enough.
Also we might eventually get a insertQueueDescriptor:beforeItem:

@monojenkins

Copy link
Copy Markdown
Contributor

Build failure

@dalexsoto dalexsoto left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just a really minor thing

NotSupported,
[iOS (10,1)]
Cancelled,
RequestTimedOut,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Missing [iOS (10,3)]?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@dalexsoto no, it's an error (output) enum so it's not needed. The one on Cancelled should be removed (not critical).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes that's why I didn't add it.
Didn't want to remove the other one to not add noise.

MPMusicPlayerControllerMutableQueue and MPMusicPlayerControllerQueue's headers show no trace
of NSCoding, NSSecureCoding or NSMutableCopying therefore we're skipping them.
@monojenkins

Copy link
Copy Markdown
Contributor

Build failure

@VincentDondain

Copy link
Copy Markdown
Contributor Author

Unrelated test failure:

@VincentDondain
VincentDondain merged commit 23d1dcf into dotnet:xcode8.3 Feb 6, 2017
@VincentDondain
VincentDondain deleted the mediaplayer-b1 branch February 6, 2017 21:04
spouliot pushed a commit to spouliot/xamarin-macios that referenced this pull request Mar 28, 2017
* introspection-ios

MPMusicPlayerControllerMutableQueue and MPMusicPlayerControllerQueue's headers show no trace of NSCoding, NSSecureCoding or NSMutableCopying therefore we're skipping them.
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.

6 participants