Sandbox compatibility #92
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Openssl appears to be unused.
danielpunkass
pushed a commit
to danielpunkass/iMedia
that referenced
this pull request
May 19, 2017
…y takes very long on large libraries The problem is caused by a potentially very expensive calculation of the so-called „persistentResourceIdentifier“ for an IMBObject if it is in fact an IMBNodeObject (representing a library’s subnode within the browsers IKImageBrowserView). Since the persistentResoureIdentifiers sole purpose is to conveniently provide an identifier (valid across app sessions) for resources dragged into the client app and IMBNodeObjects (currently) cannot be dragged into the client app anyhow we now just return nil for IMBNodeObjects. Thanks to Pierre Bernard.
danielpunkass
added a commit
to danielpunkass/iMedia
that referenced
this pull request
May 19, 2017
* commit '550bbea1fdbe9ee7eb9b7efd84c39cbbd43c6a3f': Fix issue karelia#92, Loading of "Albums" node of Photos media library takes very long on large libraries Give user guidance on what to do when Lightroom catalog version and Lightroom version are out of step Always defer check for database version until root node population is imminent Eliminate warning about unimplemented method Defer Lightroom database version check Mitigate karelia#90, Accessing the root node of a Photos library for the first time within an app session takes very long for large libraries Fix karelia#89, Lightroom library not always accessible when running sandboxed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.