Bug 75555 - QuickDiff initializes when opening an editor using the LastSaveReferenceProvider
Summary: QuickDiff initializes when opening an editor using the LastSaveReferenceProvider
Status: CLOSED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Text (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Platform-Text-Inbox CLA
QA Contact:
URL:
Whiteboard: stalebug
Keywords: performance
: 133478 (view as bug list)
Depends on:
Blocks:
 
Reported: 2004-10-04 04:12 EDT by Christof Marti CLA
Modified: 2021-09-27 14:59 EDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Christof Marti CLA 2004-10-04 04:12:23 EDT
As discussed, this could be solved by:
1. extending IQuickDiffReferenceProvider with something like: canHaveChanges(),
where initialization could be skipped if this returns false - this would also
avoid the document copies for the background job
2. returning a proxy to the input document on getReference() and redirect the
proxy on the first document change - this would avoid the additional memory
consumption of the reference document while code browsing, could add costs to
the first document change
Comment 1 Tod Creasey CLA 2005-03-07 11:57:29 EST
Adding my name to the cc list as we are now tracking performance issues more
closely. Please remove the performance keyword if this is not a performance bug.
Comment 2 Mike Wilson CLA 2005-04-25 11:16:31 EDT
Is this going to be implemented for R3.1? Note that, if new API is required, you need to get the approval 
of at least one PMC member, at this point.
Comment 3 Dani Megert CLA 2005-04-26 05:47:57 EDT
We should try to find improvements that work for all quick diff reference
providers since most people probably use a team quick diff reference provider.
Adding API that only improves the LastSaveReferenceProvider isnt' desirable.
Comment 4 Tom Hofmann CLA 2006-04-03 06:26:25 EDT
*** Bug 133478 has been marked as a duplicate of this bug. ***
Comment 5 Dani Megert CLA 2007-08-24 07:03:55 EDT
We could defer the read but we cannot get rid of it i.e. we can't use the yet unchaged document because this would result a blocked UI.
Comment 6 Eclipse Webmaster CLA 2019-09-06 15:37:36 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.
Comment 7 Eclipse Genie CLA 2021-09-27 14:59:23 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're closing this bug.

If you have further information on the current state of the bug, please add it and reopen this bug. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.