Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 319080

Summary: MarkerUpdaterImpl.UpdateMarker not cancel() friendly
Product: [Modeling] TMF Reporter: Stephane Barbey <stephane>
Component: XtextAssignee: Project Inbox <tmf.xtext-inbox>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: sebastian.zarnekow
Version: 1.0.0Flags: sebastian.zarnekow: helios+
Target Milestone: SR1   
Hardware: All   
OS: All   
Whiteboard:

Description Stephane Barbey CLA 2010-07-07 03:51:47 EDT
Shouldn't the for (Delta delta : resourceDescriptionDeltas) { ... } loop in org.eclipse.xtext.builder.builderState.MarkerUpdaterImpl.updateMarker(ResourceSet, ImmutableList<Delta>, IProgressMonitor) break when the monitor is canceled()?

There can be a large amount of deltas and addMarkers can be an expensive operation.

-Stephane
Comment 1 Sebastian Zarnekow CLA 2010-08-02 09:20:09 EDT
Fixed in HEAD.
Comment 2 Karsten Thoms CLA 2017-09-19 16:34:54 EDT
Closing bug which were set to RESOLVED before Eclipse Neon.0.