Summary: | IndexManager should not accept new jobs if the processing thread is null | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] JDT | Reporter: | Olivier Thomann <Olivier_Thomann> | ||||||
Component: | Core | Assignee: | Satyam Kandula <satyam.kandula> | ||||||
Status: | VERIFIED FIXED | QA Contact: | |||||||
Severity: | normal | ||||||||
Priority: | P3 | Flags: | Olivier_Thomann:
review+
|
||||||
Version: | 3.7 | ||||||||
Target Milestone: | 3.7 M5 | ||||||||
Hardware: | PC | ||||||||
OS: | Windows XP | ||||||||
Whiteboard: | |||||||||
Attachments: |
|
Description
Olivier Thomann
2010-10-06 13:37:20 EDT
Created attachment 185305 [details]
Proposed patch
Olivier, Please look at the patch and let me know if you are mentioning about the same thing and if so, please review!
This would throw a OperationCanceledException even if the operation was not really canceled. Why not simply adding: && this.processingThread != null inside the while condition ? Since the processingThread is nullify on shutdown, I don't think this is a big difference. Satyam, +1 for this and you can decide how you want to fix it. (In reply to comment #2) > This would throw a OperationCanceledException even if the operation was not > really canceled. > Why not simply adding: && this.processingThread != null inside the while > condition ? > Since the processingThread is nullify on shutdown, I don't think this is a big > difference. > Satyam, +1 for this and you can decide how you want to fix it. This is to ensure that client gets to know that the operation didn't complete properly for whatever reason. (In reply to comment #3) > This is to ensure that client gets to know that the operation didn't complete > properly for whatever reason. Fine for me then. Released in HEAD Created attachment 187517 [details]
Proposed fix
I wonder if it is not better to fix it that way. When no progress monitor is given, you can still end up with a hang.
This will be verified once a build with the last fix is available. Verified. |