Bug 3202 - DCR - JM - Merge Java Element Deltas with Resource Deltas (1G2B60Z)
Summary: DCR - JM - Merge Java Element Deltas with Resource Deltas (1G2B60Z)
Status: RESOLVED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 2.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 2.0 M1   Edit
Assignee: Jerome Lanneluc CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2001-10-10 22:51 EDT by Darin Wright CLA
Modified: 2002-01-14 11:08 EST (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Darin Wright CLA 2001-10-10 22:51:13 EDT
DW (10/4/00 1:04:44 PM)
	Current element delta behaviour works as follows: There are two delta queues in the 
	Java Model - one for java element deltas and one for resource deltas. When it comes
	time to fire deltas, if there are java element deltas, the resource deltas are ignored
	and the java element deltas are fire. 

	What should occurr: The resource deltas and java element deltas should be merged.
	If there is a java element delta for a resource, it takes precedence over the resource
	delta. However, java element deltas should be generated for resource deltas where
	there are no "better" java element deltas.

	This has surfaced in face of hot code replace. When a working copy is saved, the
	only delta generated is for the compilation unit, and no class file delta is generated.
	Ideally I would get both the CU delta, and deltas for the class file(s) that changed.

PM (10/11/00 11:22:51 AM)
	Will have to be addressed past oct. 15.
Comment 1 Jerome Lanneluc CLA 2001-12-07 06:07:25 EST
Reworked the firing of IJavaElementDeltas by batching them.
Java model operations now only create deltas (they don't automatically fire 
them). If a java model operation modifies any resources, let the delta 
processor do the merging of the deltas and the firing. If the java model 
operation doesn't modify any resources (like the SetClasspathOperation), do the 
firing right away.