Skip to content
This repository was archived by the owner on Dec 5, 2024. It is now read-only.

Conversation

@StanleyGoldman
Copy link
Contributor

ApplicationCache calls DefaultEnvironment.Initialize then DefaultEnvironment.InitializeRepository.
Meanwhile DefaultEnvironment.Initiailize calls InitializeRepository itself.

shana
shana previously approved these changes Oct 26, 2017
unityVersion = Application.unityVersion;
}
environment.Initialize(unityVersion, extensionInstallPath.ToNPath(), unityApplication.ToNPath(), unityAssetsPath.ToNPath());
environment.InitializeRepository(!String.IsNullOrEmpty(repositoryPath) ? repositoryPath.ToNPath() : null);
Copy link
Member

Choose a reason for hiding this comment

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

The call that should be removed is the one in DefaultEnvironment and not this one

@StanleyGoldman StanleyGoldman force-pushed the fixes/application-cache-double-call branch from 3eb3760 to 60528e7 Compare October 26, 2017 16:20
@StanleyGoldman StanleyGoldman mentioned this pull request Oct 26, 2017
2 tasks
@shana shana merged commit 2f38e6a into master Oct 26, 2017
@shana shana deleted the fixes/application-cache-double-call branch October 26, 2017 16:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants