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

MIKMIDIMapping can't be saved/loaded to/from disk on iOS #2

Closed
armadsen opened this issue Jul 29, 2013 · 2 comments
Closed

MIKMIDIMapping can't be saved/loaded to/from disk on iOS #2

armadsen opened this issue Jul 29, 2013 · 2 comments
Assignees
@armadsen
Labels
enhancement

Comments

@armadsen
Copy link
Member

MIKMIDIMapping relies on NSXMLDocument, etc. for saving to/loading from disk, meaning it doesn't work on iOS. Either provide an alternate serialization strategy on iOS, or rewrite to remove the dependence on NSXMLDocument.
@armadsen armadsen self-assigned this Feb 13, 2014
@armadsen
Copy link
Member Author

Using KissXML on iOS might be the most straightforward approach since it includes drop in replacements for NSXML* from OS X.

armadsen added a commit that referenced this issue May 15, 2014
@armadsen
Issue #2: MIKMIDIMappings can now be loaded from XML on iOS.
22e426f
armadsen added a commit that referenced this issue May 16, 2014
@armadsen
Issue #2: MIKMIDIMapping and MIKMIDIMappingItem now have -XMLStringRe…
7602cc4 
…presentation methods on iOS. Using libxml2 for XML generation.
armadsen added a commit that referenced this issue May 16, 2014
@armadsen
Issue #2: Fixed bugs that caused additional attributes on mappings an…
836b4ec 
…d mapping items to fail to be parsed from XML on iOS.
armadsen added a commit that referenced this issue May 16, 2014
@armadsen
Issue #2: Fixed newly introduced compiler errors when compiling on OS X.
fed2812
armadsen added a commit that referenced this issue May 16, 2014
@armadsen
Issue #2: Enabled -[MIKMIDIMapping writeToFileAtURL:error:] on iOS.
99ebc03
@armadsen
Copy link
Member Author

This should be working fine now.

@armadsen armadsen mentioned this issue Feb 27, 2016
Closed
@armadsen armadsen mentioned this issue Feb 7, 2018
Closed
armadsen pushed a commit that referenced this issue Nov 19, 2018
@mlostekk
Merge pull request #2 from mixedinkey-opensource/master
226700d 
merge from master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@armadsen armadsen

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@armadsen