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.

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

[backport] Clean projects in a background job when project settings' change #325

Conversation

dotta
Copy link
Member

@dotta dotta commented Feb 21, 2013

Cleaning the projects in the UI Thread was causing UI freeze. The
fix is really simple: clean the projects in a background job.

Note: The implemented solution is inspired by the Eclipse
CleanDialog component.

Fixes #1001527
(cherry picked from commit aec8edd)

Cleaning the projects in the UI Thread was causing UI freeze. The
fix is really simple: clean the projects in a background job.

Note: The implemented solution is inspired by the Eclipse
      ``CleanDialog`` component.

Fixes #1001527
(cherry picked from commit aec8edd)
@scala-jenkins
Copy link

Started jenkins job pr-validator-master-trunk at https://jenkins.scala-ide.org:8496/jenkins/job/pr-validator-master-trunk/426/

@scala-jenkins
Copy link

Started jenkins job pr-validator-master-2.9.x at https://jenkins.scala-ide.org:8496/jenkins/job/pr-validator-master-2.9.x/434/

@scala-jenkins
Copy link

jenkins job pr-validator-master-trunk: Success - https://jenkins.scala-ide.org:8496/jenkins/job/pr-validator-master-trunk/426/

@scala-jenkins
Copy link

jenkins job pr-validator-master-2.9.x: Success - https://jenkins.scala-ide.org:8496/jenkins/job/pr-validator-master-2.9.x/434/

dotta added a commit that referenced this pull request Feb 21, 2013
…ould-not-trigger-rebuild-in-ui-thread-1001527

[backport] Clean projects in a background job when project settings' change
@dotta dotta merged commit da64c1e into scala-ide:release/scala-ide-3.0.x Feb 21, 2013
@dotta dotta deleted the backport/3.0.x/project-settings-should-not-trigger-rebuild-in-ui-thread-1001527 branch February 21, 2013 15:02
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.

2 participants