Skip to content

another fix for ide freeze #41

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
Merged
merged 1 commit into from
Apr 25, 2013
Merged

another fix for ide freeze #41

merged 1 commit into from
Apr 25, 2013

Conversation

@Haehnchen
Copy link
Owner

see #32
i think this one is a better solution. the other breaks the isCallTo method. two changes:

  • check for valid ContainerCalls with PlatformPatterns (ide looks faster now)
  • in isCallTo check for called method name before resolve (i hope i dont break the idea of this method)

the depth check can be removed, you try to remove the freeze with this?
adrienbrault added a commit that referenced this pull request Apr 25, 2013
@adrienbrault
Merge pull request #41 from Haehnchen/hotfix/ide-freeze-2
3313c1d 
another fix for ide freeze
@adrienbrault adrienbrault merged commit 3313c1d into Haehnchen:master Apr 25, 2013
@adrienbrault
Copy link
Contributor

thanks :)
@Haehnchen Haehnchen deleted the hotfix/ide-freeze-2 branch May 5, 2013 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@Haehnchen @adrienbrault