Re: [cdt-patch] indexer patch (related to 68883)

[Bogdan Gheorghe <gheorghe@xxxxxxxxxx>]

Talking to Chris about it...

"Alain Magloire" <alain@xxxxxxx>
Sent by: cdt-patch-admin@xxxxxxxxxxx

11/15/2004 11:22 AM

Please respond to
cdt-patch

To	cdt-patch@xxxxxxxxxxx
cc	Bogdan Gheorghe/Ottawa/IBM@IBMCA
Subject	Re: [cdt-patch] indexer patch (related to 68883)

> 
> This patch fixes a problem where if you modify the same file repeatedly,
> the indexer eventually stops working.  Bug# 68883 was partially
related 
> to this.
> 

Bogdan,
  Could you take a look at this for CDT-2.1

Thanks.

> 
> --------------030709030703020402010108
> Content-Type: text/plain;
>  name="org.eclipse.cdt.core-indexer-patch.txt"
> Content-Transfer-Encoding: 7bit
> Content-Disposition: inline;
>  filename="org.eclipse.cdt.core-indexer-patch.txt"
> 
> Index: index/org/eclipse/cdt/internal/core/search/indexing/IndexManager.java
> ===================================================================
> RCS file: /home/tools/org.eclipse.cdt.core/index/org/eclipse/cdt/internal/core/search/indexing/IndexManager.java,v
> retrieving revision 1.35
> diff -u -r1.35 IndexManager.java
> --- index/org/eclipse/cdt/internal/core/search/indexing/IndexManager.java
               
2 Nov 2004 06:43:17 -0000            
    1.35
> +++ index/org/eclipse/cdt/internal/core/search/indexing/IndexManager.java
               
8 Nov 2004 23:36:17 -0000
> @@ -168,7 +168,7 @@
>                  
                 
               
AddCompilationUnitToIndex job = new AddCompilationUnitToIndex(resource,
indexedContainers, this);
>                  
                 
               
>                  
                 
               
//If we are in WAITING mode, we need to kick ourselves into enablement
> -                
                 
               
if (!jobSet.add(job.resource.getLocation()) &&
> +                
                 
               
if (!jobSet.add(job) &&
>                  
                 
               
                 enabledState()==ENABLED)
>                  
                 
               
                 return;
>                  
                 
               
> @@ -751,7 +751,7 @@
>                  
>                  
                 if
(job instanceof AddCompilationUnitToIndex){
>                  
                 
               
AddCompilationUnitToIndex tempJob = (AddCompilationUnitToIndex) job;
> -                
                 
               
jobSet.remove(tempJob.resource.getLocation());
> +                
                 
               
jobSet.remove(tempJob);
>                  
                 }
>                  
}
>                  
> 
> --------------030709030703020402010108--

_______________________________________________
cdt-patch mailing list
cdt-patch@xxxxxxxxxxx
http://dev.eclipse.org/mailman/listinfo/cdt-patch